Refactor sign validation tests

test/async_sign.tests.js

@@ -105,19 +105,5 @@ describe('signing a token asynchronously', function() {
         });
       });
     });
-
-    describe('secret must have a value', function(){
-      [undefined, '', 0].forEach(function(secret){
-        it('should return an error if the secret is falsy and algorithm is not set to none: ' + (typeof secret === 'string' ? '(empty string)' : secret), function(done) {
-        // This is needed since jws will not answer for falsy secrets
-          jwt.sign('string', secret, {}, function(err, token) {
-            expect(err).to.be.exist();
-            expect(err.message).to.equal('secretOrPrivateKey must have a value');
-            expect(token).to.not.exist;
-            done();
-          });
-        });
-      });
-    });
   });
 });

test/expires_format.tests.js

@@ -23,31 +23,4 @@ describe('expires option', function() {
     expect(result.exp).to.be.closeTo(Math.floor(Date.now() / 1000) + day_and_a_half_in_secs, 0.2);
   });
 
-
-  it('should throw if expires has a bad string format', function () {
-    expect(function () {
-      jwt.sign({foo: 123}, '123', { expiresIn: '1 monkey' });
-    }).to.throw(/"expiresIn" should be a number of seconds or string representing a timespan/);
-  });
-
-  it('should throw if expires is not an string or number', function () {
-    expect(function () {
-      jwt.sign({foo: 123}, '123', { expiresIn: { crazy : 213 } });
-    }).to.throw(/"expiresIn" should be a number of seconds or string representing a timespan/);
-  });
-
-  it('should throw an error if expiresIn and exp are provided', function () {
-    expect(function () {
-      jwt.sign({ foo: 123, exp: 839218392183 }, '123', { expiresIn: '5h' });
-    }).to.throw(/Bad "options.expiresIn" option the payload already has an "exp" property./);
-  });
-
-
-  it('should throw on deprecated expiresInSeconds option', function () {
-    expect(function () {
-      jwt.sign({foo: 123}, '123', { expiresInSeconds: 5 });
-    }).to.throw('"expiresInSeconds" is not allowed');
-  });
-
-
 });

test/non_object_values.tests.js

@@ -10,13 +10,6 @@ describe('non_object_values values', function() {
     expect(result).to.equal('hello');
   });
 
-  //v6 version will throw in this case:
-  it('should throw with expiresIn', function () {
-    expect(function () {
-      jwt.sign('hello', '123', { expiresIn: '12h' });
-    }).to.throw(/invalid expiresIn option for string payload/);
-  });
-
   it('should fail to validate audience when the payload is string', function () {
     var token = jwt.sign('hello', '123');
     expect(function () {