chore: run CI as non-root user (#100)

Author: DominicKramer

.circleci/config.yml

@@ -76,6 +76,7 @@ jobs:
   node4:
     docker:
       - image: 'node:4'
+        user: node
     steps: &unit_tests_steps
       - checkout
       - run: &remove_package_lock
@@ -98,18 +99,22 @@ jobs:
   node6:
     docker:
       - image: 'node:6'
+        user: node
     steps: *unit_tests_steps
   node8:
     docker:
       - image: 'node:8'
+        user: node
     steps: *unit_tests_steps
   node9:
     docker:
       - image: 'node:9'
+        user: node
     steps: *unit_tests_steps
   lint:
     docker:
       - image: 'node:8'
+        user: node
     steps:
       - checkout
       - run: *remove_package_lock
@@ -118,19 +123,26 @@ jobs:
           command: |
             npm install
             npm link
+          environment:
+            NPM_CONFIG_PREFIX: /home/node/.npm-global
       - run:
           name: Link the module being tested to the samples.
           command: |
             cd samples/
             npm link @google-cloud/error-reporting
             npm install
             cd ..
+          environment:
+            NPM_CONFIG_PREFIX: /home/node/.npm-global
       - run:
           name: Run linting.
           command: npm run lint
+          environment:
+            NPM_CONFIG_PREFIX: /home/node/.npm-global
   docs:
     docker:
       - image: 'node:8'
+        user: node
     steps:
       - checkout
       - run: *remove_package_lock
@@ -143,6 +155,7 @@ jobs:
   sample_tests:
     docker:
       - image: 'node:8'
+        user: node
     steps:
       - checkout
       - run: *remove_package_lock
@@ -157,27 +170,33 @@ jobs:
           command: |
             npm install
             npm link
+          environment:
+            NPM_CONFIG_PREFIX: /home/node/.npm-global
       - run:
           name: Link the module being tested to the samples.
           command: |
             cd samples/
             npm link @google-cloud/error-reporting
             npm install
             cd ..
+          environment:
+            NPM_CONFIG_PREFIX: /home/node/.npm-global
       - run:
           name: Run sample tests.
           command: npm run samples-test
           environment:
             GCLOUD_PROJECT: long-door-651
-            GOOGLE_APPLICATION_CREDENTIALS: /var/error-reporting/.circleci/key.json
+            GOOGLE_APPLICATION_CREDENTIALS: /home/node/error-reporting/.circleci/key.json
+            NPM_CONFIG_PREFIX: /home/node/.npm-global
       - run:
           name: Remove unencrypted key.
           command: rm .circleci/key.json
           when: always
-    working_directory: /var/error-reporting/
+    working_directory: /home/node/error-reporting/
   system_tests:
     docker:
       - image: 'node:8'
+        user: node
     steps:
       - checkout
       - run: *remove_package_lock
@@ -194,16 +213,17 @@ jobs:
           name: Run system tests.
           command: npm run system-test
           environment:
-            GOOGLE_APPLICATION_CREDENTIALS: /var/error-reporting/.circleci/key.json
-            GCLOUD_TESTS_KEY: /var/error-reporting/.circleci/key.json
+            GOOGLE_APPLICATION_CREDENTIALS: /home/node/error-reporting/.circleci/key.json
+            GCLOUD_TESTS_KEY: /home/node/error-reporting/.circleci/key.json
       - run:
           name: Remove unencrypted key.
           command: rm .circleci/key.json
           when: always
-    working_directory: /var/error-reporting/
+    working_directory: /home/node/error-reporting/
   publish_npm:
     docker:
       - image: 'node:8'
+        user: node
     steps:
       - checkout
       - run:

package.json

@@ -33,7 +33,7 @@
   "scripts": {
     "docs": "node node_modules/.bin/repo-tools exec -- jsdoc -c .jsdoc.js",
     "generate-scaffolding": "node node_modules/.bin/repo-tools generate all && node node_modules/.bin/repo-tools generate lib_samples_readme -l samples/ --config ../.cloud-repo-tools.json",
-    "lint": "node node_modules/.bin/repo-tools lint --cmd eslint -- build/src/ samples/ build/system-test/ build/test/",
+    "lint": "node node_modules/.bin/repo-tools lint --cmd eslint -- src/ samples/ system-test/ test/",
     "prettier": "node node_modules/.bin/repo-tools exec -- prettier --write src/*.js src/*/*.js samples/*.js samples/*/*.js test/*.js test/*/*.js test/*/*/*.js system-test/*.js system-test/*/*.js",
     "presystem-test": "npm run compile",
     "system-test": "nyc --exclude=\"error-message.js\" mocha ./build/system-test/*.js",