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.

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

Closes #2688: operator_test.py conversion for new test framework #2693

Merged

Conversation

stress-tess
Copy link
Member

This PR (closes #2688) converts operator_test.py to new test framework

Copy link
Contributor

@Ethan-DeBandi99 Ethan-DeBandi99 left a comment

Choose a reason for hiding this comment

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

Few things to look at, but overall looks good.

PROTO_tests/tests/operator_test.py Show resolved Hide resolved
PROTO_tests/tests/operator_test.py Show resolved Hide resolved
PROTO_tests/tests/operator_test.py Show resolved Hide resolved
PROTO_tests/tests/operator_test.py Show resolved Hide resolved
PROTO_tests/tests/operator_test.py Show resolved Hide resolved
PROTO_tests/tests/operator_test.py Outdated Show resolved Hide resolved
PROTO_tests/tests/operator_test.py Outdated Show resolved Hide resolved
Pierce Hayes added 2 commits August 21, 2023 17:35
…amework

This PR (closes Bears-R-Us#2688) converts `operator_test.py` to new test framework
@stress-tess stress-tess force-pushed the 2688_operator_test_conversion branch from 8f42c9a to b3404c3 Compare August 21, 2023 21:51
@Ethan-DeBandi99 Ethan-DeBandi99 added this pull request to the merge queue Aug 22, 2023
Merged via the queue into Bears-R-Us:master with commit 7174ea1 Aug 22, 2023
@stress-tess stress-tess deleted the 2688_operator_test_conversion branch August 23, 2023 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

operator_test.py conversion to new framework
3 participants