chore(tests): adapt changes

aurelia · Apr 13, 2019 · 9e37699 · 9e37699
1 parent ae7f540
commit 9e37699
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 7 deletions.
diff --git a/test/app-router.spec.ts b/test/app-router.spec.ts
@@ -156,6 +156,9 @@ describe('app-router', () => {
       router
         ._dequeueInstruction()
         .then(result => {
+          if (!result) {
+            throw new Error('Invalid instruction processing');
+          }
           expect(result.completed).toBe(true);
           expect(result.status).toBe('completed');
           expect(result.output).toBe(output);
@@ -184,6 +187,9 @@ describe('app-router', () => {
 
       router._dequeueInstruction()
         .then(result => {
+          if (!result) {
+            throw new Error('Invalid instruction processing');
+          }
           expect(result.completed).toBe(false);
           expect(result.status).toBe('canceled');
           expect(result.output).toBe(output);
@@ -212,6 +218,9 @@ describe('app-router', () => {
 
       router._dequeueInstruction()
         .then(result => {
+          if (!result) {
+            throw new Error('Invalid instruction processing');
+          }
           expect(result.completed).toBe(false);
           expect(result.status).toBe('rejected');
           expect(result.output).toBe(output);

diff --git a/test/pipeline.spec.ts b/test/pipeline.spec.ts
@@ -93,7 +93,7 @@ describe('Pipeline', function() {
     // { status, output, completed: status === pipelineStatus.completed }
     it('runs to "completed" when there is no step', async () => {
       let result = await pipeline.run(navInstruction);
-      expect(result.status).toBe(PipelineStatus.completed);
+      expect(result.status).toBe(PipelineStatus.Completed);
       expect(result.completed).toBe(true);
     });
 
@@ -108,7 +108,7 @@ describe('Pipeline', function() {
       pipeline.addStep(step);
       const result = await pipeline.run(navInstruction);
       expect(navInstruction.fragment).toBe(fragment);
-      expect(result.status).toBe(PipelineStatus.completed);
+      expect(result.status).toBe(PipelineStatus.Completed);
       expect(result.completed).toBe(true);
     });
 
@@ -144,7 +144,7 @@ describe('Pipeline', function() {
         expect(firstCalled).toBe(1);
         expect(secondCalled).toBe(1);
         expect(thirdCalled).toBe(0);
-        expect(result.status).toBe(PipelineStatus.rejected);
+        expect(result.status).toBe(PipelineStatus.Rejected);
       });
 
       it('completes with "rejected" status when a step invokes reject()', async () => {
@@ -178,7 +178,7 @@ describe('Pipeline', function() {
         expect(firstCalled).toBe(1);
         expect(secondCalled).toBe(1);
         expect(thirdCalled).toBe(0);
-        expect(result.status).toBe(PipelineStatus.rejected);
+        expect(result.status).toBe(PipelineStatus.Rejected);
         expect(result.output.toString()).toContain('Invalid abcdef ắếốộ');
       });
 
@@ -213,7 +213,7 @@ describe('Pipeline', function() {
         expect(firstCalled).toBe(1);
         expect(secondCalled).toBe(0);
         expect(thirdCalled).toBe(0);
-        expect(result.status).toBe(PipelineStatus.completed);
+        expect(result.status).toBe(PipelineStatus.Completed);
         expect(result.output.toString()).toBe(new Error('Valid ắếốộắếốộắếốộ').toString());
       });
 
@@ -248,7 +248,7 @@ describe('Pipeline', function() {
         expect(firstCalled).toBe(1);
         expect(secondCalled).toBe(0);
         expect(thirdCalled).toBe(0);
-        expect(result.status).toBe(PipelineStatus.canceled);
+        expect(result.status).toBe(PipelineStatus.Canceled);
         expect(result.output.toString()).toBe(new Error('Valid ắếốộắếốộắếốộ').toString());
       });
     });

diff --git a/test/route-loading/load-route-step.spec.ts b/test/route-loading/load-route-step.spec.ts
@@ -35,7 +35,7 @@ describe('RouteLoading -- LoadRouteStep', function() {
 
   it('without RouteLoader implementation -- wrapped in Promise', async () => {
     const result: PipelineResult = await loadRouteStep.run(navInstruction, next);
-    expect(result.status).toBe(PipelineStatus.canceled);
+    expect(result.status).toBe(PipelineStatus.Canceled);
     expect(result.output.toString()).toBe(new Error('Route loaders must implement "loadRoute(router, config, navigationInstruction)".').toString());
   });
 });