fixed issue with undefined job #92

inlife · Feb 8, 2019 · 048fd39 · 048fd39
1 parent a80eb97
commit 048fd39
Show file tree

Hide file tree

Showing 5 changed files with 16 additions and 13 deletions.
diff --git a/packages/nexrender-action-copy/index.js b/packages/nexrender-action-copy/index.js
@@ -13,7 +13,7 @@ module.exports = (job, settings, { output }, type) => {
     return new Promise(function(resolve, reject) {
         rd.on('error', reject)
         wr.on('error', reject)
-        wr.on('finish', resolve)
+        wr.on('finish', () => resolve(job))
         rd.pipe(wr);
     }).catch((error) => {
         rd.destroy()

diff --git a/packages/nexrender-action-copy/package-lock.json b/packages/nexrender-action-copy/package-lock.json
diff --git a/packages/nexrender-core/test/manual.js b/packages/nexrender-core/test/manual.js
@@ -42,14 +42,17 @@ const job = {
             value: 'Hello world',
         }
     ],
-    // actions: {
-    //     prerender: [
-    //         { module: __dirname + '/mytest.js' }
-    //     ],
-    //     postrender: [
-    //         { module: __dirname + '/mytest.js' }
-    //     ]
-    // },
+    actions: {
+        // prerender: [
+        //     { module: __dirname + '/mytest.js' }
+        // ],
+        postrender: [
+            {
+                module: '@nexrender/action-copy',
+                output: '/Users/inlife/output.mov',
+            }
+        ]
+    },
     // onChange: (job, state) => console.log('new job state', state)
 }
 

diff --git a/packages/nexrender-provider-s3/package-lock.json b/packages/nexrender-provider-s3/package-lock.json
diff --git a/packages/nexrender-types/job.js b/packages/nexrender-types/job.js
@@ -78,8 +78,8 @@ const validate = job => {
     assert(job.actions);
 
     [].concat(
-        job.actions.prerender,
-        job.actions.postrender
+        job.actions.prerender || [],
+        job.actions.postrender || []
     ).map(action => {
         assert(action, `job action must be defined`);
         assert(action.module, `job action must have module defined`);