POST /job can now accept an array of jobs

[ItalyPaleAle]

POST /job can now accept an array of jobs. The format is as you would expect:

$ curl -H "Content-Type: application/json" -X POST -d     '[{
       "type": "email",
       "data": {
         "title": "welcome email for tj",
         "to": "tj@learnboost.com",
         "template": "welcome-email"
       },
       "options" : {
         "attempts": 5,
         "priority": "high"
       }
     },
     {
       "type": "",
       "data": {
         "title": "welcome email for tj",
         "to": "tj@learnboost.com",
         "template": "welcome-email"
       },
       "options" : {
         "attempts": 5,
         "priority": "high"
       }
     }]' http://localhost:3000/job

# Response: (status code: 400)
[
  {
    "message": "job created",
    "id": 772
  },
  {
    "error": "Must provide job type"
  }
]

Some notes:

1. If only an object is provided, the behaviour is exactly like before
2. If an array is provided, the response is an array with the response from each job creation, in the same order as the request array.
3. In case of error, previous successes are reported. The last element of the array is an error object, and no more jobs are scheduled.

[behrad]

thank you @EgoAleSum
I'll merge this into 0.9.0

[ItalyPaleAle]

Thanks to you!
PS: why the "Duplicate" label?

[ItalyPaleAle]

PS2 could you also consider #497 ?

[ItalyPaleAle]

Oh ok, I found out about #334.
I still believe that my PR has some advantages, though, as it still returns the ids of successfully created jobs in case of error. Additionally, it doesn't depend on async (I made sure not to add more NPM modules).

[behrad]

thats ok since I'd let in lodash before... I'll take advantage of less complicated code for this

[ItalyPaleAle]

Ok, I'd still ask them if it's possible to include the IDs of successfully scheduled jobs. That's very important for us, for example, as I'll be using this code to schedule 10 future tasks which I need to be able to cancel before they're executed.

[behrad]

we can bring in this test https://github.com/LearnBoost/kue/pull/334/files#r25875077

[ItalyPaleAle]

Thanks for the merge :) I'll port the test later today!