added error handling to publish(). Closes #2

package.json

@@ -1,6 +1,6 @@
 {
   "name": "pg-boss",
-  "version": "0.3.3",
+  "version": "0.3.4",
   "description": "Queueing jobs in Node.js using PostgreSQL like a boss",
   "main": "./lib/index.js",
   "dependencies": {

src/manager.js

@@ -153,6 +153,9 @@ class Manager extends EventEmitter {
 
                         name = args[0];
                         data = args[1];
+
+                        assert(typeof data != 'function', 'publish() cannot accept a function as the payload.  Did you intend to use subscribe()?');
+
                         options = args[2];
 
                     } else if(typeof args[0] == 'object'){
@@ -168,8 +171,11 @@ class Manager extends EventEmitter {
                         options = job.options;
                     }
 
-                    assert(name, 'boss requires all jobs to have a name');
                     options = options || {};
+
+                    assert(name, 'boss requires all jobs to have a name');
+                    assert(typeof options == 'object', 'options should be an object');
+
                 } catch (error){
                     return reject(error);
                 }

test/publishTest.js

@@ -24,6 +24,20 @@ describe('publish', function(){
         });
     });
 
+    it('should fail with a function for data', function(finished) {
+        boss.publish('job', () => true).catch(error => {
+            assert(true);
+            finished();
+        });
+    });
+
+    it('should fail with a function for options', function(finished) {
+        boss.publish('job', 'data', () => true).catch(error => {
+            assert(true);
+            finished();
+        });
+    });
+
     it('should accept single string argument', function(finished) {
         var jobName = 'publishNameOnly';