Skip to content

chore(shortint): use TEST_ params as much as possible in tests#3477

Open
nsarlin-zama wants to merge 2 commits intomainfrom
ns/chore/use_test_params
Open

chore(shortint): use TEST_ params as much as possible in tests#3477
nsarlin-zama wants to merge 2 commits intomainfrom
ns/chore/use_test_params

Conversation

@nsarlin-zama
Copy link
Copy Markdown
Contributor

@nsarlin-zama nsarlin-zama commented Apr 16, 2026

closes: please link all relevant issues

PR content/description

shortint use a mix of test and default params.

I asked Claude to do a review of the params used in shortint _ci_run_filter tests before the pr:

  ┌──────────────────────┬───────┐
  │       Category       │ Count │
  ├──────────────────────┼───────┤
  │ No params            │ 9     │
  ├──────────────────────┼───────┤
  │ TEST_ only           │ 17    │
  ├──────────────────────┼───────┤
  │ PARAM_ only          │ 10    │
  ├──────────────────────┼───────┤
  │ Mixed TEST_ + PARAM_ │ 3     │
  ├──────────────────────┼───────┤
  │ Total                │ 39    │
  └──────────────────────┴───────┘

This pr standardises this and try to use TEST_ params as much as possible.

Should be merged after #3474

@cla-bot cla-bot bot added the cla-signed label Apr 16, 2026
@nsarlin-zama nsarlin-zama force-pushed the ns/chore/use_test_params branch from 81d8346 to 7537710 Compare April 16, 2026 08:49
@nsarlin-zama nsarlin-zama force-pushed the ns/chore/use_test_params branch from 7537710 to 5e64f7c Compare April 16, 2026 08:50
@nsarlin-zama nsarlin-zama changed the title fix(chore): use TEST_ params as much as possible in tests chore(shortint): use TEST_ params as much as possible in tests Apr 16, 2026
Base automatically changed from ns/fix/compact_list_without_fn to main April 20, 2026 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant