Skip to content

Commit

Permalink
ci: stash and unstash required files for cli tests
Browse files Browse the repository at this point in the history
  • Loading branch information
ewanharris authored and sgtcoolguy committed Oct 2, 2019
1 parent df9dbef commit 79ac7bb
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions Jenkinsfile
Original file line number Diff line number Diff line change
Expand Up @@ -171,6 +171,7 @@ def unitTests(os, nodeVersion, npmVersion, testSuiteBranch, testOnDevices) {
def cliUnitTests(nodeVersion, npmVersion) {
return {
node('git && osx') { // ToDo: refactor to try and run across mac, linux, and windows?
unstash 'cli-unit-tests'
nodejs(nodeJSInstallationName: "node ${nodeVersion}") {
ensureNPM(npmVersion)
sh 'npm ci'
Expand Down Expand Up @@ -237,6 +238,7 @@ timestamps {
if (runDanger) { // Stash files for danger.js later
stash includes: 'package.json,package-lock.json,dangerfile.js,.eslintignore,.eslintrc,npm_test.log,android/**/*.java', name: 'danger'
}
stash includes: 'package.json,package-lock.json,android/cli/**,iphone/cli/**', name: 'cli-unit-tests'
// was it a failure?
if (npmTestResult != 0) {
error readFile('npm_test.log')
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@
"engines": {
"node": ">=8"
},
"nyc" : {
"nyc": {
"exclude": [
"**/cli/tests/test-*.js"
],
Expand Down

0 comments on commit 79ac7bb

Please sign in to comment.