Skip to content

Commit 2d6a521

Browse files
authored
[AppSignal E2E Testing] Validate E2E Tests Are Accounted For (#1126)
*Issue #, if available:* *Description of changes:* Add new validation workflow: - This validation is to ensure that all ApplicationSignals e2e test workflows relevant to this repo are actually being used in this repo. - See: https://github.com/aws-observability/aws-application-signals-test-framework/blob/main/.github/workflows/validate-e2e-tests-are-accounted-for.yml *Testing:* <img width="3138" height="1712" alt="image" src="https://github.com/user-attachments/assets/1dfab8a9-362f-4d10-b206-e9ed2aa8ac54" /> By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
1 parent b8dffa7 commit 2d6a521

File tree

1 file changed

+16
-0
lines changed

1 file changed

+16
-0
lines changed

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

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -245,3 +245,19 @@ jobs:
245245
with:
246246
aws-region: us-east-1
247247
caller-workflow-name: 'main-build'
248+
249+
# This validation is to ensure that all test workflows relevant to this repo are actually
250+
# being used in this repo, which is referring to all the other jobs in this file.
251+
#
252+
# If this starts failing, then it most likely means that new e2e test workflow was
253+
# added to `aws-observability/aws-application-signals-test-framework`, but was not
254+
# added to this file. It could also mean that a test in this file has been removed.
255+
#
256+
# If a particular test file is intended to not be tested in this repo and should not
257+
# be failing this particular validation, then choose one of the following options:
258+
# - Add the test file to the exclusions input (CSV format) to the workflow
259+
# (see: https://github.com/aws-observability/aws-application-signals-test-framework/blob/main/.github/workflows/validate-e2e-tests-are-accounted-for.yml#L1)
260+
# - Update the `validate-e2e-tests-are-accounted-for` job to change which "workflow files are expected to be used by this repo"
261+
# (see: https://github.com/aws-observability/aws-application-signals-test-framework/blob/main/.github/workflows/validate-e2e-tests-are-accounted-for.yml)
262+
validate-all-tests-are-accounted-for:
263+
uses: aws-observability/aws-application-signals-test-framework/.github/workflows/validate-e2e-tests-are-accounted-for.yml@main

0 commit comments

Comments
 (0)