spawn-rx 2.0.12 → 4.0.0-beta.1

package/test/asserttest.ts

@@ -1,15 +1,16 @@
-import { expect } from 'chai';
-import './support';
-
-function delay(ms: number) {
-  return new Promise((resolve) => {
-    setTimeout(resolve, ms);
-  });
-}
-
-describe('The test runner', function () {
-  it('should pass this test', async function () {
-    await delay(1000);
-    expect(true).to.be.ok;
-  });
-});
+/* eslint-disable @typescript-eslint/no-unused-expressions */
+import { expect } from "chai";
+import "./support";
+
+function delay(ms: number) {
+  return new Promise((resolve) => {
+    setTimeout(resolve, ms);
+  });
+}
+
+describe("The test runner", function () {
+  it("should pass this test", async function () {
+    await delay(1000);
+    expect(true).to.be.ok;
+  });
+});

package/test/spawn.ts

@@ -1,102 +1,139 @@
-import { expect } from 'chai';
-import './support';
-
-import { spawn, spawnPromise, spawnDetachedPromise } from '../src/index';
-
-import { Observable } from 'rxjs';
-
-const uuidRegex = /[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}/i;
-
-describe('The spawnPromise method', function() {
-  it('should return a uuid when we call uuid', async function() {
-    // NB: Since we get run via npm run test, we know that npm bins are in our
-    // PATH.
-    let result = await spawnPromise('uuid', []);
-    expect(result.match(uuidRegex)).to.be.ok;
-  });
-});
-
-describe('The spawnDetachedPromise method', function() {
-  it('should return a uuid when we call uuid', async function() {
-    // NB: Since we get run via npm run test, we know that npm bins are in our
-    // PATH.
-    let result = await spawnDetachedPromise('uuid', ['--help']);
-    expect(result.length > 10).to.be.ok;
-  });
-});
-
-function wrapSplitObservableInPromise(obs: Observable<{
-  source: any,
-  text: any
-}>): Promise<{
-  stderr: string,
-  stdout: string,
-  error: Error | undefined
-}> {
-  return new Promise((res) => {
-    let out = {stderr: '', stdout: '', error: undefined };
-
-    obs.subscribe(
-      (x) => {
-        if (x.source === 'stdout') {
-          out.stdout += x.text;
-        } else {
-          out.stderr += x.text;
-        }
-      },
-      (e) => { out.error = e; res(out); },
-      () => res(out));
-  });
-}
-
-describe('The spawn method', function() {
-  it('should return a disposable subscription', async function() {
-    // this only check the unsubscribe goes w/o error, not that the spawned process is killed
-    // (difficult to do that, maybe iterate through child processes and check ?)
-    spawn('sleep', ['2']).subscribe().unsubscribe();
-  });
-
-  it('should return split stderr in a inner tag when called with split', async function() {
-    // provide an invalid param to uuid so it complains on stderr
-    let rxSpawn: Observable<{ source: any, text: any }> = spawn('uuid', ['foo'], {split: true}) as any;
-    let result = await wrapSplitObservableInPromise(rxSpawn);
-    expect(result.stderr.length > 10).to.be.ok;
-    expect(result.stdout).to.be.empty;
-    expect(result.error).to.be.an('error');
-  });
-
-  it('should return split stdout in a inner tag when called with split', async function() {
-    let rxSpawn: Observable<{ source: any, text: any }> = spawn('uuid', [], {split: true});
-    let result = await wrapSplitObservableInPromise(rxSpawn);
-    expect(result.stdout.match(uuidRegex)).to.be.ok;
-    expect(result.stderr).to.be.empty;
-    expect(result.error).to.be.undefined;
-  });
-
-  it('should ignore stderr if options.stdio = ignore', async function() {
-    let rxSpawn: Observable<{ source: any, text: any }> = spawn('uuid', ['foo'], {split: true, stdio: [null, null, 'ignore']});
-    let result = await wrapSplitObservableInPromise(rxSpawn);
-    expect(result.stderr).to.be.empty;
-  });
-
-  it('should ignore stdout if options.stdio = inherit', async function() {
-    let rxSpawn: Observable<{ source: any, text: any }> = spawn('uuid', [], {split: true, stdio: [null, 'inherit', null]});
-    let result = await wrapSplitObservableInPromise(rxSpawn);
-    expect(result.stdout).to.be.empty;
-  });
-
-  it('should croak if stdin is provided but stdio.stdin is disabled', async function() {
-    let stdin = Observable.of('a');
-    let rxSpawn: Observable<{ source: any, text: any }>  = spawn('marked', [], {split: true, stdin: stdin, stdio: ['ignore', null, null]});
-    let result = await wrapSplitObservableInPromise(rxSpawn);
-    expect(result.error).to.be.an('error');
-  });
-
-  it('should subscribe to provided stdin', async function() {
-    let stdin = Observable.of('a');
-    let rxSpawn: Observable<{ source: any, text: any }> = spawn('marked', [], {split: true, stdin: stdin});
-    let result = await wrapSplitObservableInPromise(rxSpawn);
-    expect(result.stdout.trim()).to.be.equal('<p>a</p>');
-  });
-
-});
+/* eslint-disable @typescript-eslint/no-explicit-any */
+/* eslint-disable @typescript-eslint/no-unused-expressions */
+import { expect } from "chai";
+import "./support";
+
+import { spawn, spawnPromise, spawnDetachedPromise } from "../src/index";
+
+import type { Observable } from "rxjs";
+import { of } from "rxjs";
+
+const uuidRegex =
+  /[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}/i;
+
+describe("The spawnPromise method", function () {
+  it("should return a uuid when we call uuid", async function () {
+    // NB: Since we get run via npm run test, we know that npm bins are in our
+    // PATH.
+    const result = await spawnPromise("uuid", []);
+    expect(result.match(uuidRegex)).to.be.ok;
+  });
+});
+
+describe("The spawnDetachedPromise method", function () {
+  it("should return a uuid when we call uuid", async function () {
+    // NB: Since we get run via npm run test, we know that npm bins are in our
+    // PATH.
+    const result = await spawnDetachedPromise("uuid", ["--help"]);
+    expect(result.length > 10).to.be.ok;
+  });
+});
+
+function wrapSplitObservableInPromise(
+  obs: Observable<{
+    source: any;
+    text: any;
+  }>,
+): Promise<{
+  stderr: string;
+  stdout: string;
+  error: Error | undefined;
+}> {
+  return new Promise((res) => {
+    const out = { stderr: "", stdout: "", error: undefined };
+
+    obs.subscribe(
+      (x) => {
+        if (x.source === "stdout") {
+          out.stdout += x.text;
+        } else {
+          out.stderr += x.text;
+        }
+      },
+      (e) => {
+        out.error = e;
+        res(out);
+      },
+      () => res(out),
+    );
+  });
+}
+
+describe("The spawn method", function () {
+  it("should return a disposable subscription", async function () {
+    // this only check the unsubscribe goes w/o error, not that the spawned process is killed
+    // (difficult to do that, maybe iterate through child processes and check ?)
+    spawn("sleep", ["2"]).subscribe().unsubscribe();
+  });
+
+  it("should return split stderr in a inner tag when called with split", async function () {
+    // provide an invalid param to uuid so it complains on stderr
+    const rxSpawn: Observable<{ source: any; text: any }> = spawn(
+      "uuid",
+      ["foo"],
+      { split: true },
+    ) as any;
+    const result = await wrapSplitObservableInPromise(rxSpawn);
+    expect(result.stderr.length > 10).to.be.ok;
+    expect(result.stdout).to.be.empty;
+    expect(result.error).to.be.an("error");
+  });
+
+  it("should return split stdout in a inner tag when called with split", async function () {
+    const rxSpawn: Observable<{ source: any; text: any }> = spawn("uuid", [], {
+      split: true,
+    });
+    const result = await wrapSplitObservableInPromise(rxSpawn);
+    expect(result.stdout.match(uuidRegex)).to.be.ok;
+    expect(result.stderr).to.be.empty;
+    expect(result.error).to.be.undefined;
+  });
+
+  it("should ignore stderr if options.stdio = ignore", async function () {
+    const rxSpawn: Observable<{ source: any; text: any }> = spawn(
+      "uuid",
+      ["foo"],
+      { split: true, stdio: [null, null, "ignore"] },
+    );
+    const result = await wrapSplitObservableInPromise(rxSpawn);
+    expect(result.stderr).to.be.empty;
+  });
+
+  it("should ignore stdout if options.stdio = inherit", async function () {
+    const rxSpawn: Observable<{ source: any; text: any }> = spawn("uuid", [], {
+      split: true,
+      stdio: [null, "inherit", null],
+    });
+    const result = await wrapSplitObservableInPromise(rxSpawn);
+    expect(result.stdout).to.be.empty;
+  });
+
+  it("should croak if stdin is provided but stdio.stdin is disabled", async function () {
+    const stdin = of("a");
+    const rxSpawn: Observable<{ source: any; text: any }> = spawn(
+      "marked",
+      [],
+      {
+        split: true,
+        stdin: stdin,
+        stdio: ["ignore", null, null],
+      },
+    );
+    const result = await wrapSplitObservableInPromise(rxSpawn);
+    expect(result.error).to.be.an("error");
+  });
+
+  it("should subscribe to provided stdin", async function () {
+    const stdin = of("a");
+    const rxSpawn: Observable<{ source: any; text: any }> = spawn(
+      "marked",
+      [],
+      {
+        split: true,
+        stdin: stdin,
+      },
+    );
+    const result = await wrapSplitObservableInPromise(rxSpawn);
+    expect(result.stdout.trim()).to.be.equal("<p>a</p>");
+  });
+});

