braid-http 1.3.94 → 1.3.95

package/README.md

@@ -385,45 +385,23 @@ var braidify = require('braid-http').http-server
 nbraidify.enable_multiplex = true   // or false
 ```
 
-## Test Procedure
+## Testing
 
-Run tests from the command line:
+Run all tests from the command line:
 
 ```
-node test/test.js
+npm test
 ```
 
-Or run tests in a browser by starting the test server:
+Run tests in a browser (auto-opens):
 
 ```
-node test/test.js --browser
+npm run test:browser
 ```
 
-Then open https://localhost:9000 and make sure all the boxes turn green.
-
 You can also filter tests by name:
 
 ```
 node test/test.js --filter="version"
 ```
 
-For the complete browser test (including demos), use 3 terminals. In the first terminal start the demo chat server:
-
-```
-cd demos/chat
-node server.js
-```
-
-In the second terminal start the demo blog server:
-```
-cd demos/blog
-node server.js
-```
-
-And in the third terminal, start the test server:
-```
-node test/test.js --browser
-```
-
-Now open https://localhost:9000, make sure all the boxes turn green, and try out the demo chat and blog, sending a message in each.
-

package/braid-http-client.js

@@ -483,7 +483,7 @@ async function braid_fetch (url, params = {}) {
 
                 if (subscription_cb && res.ok) start_subscription(subscription_cb, subscription_error)
 
-                if (subscription_cb && res.ok && params.onSubscriptionStatus) {
+                if (params.subscribe && params.onSubscriptionStatus && res.ok) {
                     subscription_online = true
                     params.onSubscriptionStatus({online: true})
                 }

package/package.json

@@ -1,9 +1,10 @@
 {
   "name": "braid-http",
-  "version": "1.3.94",
+  "version": "1.3.95",
   "description": "An implementation of Braid-HTTP for Node.js and Browsers",
   "scripts": {
-    "test": "node test/server.js"
+    "test": "node test/test.js",
+    "test:browser": "node test/test.js --browser"
   },
   "author": "Braid Working Group",
   "repository": "braid-org/braid-http",