Merge pull request #1782 from DFE-Digital/paas/fix-artifacts

Fix Artifacts
DFE-Digital · Jun 23, 2021 · f766411 · f766411
2 parents 213a065 + c5b0415
commit f766411
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -135,7 +135,7 @@ jobs:
         uses: actions/upload-artifact@v2
         with:
           name: unit_tests
-          path: ${{ github.workspace }}/out/*
+          path: ${{ github.workspace }}/out/test-report.xml
 
       - name: Fixup report file paths
         run:  sudo sed -i "s?\"/app/?\"${PWD}/?" coverage/coverage.json || true
@@ -145,7 +145,7 @@ jobs:
         uses: actions/upload-artifact@v2
         with:
           name: Code_Coverage
-          path: ${{ github.workspace }}/coverage/*
+          path: ${{ github.workspace }}/coverage/coverage.json
 
   security_tests:
     name: Security Tests
@@ -237,7 +237,7 @@ jobs:
         uses: actions/upload-artifact@v2
         with:
           name: cucumber_tests
-          path: ${{ github.workspace }}/out/*
+          path: ${{ github.workspace }}/out/test-report.xml
 
   selenium_cucumber_tests:
     name: Chrome Cucumber Tests
@@ -295,7 +295,7 @@ jobs:
         uses: actions/upload-artifact@v2
         with:
           name: selenium_cucumber_tests
-          path: ${{ github.workspace }}/out/*
+          path: ${{ github.workspace }}/out/test-report.xml
 
   sonarcloud:
     name: SonarCloud

diff --git a/terraform/paas/variables.tf b/terraform/paas/variables.tf
@@ -95,4 +95,5 @@ variable "paas_docker_image" {
 
 variable "alerts" {
   type = map(any)
+  default = {}
 }