Fix CI configuration errors (#355)

* Fix CI configuration errors * Remove downloading artifacts * Add external NPM task
Azure · Jun 3, 2019 · 8b4fc43 · 8b4fc43
1 parent 2efcc75
commit 8b4fc43
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 9 deletions.
diff --git a/.devops/azure-pipelines-ci.yml b/.devops/azure-pipelines-ci.yml
@@ -30,12 +30,12 @@ jobs:
     inputs:
       pathtoPublish: $(Build.SourcesDirectory)/dist
       artifactName: dist
-- template: azure-pipelines-job.yml
+- template: azure-pipelines-test.yml
   parameters:
     nodeVersion: '8'
-- template: azure-pipelines-job.yml
+- template: azure-pipelines-test.yml
   parameters:
     nodeVersion: '10'
-- template: azure-pipelines-job.yml
+- template: azure-pipelines-test.yml
   parameters:
     nodeVersion: '12'
diff --git a/.devops/azure-pipelines-pr.yml b/.devops/azure-pipelines-pr.yml
@@ -1,4 +1,20 @@
 jobs:
+- job: Build
+  pool:
+    vmImage: 'Ubuntu 16.04'
+  steps:
+  - task: Npm@1
+    displayName: 'npm install'
+    inputs:
+      command: custom
+      verbose: false
+      customCommand: install
+  - task: Npm@1
+    displayName: 'npm run build'
+    inputs:
+      command: custom
+      verbose: false
+      customCommand: run build
 - job: Check_Everything
   pool:
     vmImage: 'Ubuntu 16.04'

diff --git a/.devops/azure-pipelines-test.yml b/.devops/azure-pipelines-test.yml
@@ -5,7 +5,7 @@ jobs:
 - job: 'testOn${{parameters.nodeVersion}}'
   pool:
     vmImage: 'Ubuntu 16.04'
-  dependsOn: Pack
+  dependsOn: Build
   condition: succeeded()
   steps:
   - script: 'sudo apt-get update'
@@ -20,12 +20,12 @@ jobs:
     displayName: 'npm install'
     inputs:
       verbose: false
-  - task: DownloadBuildArtifacts@0
-    displayName: 'Download Build Artifacts'
+  - task: Npm@1
+    displayName: 'npm test'
     inputs:
-      artifactName: dist
-      downloadPath: $(System.DefaultWorkingDirectory)
-  - script: 'npm test'
+      command: custom
+      verbose: false
+      customCommand: test
   - task: PublishTestResults@2
     inputs:
       testResultsFiles: '$(System.DefaultWorkingDirectory)/test-results.xml'