package/test/support.ts

@@ -1,14 +1,15 @@
-import * as chai from 'chai';
-import * as chaiAsPromised from 'chai-as-promised';
-
-declare const global: any;
-
-chai.should();
-chai.use(chaiAsPromised);
-
-global.chai = chai;
-global.chaiAsPromised = chaiAsPromised;
-global.expect = chai.expect;
-global.AssertionError = chai.AssertionError;
-global.assert = chai.assert;
-global.Assertion = (chai as any).Assertion; //'Assertion' is not existing?
+/* eslint-disable @typescript-eslint/no-explicit-any */
+import * as chai from "chai";
+import * as chaiAsPromised from "chai-as-promised";
+
+declare const global: any;
+
+chai.should();
+chai.use(chaiAsPromised);
+
+global.chai = chai;
+global.chaiAsPromised = chaiAsPromised;
+global.expect = chai.expect;
+global.AssertionError = chai.AssertionError;
+global.assert = chai.assert;
+global.Assertion = (chai as any).Assertion; // 'Assertion' is not existing?

package/tsconfig.json

@@ -1,29 +1,25 @@
-{
-  "compilerOptions": {
-    "removeComments": false,
-    "preserveConstEnums": true,
-    "sourceMap": true,
-    "declaration": true,
-    "noImplicitAny": true,
-    "noImplicitReturns": true,
-    "suppressImplicitAnyIndexErrors": true,
-    "strictNullChecks": true,
-    "noUnusedLocals": true,
-    "noImplicitThis": true,
-    "noUnusedParameters": true,
-    "module": "commonjs",
-    "moduleResolution": "node",
-    "pretty": true,
-    "target": "es5",
-    "outDir": "lib",
-    "lib": ["dom", "es2015"]
-  },
-  "formatCodeOptions": {
-    "indentSize": 2,
-    "tabSize": 2
-  },
-  "exclude": [
-    "node_modules",
-    "lib"
-  ]
-}
+{
+  "compilerOptions": {
+    "removeComments": false,
+    "preserveConstEnums": true,
+    "sourceMap": true,
+    "declaration": true,
+    "noImplicitAny": true,
+    "noImplicitReturns": true,
+    "strictNullChecks": true,
+    "noUnusedLocals": true,
+    "noImplicitThis": true,
+    "noUnusedParameters": true,
+    "module": "commonjs",
+    "moduleResolution": "node",
+    "pretty": true,
+    "target": "es5",
+    "outDir": "lib",
+    "lib": ["dom", "es2015"]
+  },
+  "formatCodeOptions": {
+    "indentSize": 2,
+    "tabSize": 2
+  },
+  "exclude": ["node_modules", "lib"]
+}

package/.npmignore

@@ -1,3 +0,0 @@
-vendor/jobber/Jobber
-vendor/jobber/Jobber.sln
-lib/test

package/.travis.yml

@@ -1,24 +0,0 @@
-sudo: required
-dist: trusty
-os:
-  - linux
-  - osx
-cache:
-  directories:
-    - node_modules
-notifications:
-  email: false
-language: node_js
-node_js:
-  - 7
-  - 6
-  - 4
-install:
-  - npm install
-branches:
-  only:
-    - master
-before_script:
-  - npm prune
-script:
-  - npm run build

package/appveyor.yml

@@ -1,21 +0,0 @@
-
-platform:
-  - x64
-
-cache:
-  - node_modules
-
-environment:
-  matrix:
-    - nodejs_version: "7"
-    - nodejs_version: "6"
-    - nodejs_version: "4"
-
-install:
-  - ps: Install-Product node $env:nodejs_version $env:platform
-  - npm install
-
-test_script:
-  - npm run build
-
-build: off