Skip to content

Add mathkeyboard test using random inputs #90

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
Merged
Happypig375 merged 7 commits into from
Nov 10, 2019
Merged

Add mathkeyboard test using random inputs #90

Happypig375 merged 7 commits into from
Nov 10, 2019

Conversation

@charlesroddie
Copy link
Collaborator

@charlesroddie charlesroddie commented Nov 8, 2019

Adds 100 random keypresses to a MathKeyboard and gets LaTeX and checks that there is no crash

@FoggyFinder
Copy link
Collaborator

FoggyFinder commented Nov 8, 2019

Why add another test project?
Also, why use nunit instead of xUnit?

@charlesroddie
Copy link
Collaborator Author

charlesroddie commented Nov 8, 2019

So I can use F#. You are welcome to change from nunit to xUnit if you'd like.

@charlesroddie
Copy link
Collaborator Author

charlesroddie commented Nov 8, 2019 •
edited
Loading 

System.Exception : Exeption: Object reference not set to an instance of an object. inputs: [Power; Up; Left; B; B; Power]

That's the issue you found @FoggyFinder

@FoggyFinder
Copy link
Collaborator

FoggyFinder commented Nov 8, 2019

Yep, it is: #78. I used subscript for description there but the problem is exactly the same.

@charlesroddie
Copy link
Collaborator Author

charlesroddie commented Nov 8, 2019

It gets the shortest error case and tends to find [Power; Left; Power] now, again the same error.

@Happypig375 Happypig375 added the Type/Housekeeping This includes internal only changes. label Nov 8, 2019
@Happypig375 Happypig375 added the Resolution/Implemented The described enhancement or housekeeping work has been implemented. label Nov 10, 2019
@Happypig375 Happypig375 merged commit 3b3bfca into master Nov 10, 2019
@Happypig375 Happypig375 mentioned this pull request Dec 19, 2019
Closed
@Happypig375 Happypig375 deleted the AddRandomTestForMathKeyboard2 branch March 20, 2020 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Resolution/Implemented The described enhancement or housekeeping work has been implemented. Type/Housekeeping This includes internal only changes.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@charlesroddie @FoggyFinder @Happypig375