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.

Sign up for GitHub

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

Jump to bottom

add test to cover a case in didLoseStock #63

Merged

vinceau merged 3 commits into project-slippi:master from DustinAlandzes:da/april152021/cover-case-in-didLoseStock

Apr 16, 2021

Contributor

DustinAlandzes commented Apr 15, 2021 •

edited

Loading

noticed that this is uncovered
https://coveralls.io/builds/38798115/source?filename=src%2Fstats%2Fcommon.ts#L187


          cover a case in didLoseStock

DustinAlandzes changed the title ~~cover a case in didLoseStock~~ add test to cover a case in didLoseStock


          tests: merge undefined and null tests

28e36a4

vinceau approved these changes

View reviewed changes


          Merge branch 'master' into da/april152021/cover-case-in-didLoseStock

6c6a6f8

vinceau merged commit 5970718 into project-slippi:master

DustinAlandzes deleted the da/april152021/cover-case-in-didLoseStock branch

April 16, 2021 23:28

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment