Skip to content

Fix test_tools#63

Merged
dvalinrh merged 1 commit intomasterfrom
fix_test_tools
Mar 27, 2026
Merged

Fix test_tools#63
dvalinrh merged 1 commit intomasterfrom
fix_test_tools

Conversation

@dvalinrh
Copy link
Copy Markdown
Contributor

Description

  1. Removes duplicate code in the test_tools new code
  2. Fixes exit_out not existing

Before/After Comparison

Before: If we failed, we would get an error calling exit_out as it does not exist.
After: Error no longer occurs.

Clerical Stuff

This closes #62

Relates to JIRA: RPOPC-901

Testing done:
Verified each method works, and we get a proper message if we fail.

@dvalinrh dvalinrh requested a review from grdumas March 25, 2026 10:46
Copy link
Copy Markdown

@grdumas grdumas left a comment

Choose a reason for hiding this comment

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

LGTM

@grdumas grdumas added bug Something isn't working group_review_lgtm Indicates approval after a group review meeting labels Mar 26, 2026
@dvalinrh dvalinrh merged commit 6515f0e into master Mar 27, 2026
3 of 5 checks passed
@dvalinrh dvalinrh deleted the fix_test_tools branch March 27, 2026 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working group_review_lgtm Indicates approval after a group review meeting

Projects

None yet

Development

Successfully merging this pull request may close these issues.

When test_tools template was inserted, created issues with exit_out

2 participants