Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: Create ingestion verification test cases with parallel processing #138

Merged
merged 1 commit into from
May 6, 2022

Conversation

adebayor123
Copy link
Member

Problem:

  • Currently smoke tests are performed manually, and in order to gain assurance on newly released web client's correct behavior, automated smoke test is required to (1) verify web client recording events and delivering to data plane successfully (2) verify the complete ingestion of recorded events.

This PR:

  • As mentioned above, there are two levels of verification. This PR contains the test cases that verify ingestion of events (separate cases by event types).

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@adebayor123 adebayor123 changed the title Smoke ingestion Smoke test - ingestion logic Apr 25, 2022
@adebayor123 adebayor123 changed the title Smoke test - ingestion logic test: Create ingestion verification test cases with parallel processing Apr 25, 2022
Comment on lines +51 to +54
"smoke:local:headless": "cross-env URL=$URL MONITOR_ID=$MONITOR ENDPOINT=$ENDPOINT NAME=$NAME npx playwright test --config=playwright.local.config.ts",
"smoke:local": "cross-env URL=$URL MONITOR_ID=$MONITOR ENDPOINT=$ENDPOINT NAME=$NAME npx playwright test --config=playwright.local.config.ts --headed",
"smoke": "cross-env URL=$URL MONITOR_ID=$MONITOR ENDPOINT=$ENDPOINT NAME=$NAME npx playwright test --headed",
"smoke:headless": "cross-env URL=$URL MONITOR_ID=$MONITOR ENDPOINT=$ENDPOINT NAME=$NAME npx playwright test"
Copy link
Contributor

@maniator maniator May 2, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

May want to rebase off main branch

@adebayor123
Copy link
Member Author

Merging to main as the last commit was a rebase on top of previously merged commit (#137)

@adebayor123 adebayor123 merged commit 9ff95b7 into aws-observability:main May 6, 2022
@adebayor123 adebayor123 deleted the smoke-ingestion branch May 11, 2022 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants