Skip to content

minor testscripts improvement#244

Open
umlaeute wants to merge 2 commits intomainfrom
improvement/testscripts
Open

minor testscripts improvement#244
umlaeute wants to merge 2 commits intomainfrom
improvement/testscripts

Conversation

@umlaeute
Copy link
Collaborator

@umlaeute umlaeute commented Feb 2, 2026

  • use the same interface for all test wrappers (tests/*.sh): <scriptname> <testfile-without-extension> [<mysofa2json>]

  • improve argument check, and print usage information if the test-scripts are called with the wrong args

  • consistent indentation and removed trailing whitespace

  • pass the path to mysofa2json to the tests/nocrash.sh script as well (like for the other wrappers)

  • Related: v1.3.4 test suite fails #243

also:
- use named variables
- more (and modernized) quoting
- strip trailing whitespace
- consistent indentation
@umlaeute
Copy link
Collaborator Author

umlaeute commented Feb 2, 2026

some more ideas for improving the test-scripts:

  • should usage errors always return 128? (currently, if my new argument checks fail, it will simply return 1)
  • should we use named options for passing the mysofa2json path? e.g.
    -tests/compare.sh tests/MIT_KEMAR_large_pinna path/to/mysofa2json
    +tests/compare.sh -m path/to/mysofa2json tests/MIT_KEMAR_large_pinna
    this appears to be more extendible (what if we ever want to pass more arguments)

i can implement both if you want me to.

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.

2 participants

Comments