noflo 1.4.2 → 1.5.0

package/spec/AsPromise.js

@@ -317,6 +317,44 @@ describe('asPromise interface', () => {
 INPORT=Async.IN:IN
 OUTPORT=Stream.OUT:OUT
 Async(process/Async) OUT -> IN Stream(process/Streamify)\
+`, (err, g) => {
+        if (err) {
+          done(err);
+          return;
+        }
+        graph = g;
+        wrapped = noflo.asPromise(graph, {
+          loader,
+          asyncDelivery: true,
+        });
+        done();
+      });
+    });
+    it('should execute network with input map and provide output map with streams as arrays', () => wrapped({
+      in: 'hello world',
+    })
+      .then((out) => {
+        chai.expect(out.out).to.eql([
+          ['h', 'e', 'l', 'l', 'o'],
+          ['w', 'o', 'r', 'l', 'd'],
+        ]);
+      }));
+    it('should execute network with simple input and and provide simple output with streams as arrays', () => wrapped('hello there')
+      .then((out) => {
+        chai.expect(out).to.eql([
+          ['h', 'e', 'l', 'l', 'o'],
+          ['t', 'h', 'e', 'r', 'e'],
+        ]);
+      }));
+  });
+  describe('with a graph instead of component name (synchronous)', () => {
+    let graph = null;
+    let wrapped = null;
+    before((done) => {
+      noflo.graph.loadFBP(`\
+INPORT=Async.IN:IN
+OUTPORT=Stream.OUT:OUT
+Async(process/Async) OUT -> IN Stream(process/Streamify)\
 `, (err, g) => {
         if (err) {
           done(err);

package/spec/CommonJS.cjs

@@ -0,0 +1,10 @@
+const { describe, it } = require('node:test');
+const assert = require('node:assert/strict');
+const noflo = require('../lib/NoFlo.js');
+
+describe('NoFlo loaded as CommonJS module', () => {
+  it('should be runnable', () => {
+    assert.equal(typeof noflo, 'object');
+    assert.equal(typeof noflo.Component, 'function');
+  });
+});

package/spec/ComponentLoader.js

@@ -378,7 +378,7 @@ describe('ComponentLoader with no external packages installed', () => {
         }
         chai.expect(component).to.be.an('object');
         chai.expect(component.code).to.be.a('string');
-        chai.expect(component.code.indexOf('noflo.Component')).to.not.equal(-1);
+        chai.expect(component.code.indexOf('Component')).to.not.equal(-1);
         chai.expect(component.code.indexOf('exports.getComponent')).to.not.equal(-1);
         chai.expect(component.name).to.equal('Graph');
         chai.expect(component.library).to.equal('');
@@ -444,7 +444,7 @@ describe('ComponentLoader with no external packages installed', () => {
         }
         chai.expect(component).to.be.an('object');
         chai.expect(component.code).to.be.a('string');
-        chai.expect(component.code.indexOf('noflo.Component')).to.not.equal(-1);
+        chai.expect(component.code.indexOf('Component')).to.not.equal(-1);
         chai.expect(component.code.indexOf('exports.getComponent')).to.not.equal(-1);
         chai.expect(component.name).to.equal('Graph');
         chai.expect(component.library).to.equal('');
@@ -806,42 +806,6 @@ exports.getComponent = function() {
           });
         });
       });
-      describe('with deprecated process callback', () => {
-        let nonWorkingSource = `\
-var noflo = require('noflo');
-exports.getComponent = function() {
-  var c = new noflo.Component();
-
-  c.inPorts.add('in', {
-    process: function(packet, outPorts) {
-      if (packet.event !== 'data') {
-        return;
-      }
-      // Do something with the packet, then
-      c.outPorts.out.send(packet.data);
-    }
-  });
-
-  c.outPorts.add('out');
-
-  return c;
-};`;
-
-        it('should be able to set the source', (done) => {
-          if (!noflo.isBrowser()) {
-            nonWorkingSource = nonWorkingSource.replace("'noflo'", localNofloPath);
-          }
-          l.setSource('foo', 'NotWorkingProcess', nonWorkingSource, 'js', done);
-        });
-        it('should not be a loadable component', (done) => {
-          l.load('foo/NotWorkingProcess', (err, inst) => {
-            chai.expect(err).to.be.an('error');
-            chai.expect(err.message).to.contain('process callback is deprecated');
-            chai.expect(inst).to.be.an('undefined');
-            done();
-          });
-        });
-      });
     });
   });
 });

package/spec/ESModule.mjs

@@ -0,0 +1,11 @@
+import { describe, it } from 'node:test';
+import assert from 'node:assert/strict';
+import * as noflo from '../src/lib/NoFlo.js';
+
+describe('NoFlo loaded as ES module', () => {
+  it('should be runnable', () => {
+    assert.equal(typeof noflo, 'object');
+    assert.equal(typeof noflo.Component, 'function');
+  });
+});
+

package/spec/Network.js

@@ -41,17 +41,18 @@ describe('NoFlo Network', () => {
   describe('with an empty graph', () => {
     let g = null;
     let n = null;
-    before((done) => {
+    before(() => {
       g = new noflo.Graph();
-      noflo.createNetwork(g, {
+      return noflo.createNetwork(g, {
         subscribeGraph: false,
+        asyncDelivery: true,
         delay: true,
         baseDir,
       })
         .then((network) => {
           n = network;
-          n.connect(done);
-        }, done);
+          return n.connect();
+        });
     });
     it('should initially be marked as stopped', () => {
       chai.expect(n.isStarted()).to.equal(false);
@@ -91,16 +92,14 @@ describe('NoFlo Network', () => {
       chai.expect(n.uptime()).to.be.at.least(0);
     });
     describe('with new node', () => {
-      it('should contain the node', (done) => {
-        n.addNode({
+      it('should contain the node', () => n
+        .addNode({
           id: 'Graph',
           component: 'Graph',
           metadata: {
             foo: 'Bar',
           },
-        },
-        done);
-      });
+        }));
       it('should have registered the node with the graph', () => {
         const node = g.getNode('Graph');
         chai.expect(node).to.be.an('object');
@@ -223,6 +222,7 @@ describe('NoFlo Network', () => {
       g.addInitial('Foo', 'Merge', 'in');
       return noflo.createNetwork(g, {
         subscribeGraph: false,
+        asyncDelivery: true,
         delay: true,
         baseDir,
       })
@@ -377,6 +377,7 @@ describe('NoFlo Network', () => {
             );
             return noflo.createNetwork(graph, {
               subscribeGraph: false,
+              asyncDelivery: true,
               delay: false,
               componentLoader: n.loader,
             });
@@ -428,6 +429,7 @@ describe('NoFlo Network', () => {
       };
       noflo.createNetwork(g, {
         subscribeGraph: false,
+        asyncDelivery: true,
         delay: true,
         baseDir,
       })
@@ -450,6 +452,7 @@ describe('NoFlo Network', () => {
       g.addInitial('from-edge', 'Merge', 'in');
       noflo.createNetwork(g, {
         subscribeGraph: false,
+        asyncDelivery: true,
         delay: true,
         baseDir,
       })
@@ -471,6 +474,7 @@ describe('NoFlo Network', () => {
       g.addInitial('from-IIP', 'Def', 'in');
       noflo.createNetwork(g, {
         subscribeGraph: false,
+        asyncDelivery: true,
         delay: true,
         baseDir,
       })
@@ -505,6 +509,7 @@ describe('NoFlo Network', () => {
         noflo.createNetwork(g, {
           delay: true,
           subscribeGraph: false,
+          asyncDelivery: true,
           baseDir,
         })
           .then((nw) => {
@@ -583,6 +588,11 @@ describe('NoFlo Network', () => {
   });
   describe('with a very large network', () => {
     it('should be able to connect without errors', function (done) {
+      if (noflo.isBrowser()) {
+        // Async mode is too much for Puppeteer here
+        this.skip();
+        return;
+      }
       let n;
       this.timeout(100000);
       const g = new noflo.Graph();
@@ -596,8 +606,7 @@ describe('NoFlo Network', () => {
       }
       g.addInitial(() => {
         called++;
-      },
-      'Callback', 'callback');
+      }, 'Callback', 'callback');
       for (n = 0; n <= 10000; n++) {
         g.addInitial(n, `Repeat${n}`, 'in');
       }
@@ -605,6 +614,7 @@ describe('NoFlo Network', () => {
       noflo.createNetwork(g, {
         delay: true,
         subscribeGraph: false,
+        asyncDelivery: true,
         baseDir,
       })
         .then((nw) => {
@@ -637,6 +647,7 @@ describe('NoFlo Network', () => {
       return noflo.createNetwork(g, {
         delay: true,
         subscribeGraph: false,
+        asyncDelivery: true,
         componentLoader: loader,
       })
         .then((nw) => nw.connect()
@@ -656,6 +667,7 @@ describe('NoFlo Network', () => {
       return noflo.createNetwork(g, {
         delay: true,
         subscribeGraph: false,
+        asyncDelivery: true,
         componentLoader: loader,
       })
         .then((nw) => nw.connect()
@@ -675,6 +687,7 @@ describe('NoFlo Network', () => {
       return noflo.createNetwork(g, {
         delay: true,
         subscribeGraph: false,
+        asyncDelivery: true,
         componentLoader: loader,
       })
         .then((nw) => nw.connect()
@@ -695,6 +708,7 @@ describe('NoFlo Network', () => {
       return noflo.createNetwork(g, {
         delay: true,
         subscribeGraph: false,
+        asyncDelivery: true,
         componentLoader: loader,
       })
         .then((nw) => nw.connect()
@@ -715,6 +729,7 @@ describe('NoFlo Network', () => {
       return noflo.createNetwork(g, {
         delay: true,
         subscribeGraph: false,
+        asyncDelivery: true,
         componentLoader: loader,
       })
         .then((nw) => nw.connect()
@@ -732,6 +747,7 @@ describe('NoFlo Network', () => {
       return noflo.createNetwork(g, {
         delay: true,
         subscribeGraph: false,
+        asyncDelivery: true,
         componentLoader: loader,
       })
         .then((nw) => nw.connect())
@@ -759,6 +775,7 @@ describe('NoFlo Network', () => {
       return noflo.createNetwork(g, {
         delay: true,
         subscribeGraph: false,
+        asyncDelivery: true,
         componentLoader: loader,
       })
         .then((nw) => nw.connect())
@@ -788,6 +805,7 @@ describe('NoFlo Network', () => {
       return noflo.createNetwork(g, {
         delay: true,
         subscribeGraph: false,
+        asyncDelivery: true,
       })
         .then((nw) => {
           chai.expect(nw.baseDir).to.equal(baseDir);
@@ -802,6 +820,7 @@ describe('NoFlo Network', () => {
       return noflo.createNetwork(g, {
         delay: true,
         subscribeGraph: false,
+        asyncDelivery: true,
       })
         .then((nw) => {
           chai.expect(nw.baseDir).to.equal(process.cwd());
@@ -812,6 +831,7 @@ describe('NoFlo Network', () => {
       return noflo.createNetwork(g, {
         delay: true,
         subscribeGraph: false,
+        asyncDelivery: true,
         baseDir,
       })
         .then((nw) => {
@@ -827,6 +847,7 @@ describe('NoFlo Network', () => {
       g = new noflo.Graph();
       return noflo.createNetwork(g, {
         subscribeGraph: false,
+        asyncDelivery: true,
         delay: true,
         baseDir,
       })