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

Opportunities and Priorities for Comprehensive Test Plan #18808

Closed
3 tasks
Tracked by #18396 ...
gracekretschmer-metrostar opened this issue Jul 31, 2024 · 4 comments
Closed
3 tasks
Tracked by #18396 ...
Assignees
Labels
CMS Team CMS Product team that manages both editor exp and devops

Comments

@gracekretschmer-metrostar
Copy link

gracekretschmer-metrostar commented Jul 31, 2024

User Story or Problem Statement

The CMS team needs to determine the priorities for filling in the testing gaps for critical pathways within CMS and how to implement the comprehensive test plan to move from current state of testing to ideal state of testing.

Description or Additional Context

The following opportunities have been identified by @keisterj-oddball:

  • Collaborate with CMS and sitewide teams to create test scripts that missing for critical pathways.
  • Separate test runs to make improve the performance of test scripts.
  • Expand logging for test scripts.

Reference Links

Steps for Implementation

  • Document the opportunities for filling the gaps with the comprehensive test plan.
  • Meet with CMS PO, PM, and tech lead to determine priorities.
  • Create a plan of action in GitHub to implement opportunities that will be prioritized into future sprints.

Acceptance Criteria

  • Determine the priorities for implementing the opportunities.
  • Determine sequencing for the implementing the opportunities.
  • Determine Jon's role versus the developer's role in test creation.
@keisterj-oddball
Copy link
Collaborator

PRIORITY 1: Increase critical path test coverage

  • Currently, only 12 of 53 content types are covered by E2E tests proving their ability to be created and published by properly provisioned users.

Pros:

  • Increasing critical path test coverage will help ensure the most important functionality in the CMS works as intended after each new change is introduced.

  • Reduces the need for manual testing in cases requiring full regression testing efforts.

Considerations:

  • Adding 41 tests to the "unsegmented" test suite may significantly impact the duration of test runs (impacts developer speed and build pipelines).

  • Requires prioritization of content types to be automated.

  • Likely requires Sitewide involvement for test creation.

PRIORITY 2: Separate existing test suite into "critical path" and "regression" test runs

Pros:

  • Prioritizing the "critical path" test run could speed up the build pipeline.

  • Regression could still be run on a regular schedule outside of the CMS pipeline providing frequent feedback on new changes.

  • Developers could optionally run full regression or critical path in PR environments based on the risk of changes being introduced.

Considerations:

  • This would require some input from Sitewide about where existing tests belong.

  • Running only "critical path" tests in CI without increasing critical path coverage reduces the amount of test coverage for new changes in the short term, although the value provided by some of the existing tests could be debated.

  • Could likely be completed internally by the CMS team.

PRIORITY 3: Persist test results outside of GH and Jenkins

Pros:

  • This will assist in the assessment of test result trends and identification of flaky tests.

  • This will enable the CMS team to identify common types of test failures and develop run books on how to address them.

Considerations:

  • We want to be thoughtful on which test runs to persist results for, as well as retention periods due to storage constraints.

@keisterj-oddball
Copy link
Collaborator

I will create and link a separate ticket for each priority listed above before closing this ticket.

@keisterj-oddball
Copy link
Collaborator

#19002, #19004, and #19006 linked above in Reference Links.

@gracekretschmer-metrostar
Copy link
Author

Note to self: remove from sprint 18.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CMS Team CMS Product team that manages both editor exp and devops
Projects
None yet
Development

No branches or pull requests

3 participants