Skip to content

Conversation

@deads2k
Copy link
Collaborator

@deads2k deads2k commented Dec 18, 2025

This adds the ability to easily test the cosmos DB entries that result from frontend requests. It is a necessary prereq to the data migration in #3651 where the records in cosmos will change.

@deads2k deads2k force-pushed the cs-78-frontend-testing branch from 1f29f3e to 0e80430 Compare December 19, 2025 21:29
@deads2k deads2k force-pushed the cs-78-frontend-testing branch from 0e80430 to 1e56d4b Compare January 5, 2026 19:01
@deads2k deads2k changed the title [wip] new testin new testin Jan 5, 2026
@deads2k deads2k changed the title new testin Expand the step-style integration tests to make http frontend testing easier Jan 5, 2026
@deads2k deads2k force-pushed the cs-78-frontend-testing branch from 1e56d4b to 6245cb8 Compare January 6, 2026 15:30
Copy link
Collaborator

@mbarnes mbarnes left a comment

Choose a reason for hiding this comment

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

There's a lot here and I think I'm gonna have to play with this to fully grok it. Fortunately I've got a couple bugs on my plate where I think I can leverage this test framework you've built. I didn't go through 100% of the code in detail, but I think I get the gist of it and it looks good to me.

/lgtm
/approve

@openshift-ci
Copy link

openshift-ci bot commented Jan 6, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: deads2k, mbarnes

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD 0a58d4d and 2 for PR HEAD 6245cb8 in total

@openshift-ci
Copy link

openshift-ci bot commented Jan 6, 2026

@deads2k: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-parallel 6245cb8 link unknown /test e2e-parallel

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@deads2k
Copy link
Collaborator Author

deads2k commented Jan 6, 2026

/retest

@openshift-merge-bot openshift-merge-bot bot merged commit c393c2f into main Jan 6, 2026
21 of 22 checks passed
@openshift-merge-bot openshift-merge-bot bot deleted the cs-78-frontend-testing branch January 6, 2026 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants