Skip to content

Update the pgaux.t test for subtests and add some new#1428

Merged
Alex-Jordan merged 3 commits into
openwebwork:PG-2.21from
pstaabp:update-pgaux-test
Jul 14, 2026
Merged

Update the pgaux.t test for subtests and add some new#1428
Alex-Jordan merged 3 commits into
openwebwork:PG-2.21from
pstaabp:update-pgaux-test

Conversation

@pstaabp

@pstaabp pstaabp commented Jun 3, 2026

Copy link
Copy Markdown
Member

Add some new rounding test that will be updated with #1422. This also switches tests to subtest for organization.

Comment thread t/macros/pgaux.t Outdated

@drgrice1 drgrice1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There is some code clean up needed, but the separation into subtests is nice.

Comment thread t/macros/pgaux.t Outdated
Comment thread t/macros/pgaux.t Outdated
Comment thread t/macros/pgaux.t Outdated
Comment thread t/macros/pgaux.t Outdated
Comment thread t/macros/pgaux.t

@Alex-Jordan Alex-Jordan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Marking for approval, but I won't merge given the unresolved comments.

@pstaabp
pstaabp force-pushed the update-pgaux-test branch from 1c64f73 to 2fb9ab8 Compare July 14, 2026 14:16
@pstaabp

pstaabp commented Jul 14, 2026

Copy link
Copy Markdown
Member Author

Updated.

@drgrice1 drgrice1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks good now.

@Alex-Jordan
Alex-Jordan merged commit dde49a1 into openwebwork:PG-2.21 Jul 14, 2026
3 checks passed
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.

3 participants