Merge pull request #66 from nodejs/srl295/issue65

add an integration test

Author: srl295

.github/workflows/test.yml

@@ -0,0 +1,31 @@
+name: Integration Test
+
+on: [push, pull_request]
+
+jobs:
+  build:
+    strategy:
+      fail-fast: false
+      matrix:
+        container:
+        # Official Node.js images.
+        - node:12
+        - node:14
+        - node:16
+        # RHEL Node.js builds are icu_small without installing the
+        # optional nodejs-full-i18n rpm.
+        - registry.access.redhat.com/ubi8/nodejs-12
+        - registry.access.redhat.com/ubi8/nodejs-14
+        # - registry.access.redhat.com/ubi8/nodejs-16
+        - node:12-slim
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v2
+    - uses: actions/checkout@v2
+      with:
+        repository: nodejs/full-icu-test
+        path: full-icu-test
+    - name: Install full-icu
+      run: docker run --rm  -v $(pwd):/usr/src/app -w /usr/src/app  ${{ matrix.container }} npm i --no-package-lock --unsafe-perm
+    - name: Test full-icu
+      run: docker run --rm  -v $(pwd):/usr/src/app -w /usr/src/app  ${{ matrix.container }} --icu-data-dir=. ./full-icu-test/test.js

full-icu-test

@@ -0,0 +1 @@
+Subproject commit 0842b63e8d0c50c319631c1a2a4ace83cd006355

install-spawn.js

@@ -8,6 +8,9 @@ const spawnSync = require('child_process').spawnSync
 
 const isglobal = process.env.npm_config_global === 'true'
 const npmrc = '.npmrc'
+if (!process.env.INIT_CWD) {
+  throw Error('INIT_CWD was not set- does not seem like we were launched from npm')
+}
 const npmrcPath = path.resolve(process.env.INIT_CWD, npmrc)
 
 // uses semver regex from https://semver.org/