Skip to content

Commit 4556edc

Browse files
authored
[AppSignal E2E Testing] Validate E2E Tests Are Accounted For (#231)
1 parent 81bfc3c commit 4556edc

File tree

1 file changed

+19
-1
lines changed

1 file changed

+19
-1
lines changed

.github/workflows/application-signals-e2e-test.yml

Lines changed: 19 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -224,4 +224,22 @@ jobs:
224224
secrets: inherit
225225
with:
226226
aws-region: us-east-1
227-
caller-workflow-name: 'main-build'
227+
caller-workflow-name: 'main-build'
228+
229+
# This validation is to ensure that all test workflows relevant to this repo are actually
230+
# being used in this repo, which is referring to all the other jobs in this file.
231+
#
232+
# If this starts failing, then it most likely means that new e2e test workflow was
233+
# added to `aws-observability/aws-application-signals-test-framework`, but was not
234+
# added to this file. It could also mean that a test in this file has been removed.
235+
#
236+
# If a particular test file is intended to not be tested in this repo and should not
237+
# be failing this particular validation, then choose one of the following options:
238+
# - Add the test file to the exclusions input (CSV format) to the workflow
239+
# (see: https://github.com/aws-observability/aws-application-signals-test-framework/blob/main/.github/workflows/validate-e2e-tests-are-accounted-for.yml#L1)
240+
# - Update the `validate-e2e-tests-are-accounted-for` job to change which "workflow files are expected to be used by this repo"
241+
# (see: https://github.com/aws-observability/aws-application-signals-test-framework/blob/main/.github/workflows/validate-e2e-tests-are-accounted-for.yml)
242+
validate-all-tests-are-accounted-for:
243+
uses: aws-observability/aws-application-signals-test-framework/.github/workflows/validate-e2e-tests-are-accounted-for.yml@main
244+
with:
245+
exclusions: dotnet-ec2-nuget-test.yml,dotnet-ec2-asg-test.yml

0 commit comments

Comments
 (0)