fixup! Add test to check executionTimeout is capped

postmanlabs · Nov 22, 2019 · c74b3b8 · c74b3b8
1 parent f4c4535
commit c74b3b8
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/test/unit/sandbox-timeout.test.js b/test/unit/sandbox-timeout.test.js
@@ -47,11 +47,12 @@
 
     it('should set a maximum allowed timeout', function (done) {
         var options = {
-            timeout: 10 * 60 * 1000
+            timeout: 10 * 60 * 1000 // 10 minutes
         };
 
         Sandbox.createContext(options, function (err, ctx) {
-            expect(ctx.executionTimeout).to.equal(5 * 60 * 1000);
+            expect(err).to.not.exist;
+            expect(ctx.executionTimeout).to.equal(5 * 60 * 1000); // 5 minutes
             done();
         });
     });