fix: fixed limit of pop action

fix #28

Author: leaves4j

src/lib/Router.ts

@@ -143,12 +143,8 @@ export default class StackRouter<Component> extends BaseEventEmitter<RouterEvent
    * @memberof Router
    */
   public pop<T extends Partial<PopNavigationOptions>>(option?: T): void {
-    if (this.routeStack.length <= 1) return;
-    let { n = 1 } = option || {};
+    const { n = 1 } = option || {};
     const { transition } = option || {};
-    if (n > this.routeStack.length - 1) {
-      n = this.routeStack.length - 1;
-    }
     const payload: DriverPayload = { transition };
     this.driver.pop(n, payload);
   }
@@ -312,7 +308,7 @@ export default class StackRouter<Component> extends BaseEventEmitter<RouterEvent
       case RouteActionType.POP:
         const index = this.routeStack.findIndex(i => routeInfo.route.id === i.route.id);
         if (index === -1) {
-          this.routeStack.push(routeInfo);
+          this.routeStack = [routeInfo];
           this.componentChange(type, transition);
         } else {
           const destroyedIds = this.routeStack

test/lib/Router.test.ts

@@ -163,10 +163,10 @@ describe('src/lib/Router.ts', () => {
     });
   });
   describe('Router#pop', () => {
-    it('empty stack should be ok', () => {
+    it('empty stack should be ok', (done) => {
       class TestDriver extends ServerDriver {
         public pop(n: number, payload?: unknown): void {
-          assert.fail('should not trigger');
+          done()
         }
       }
       const routeManager = new RouteManager<RouteConfig<string>>();
@@ -196,26 +196,6 @@ describe('src/lib/Router.ts', () => {
       router.push('test1');
       router.pop();
     });
-    it('the wrong parameter n should be ok', () => {
-      class TestDriver extends ServerDriver {
-        public pop(n: number, payload?: unknown): void {
-          assert.equal(n, 2);
-          assert.equal((payload as any).transition, undefined);
-        }
-      }
-      const routes = [
-        {
-          path: 'test1',
-          component: 'aa'
-        }
-      ];
-      const driver = new TestDriver();
-      driver.push('/test1?a=2');
-      const router = new StackRouter({ routes }, driver);
-      driver.push('/test1?a=3');
-      driver.push('/test1?a=4');
-      router.pop({ n: 100 });
-    });
   });
   it('Router#replace');
   it('Router#popToBottom', () => {