openshift / openshift/oadp-operator
E2E non-knownflake fail should print prior error
Open
Nobody has claimed this yet.
lifecycle/frozen
testing
- Dominant language
- Go
- Stars
- 92
- Forks
- 93
- Avg merge
- 1d 23h
- Merged PRs (30d)
- 43
Description
Flake fail should print why tests fail and why it's being retried from prior run instead of "No known FLAKE found in a previous run, marking test as failed."
It should save a prior error and print out during Fail() call
DescribeTable("Backup and restore applications",
func(brCase BackupRestoreCase, expectedErr error) {
if CurrentSpecReport().NumAttempts > 1 && !knownFlake {
Fail("No known FLAKE found in a previous run, marking test as failed.") // <--
}
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start from the DescribeTable callback and Fail() call shown in the issue, then trace how the prior run and knownFlake state are handled. Done means a non-known-flake retry preserves the prior error and prints why the test failed and was retried instead of only showing the generic message.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- testing-qa
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100