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.

Sign up for GitHub

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

Jump to bottom

(testing): Add happy path e2e test #86

Closed
wants to merge 5 commits into from
Closed

(testing): Add happy path e2e test #86

wants to merge 5 commits into from

Conversation

everettraven
Copy link
Collaborator

Description

  • Add e2e testing infrastructure and a happy path test

Motivation

@everettraven
add e2e happy path tests
ff7407f 
Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
@everettraven
missed some files...
401d297 
Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
@everettraven
remove commented code
f13df42 
Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
@everettraven
add gh actions e2e workflow
d440c14 
Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
@everettraven
fix e2e job name
23a39b1 
Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
@everettraven
Copy link
Collaborator Author

Need to rebase

/hold

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jun 7, 2023
@everettraven
Copy link
Collaborator Author

For some reason GH didn't like that I force pushed to this branch. Opened a new pr here: #87

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Write e2e tests
1 participant
@everettraven