Make nodejs4.3 explicit in the npm module

DavidTanner · Apr 3, 2018 · aa576e4 · aa576e4
1 parent 81c9d6c
commit aa576e4
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/index.js b/index.js
@@ -20,7 +20,7 @@ var ENV_ARGS = [].concat.apply([], ENV_VARS.map(function(x) { return ['-e', x] }
 // Will spawn `docker run` synchronously and return stdout
 module.exports = function runSync(options) {
   options = options || {}
-  var dockerImage = options.dockerImage || 'lambci/lambda'
+  var dockerImage = options.dockerImage || 'lambci/lambda:nodejs4.3'
   var handler = options.handler || 'index.handler'
   var event = options.event || {}
   var taskDir = options.taskDir == null ? process.cwd() : options.taskDir

diff --git a/test.js b/test.js
@@ -24,7 +24,7 @@ captured.args.should.eql([
   '-v',
   __dirname + ':/var/task',
   '--rm',
-  'lambci/lambda',
+  'lambci/lambda:nodejs4.3',
   'index.handler',
   '{}',
 ])
@@ -64,7 +64,9 @@ captured.args.should.eql([
   'AWS_LAMBDA_FUNCTION_HANDLER',
   '-e',
   'AWS_LAMBDA_EVENT_BODY',
-  'lambci/lambda',
+  '-e',
+  'DOCKER_LAMBDA_USE_STDIN',
+  'lambci/lambda:nodejs4.3',
   'index.handler',
   '{}',
 ])