dnode - npm Package Compare versions

Comparing version 0.9.10 to 0.9.11

index.js

@@ -103,2 +103,6 @@ var net = require('net');
     function attachDnode() {
+        if (!stream) {
+            self.emit('error', new Error('Could not create a stream with this information ' + JSON.stringify(params)))
+            return null
+        }
         client = createClient(self.proto, stream);
@@ -105,0 +109,0 @@

package.json

 {
     "name" : "dnode",
-    "version" : "0.9.10",
+    "version" : "0.9.11",
     "description" : "freestyle RPC",
@@ -5,0 +5,0 @@ "main" : "./index.js",

test/error.js

 var dnode = require('../');
-var sys = require('sys');
 var test = require('tap').test;
@@ -102,2 +101,13 @@
     server.listen(port);
-})
+});
+
+test('bad connection string', function(t) {
+    try {
+        var client = dnode.connect('garbage', function (remote, conn) {
+            assert.fail('should have been refused, very unlikely');
+        });
+    } catch (err) {
+        t.ok(/Could not create a stream/.test(err.message));
+        t.end();
+    }
+});

test/single.js

 var dnode = require('../')
 var test = require('tap').test;
+var util = require('util');
 
@@ -14,3 +15,3 @@ test('simple', function (t) {
         print : function (n,reply) {
-            reply(sys.inspect(n));
+            reply(util.inspect(n));
         },
@@ -17,0 +18,0 @@ }).listen(port);

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

147606

increased by0.34%

Lines of code

1806

increased by0.78%

No dependency changes