Skip to content

Update parameterize test#37

Merged
Ultramann merged 9 commits intomainfrom
update-parameterize-test
Apr 5, 2026
Merged

Update parameterize test#37
Ultramann merged 9 commits intomainfrom
update-parameterize-test

Conversation

@Ultramann
Copy link
Copy Markdown
Owner

@Ultramann Ultramann commented Apr 5, 2026

This PR

  1. Updates how shifu_parameterize_test works
    • removes the need to pass n_args
    • introduces a new function shifu_test_params which is used in the tests that are parameterized. It takes any number of strings before -- $@, and assigns values from $@ to variables named for those strings. Theres one magic thing it does, if the variable name starts with @ it will run _shifu_set_for_looping on the value to make the word splitting stuff work right on the inner function
  2. The completion tests were rewritten using the new parameterization system
  3. Updates how parameterized tests are reported
    • each parameterized test case now counts as one test for the purpose of counting tests in the summary
    • sub tests are reported separately, failures always being reported, and both successes and failures reported in verbose mode
    • the test summary line includes (successes/total) values instead of just percentages

@Ultramann Ultramann merged commit 3d3903a into main Apr 5, 2026
1 check passed
@Ultramann Ultramann deleted the update-parameterize-test branch April 5, 2026 20:35
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.

1 participant