[openwhisk-wskdebug] branch master updated: remove temporary test directories after test run (#29)

[alexkli]

This is an automated email from the ASF dual-hosted git repository.

alexkli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/openwhisk-wskdebug.git


The following commit(s) were added to refs/heads/master by this push:
     new b4209ed  remove temporary test directories after test run (#29)
b4209ed is described below

commit b4209edb7c1fe87f2289d02cbc6143808bfd77e0
Author: Alexander Klimetschek <aklim...@adobe.com>
AuthorDate: Sat Mar 28 20:28:08 2020 -0700

    remove temporary test directories after test run (#29)
    
    * remove temporary test directories after test run
    * create temp dirs under build/
    * set chmod 755 permissions on temp dirs to work in Travis
---
 package-lock.json   |  6 ++++++
 package.json        |  1 +
 test/nodejs.test.js | 25 ++++++++++++++++++++-----
 3 files changed, 27 insertions(+), 5 deletions(-)

diff --git a/package-lock.json b/package-lock.json
index d3a0705..83451a2 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -529,6 +529,12 @@
             "integrity": 
"sha512-mT8iDcrh03qDGRRmoA2hmBJnxpllMR+0/0qlzjqZES6NdiWDcZkCNAk4rPFZ9Q85r27unkiNNg8ZOiwZXBHwcA==",
             "dev": true
         },
+        "chmodr": {
+            "version": "1.2.0",
+            "resolved": "https://registry.npmjs.org/chmodr/-/chmodr-1.2.0.tgz";,
+            "integrity": 
"sha512-Y5uI7Iq/Az6HgJEL6pdw7THVd7jbVOTPwsmcPOBjQL8e3N+pz872kzK5QxYGEy21iRys+iHWV0UZQXDFJo1hyA==",
+            "dev": true
+        },
         "chokidar": {
             "version": "3.3.1",
             "resolved": 
"https://registry.npmjs.org/chokidar/-/chokidar-3.3.1.tgz";,
diff --git a/package.json b/package.json
index d1af06d..1149666 100644
--- a/package.json
+++ b/package.json
@@ -51,6 +51,7 @@
         "yargs": "^15.1.0"
     },
     "devDependencies": {
+        "chmodr": "^1.2.0",
         "clone": "^2.1.2",
         "codecov": "^3.6.5",
         "eslint": "^6.8.0",
diff --git a/test/nodejs.test.js b/test/nodejs.test.js
index 1f4b419..9eb2f54 100644
--- a/test/nodejs.test.js
+++ b/test/nodejs.test.js
@@ -32,9 +32,21 @@ const test = require('./test');
 const assert = require('assert');
 const fse = require('fs-extra');
 const fs = require('fs');
-const os = require("os");
-const path = require("path");
 const sleep = require('util').promisify(setTimeout);
+const tmp = require('tmp');
+const chmodr = require('chmodr');
+
+const BUILD_DIR = "build";
+
+function makeTempDir() {
+    tmp.setGracefulCleanup();
+    fse.ensureDirSync(BUILD_DIR)
+    const tmpobj = tmp.dirSync({
+        dir: BUILD_DIR,
+        unsafeCleanup: true
+    });
+    return tmpobj.name;
+}
 
 describe('nodejs', function() {
     this.timeout(30000);
@@ -423,8 +435,9 @@ describe('nodejs', function() {
         this.timeout(10000);
 
         // create copy in temp dir so we can modify it
-        const tmpDir = path.join(os.tmpdir(), 
fs.mkdtempSync("wskdebug-test-"));
+        const tmpDir = makeTempDir();
         fse.copySync("test/nodejs/plain-flat", tmpDir);
+        chmodr.sync(tmpDir, 0o755);
         process.chdir(tmpDir);
 
         test.mockActionDoubleInvocation(
@@ -461,8 +474,9 @@ describe('nodejs', function() {
         this.timeout(10000);
 
         // create copy in temp dir so we can modify it
-        const tmpDir = path.join(os.tmpdir(), 
fs.mkdtempSync("wskdebug-test-"));
+        const tmpDir = makeTempDir();
         fse.copySync("test/nodejs/commonjs-flat", tmpDir);
+        chmodr.sync(tmpDir, 0o755);
         process.chdir(tmpDir);
 
         test.mockActionDoubleInvocation(
@@ -499,8 +513,9 @@ describe('nodejs', function() {
         this.timeout(10000);
 
         // create copy in temp dir so we can modify it
-        const tmpDir = path.join(os.tmpdir(), 
fs.mkdtempSync("wskdebug-test-"));
+        const tmpDir = makeTempDir();
         fse.copySync("test/nodejs/commonjs-deps", tmpDir);
+        chmodr.sync(tmpDir, 0o755);
         process.chdir(tmpDir);
 
         test.mockActionDoubleInvocation(