Optimize Queue.summary query #38

grantcarthew commented 7 years ago

Looks good @thomasmodeneis. Can you fix the conflict? Pull the master branch and then do a git checkout origin/master tests/test-current.js This is a good reason to not use the test-current.js file for single module testing.

thomasmodeneis commented 7 years ago

Just did it, sorry for the delay! :) Have a great weekend. Cheers.

grantcarthew commented 7 years ago

This is fantastic work @thomasmodeneis

thomasmodeneis commented 7 years ago

@grantcarthew Thanks mate, I will be looking for other ways to continue helping out! :) Cheers.

grantcarthew commented 7 years ago

@thomasmodeneis - Checkout the README under the documentation heading. 👍

grantcarthew commented 7 years ago

@thomasmodeneis - Also mate, I just changed the test-current test runner module so it supports arguments.

Example:


npm run tc jobOptions

If you leave out the argument it gives you a list of module names.


 ==============================================
   INVALID TEST NAME!
   Use one of the test names below:
 ==============================================
   enums
   is
   datetime
   dbAssertDatabase
   dbAssertTable
   dbAssertIndex
   dbAssert
   dbDriver
   dbResult
   dbReview
   job
   jobOptions
   jobParse
   jobProgress
   jobCompleted
   jobUpdate
   jobFailed
   jobLog
   queue
   queueDb
   queueState
   queueAddJob
   queueGetJob
   queueFindJob
   queueGetNextJob
   queueReanimateJob
   queueProcess
   queueChange
   queueInterruption
   queueCancelJob
   queueRemoveJob
   queueReset
   queueStop
   queueDrop
   queueSummary
 ==============================================
   Example: npm run tc jobOptions
 ==============================================

A little cleaner.

grantcarthew / node-rethinkdb-job-queue

Optimize Queue.summary query #38 #39