Skip to content

Commit 53fb3e2

Browse files
committed
wip
1 parent df165f0 commit 53fb3e2

File tree

2 files changed

+15
-16
lines changed

2 files changed

+15
-16
lines changed

.github/actions/test-provider-tfe/action.yml

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,9 @@ inputs:
5151
description: The mock run tasks URL to use for testing.
5252
required: false
5353
default: "http://testing-mocks.tfe:22180/runtasks/pass"
54+
test_name:
55+
description: Name of the test run
56+
required: true
5457

5558
runs:
5659
using: composite
@@ -76,7 +79,7 @@ runs:
7679
uses: dawidd6/action-download-artifact@09f2f74827fd3a8607589e5ad7f9398816f540fe # v3.1.4
7780
with:
7881
workflow_conclusion: success
79-
name: junit-test-summary
82+
name: junit-${{ inputs.test_name }}-summary
8083
if_no_artifact_found: warn
8184
branch: main
8285

@@ -121,6 +124,6 @@ runs:
121124
- name: Upload test artifacts
122125
uses: actions/upload-artifact@65c4c4a1ddee5b72f698fdd19549f0f0fb45cf08 # v4.6.0
123126
with:
124-
name: junit-test-summary-${{ matrix.index }}
127+
name: junit-${{ inputs.test_name }}-summary-${{ matrix.index }}
125128
path: summary.xml
126129
retention-days: 1

.github/workflows/ci.yml

Lines changed: 10 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -56,19 +56,18 @@ jobs:
5656
# which runs against all tests using the list arg
5757
# lists_tests regex is used to skip the TestAccTFESAMLSettings_omnibus test suite and any test matching TestAcc*_RunDependent for CI tests only
5858
list_tests: "[^(TestAccTFESAMLSettings_omnibus|TestAcc.*_RunDependent)]"
59+
test_name: "tests"
5960

6061
run-dependent-tests:
61-
name: run-dependent-tests
62-
needs:
63-
- tests
62+
name: run
6463
runs-on: ubuntu-latest
6564
timeout-minutes: 40
6665
strategy:
6766
fail-fast: false
6867
matrix:
6968
# If you adjust these parameters, also adjust the jrm input files on the "Merge reports" step below
70-
total: [ 6 ]
71-
index: [ 5 ]
69+
total: [ 1 ]
70+
index: [ 0 ]
7271
steps:
7372
- name: Fetch Outputs
7473
id: tflocal
@@ -97,14 +96,13 @@ jobs:
9796
# Run terminal cmd 'go help testflag' to learn more about -list flag
9897
# action.yml uses https://github.com/hashicorp-forge/go-test-split-action/blob/main/action.yml to split acceptance tests
9998
# which runs against all tests using the list arg
100-
# lists_tests regex is used to include the TestAccTFESAMLSettings_omnibus test suite and skip any test matching TestAcc*_RunDependent for CI tests only
101-
list_tests: "^(?!(?:TestAccTFESAMLSettings_omnibus$|TestAcc.*_RunDependent$)).*"
99+
# lists_tests regex is used to include the any test matching TestAcc*_RunDependent for CI tests only
100+
list_tests: "TestAcc.*_RunDependent"
101+
test_name: "run-dependent-tests"
102102

103103
tests-combine-summaries:
104104
name: Combine Test Reports
105-
needs:
106-
- tests
107-
- run-dependent-tests
105+
needs: [ tests, run-dependent-tests ]
108106
runs-on: ubuntu-latest
109107
steps:
110108
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0
@@ -118,7 +116,7 @@ jobs:
118116
run: npm install -g junit-report-merger
119117

120118
- name: Merge reports
121-
run: jrm ./ci-summary-provider.xml "junit-test-summary-0/*.xml" "junit-test-summary-1/*.xml" "junit-test-summary-2/*.xml" "junit-test-summary-3/*.xml" "junit-test-summary-4/*.xml" "junit-test-summary-5/*.xml"
119+
run: jrm ./ci-summary-provider.xml "junit-tests-summary-0/*.xml" "junit-tests-summary-1/*.xml" "junit-tests-summary-2/*.xml" "junit-tests-summary-3/*.xml" "junit-tests-summary-4/*.xml" "junit-tests-summary/*.xml" "junit-run-dependent-tests-summary-0/*.xml"
122120

123121
- name: Upload test artifacts
124122
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
@@ -128,9 +126,7 @@ jobs:
128126

129127
tests-summarize:
130128
name: Summarize Tests
131-
needs:
132-
- tests
133-
- run-dependent-tests
129+
needs: [ tests ]
134130
runs-on: ubuntu-latest
135131
if: ${{ always() }}
136132
steps:

0 commit comments

Comments
 (0)