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

Improve nix flake. Add test suite with bash tinytestlib library to en… #84

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pmarreck
Copy link
Contributor

…sure basic functionality works as advertised (run with test/test). Make sure test seems deterministic enough to rely upon.

NOTE: I disabled the tests asserting exitcode 0, because the commandline interface doesn't exit code 0 for some reason. But I hope this is a reasonable possible initial test suite to assert on the commandline UI.

…sure basic functionality works as advertised (run with test/test). Make sure test seems deterministic enough to rely upon.
@pmarreck
Copy link
Contributor Author

During development of this, I saw some flagging behavior (things sometimes segfaulted) but I think I found a combo of arguments that (almost) never does that anymore. I used the following bash function to repeatedly run the test with repeat 50 test/test in order to make sure things were stable:

repeat() {
  local count=$1;
  local retcodes=0;
  shift;
  cmd=($@);
  for ((i = 0; i < count; i++))
  do
    eval "${cmd[@]}";
    (( retcodes+=$? ));
  done;
  return $retcodes
}

@Aziz87
Copy link

Aziz87 commented May 12, 2023

hmm.. not very helpful update..(
i stil wait support use regex on GPU

@pmarreck
Copy link
Contributor Author

That would be cool, but I'm not sure how a test suite of commandline usage wouldn't be helpful, since there is none

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