Fixing tests, they were expecting strings because of the issue

id's should be numbers now.
Automattic · Apr 5, 2016 · da3f2ac · da3f2ac
1 parent 20e7b98
commit da3f2ac
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/test/tdd/redis.spec.js b/test/tdd/redis.spec.js
@@ -108,7 +108,7 @@ describe('redis', function() {
       it('should strip the prefix on the id', function () {
         var client = redis.createClient();
         var id = client.stripFIFO( '03|123' );
-        id.should.equal('123');
+        id.should.equal(123);
       });
     });
 

diff --git a/test/test.coffee b/test/test.coffee
@@ -135,7 +135,7 @@ describe 'Kue Tests', ->
       jobs.process 'email-to-be-completed', (job, jdone)->
         jdone( null, { prop: 'val' } )
       jobs.on 'job complete', (id, result) ->
-        id.should.be.equal testJob.id+''
+        id.should.be.equal testJob.id
         result.prop.should.be.equal 'val'
         done()
       job_data =
@@ -154,12 +154,12 @@ describe 'Kue Tests', ->
         title: 'Test Email Job'
         to: 'tj@learnboost.com'
       jobs.on 'job failed attempt', (id, errMsg, doneAttempts) ->
-        id.should.be.equal newJob.id+''
+        id.should.be.equal newJob.id
         errMsg.should.be.equal errorMsg
         doneAttempts.should.be.equal 1
         total--
       .on 'job failed', (id, errMsg)->
-        id.should.be.equal newJob.id+''
+        id.should.be.equal newJob.id
         errMsg.should.be.equal errorMsg
         (--total).should.be.equal 0
         done()