Skip to content

Commit

Permalink
test/e2e: Fix 'resource name may not be empty' failure in catalog tem…
Browse files Browse the repository at this point in the history
…plating test case (#2457)

* test/e2e: Fix 'resource name may not be empty' failure in catalog templating test case

Update the 'adding catalog template adjusts image used' test case and
fix the issue where get requests for the test catalogsource were failing
due to the source.GetName() and source.GetNamespace() being empty, and
therefore the test catalogsource could never be updated.

Updating the test logic to avoid overwriting the `source` variable
during every poll by suppressing the variable returned appeared to be
the culprit. This was consistently reproducible on a cluster when
focusing on this single test and running the e2e suite directly.
After making these changes, I wasn't able to reproduce locally anymore.

Signed-off-by: timflannagan <timflannagan@gmail.com>

* test/e2e: Replace context.TODO references with context.Background in the catalog e2e package

Signed-off-by: timflannagan <timflannagan@gmail.com>
  • Loading branch information
timflannagan committed Nov 18, 2021
1 parent 6dc9026 commit f254e91
Showing 1 changed file with 53 additions and 47 deletions.
Loading

0 comments on commit f254e91

Please sign in to comment.