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

Add some tests #1

Closed
DyspC opened this issue Sep 23, 2020 · 5 comments
Closed

Add some tests #1

DyspC opened this issue Sep 23, 2020 · 5 comments
Assignees
Labels

Comments

@DyspC
Copy link
Owner

DyspC commented Sep 23, 2020

This is an old decompiled / imported / tweaked multiple times project with lots of complex per-character rules that make it hard to find regressions at development time.

That being said it lacks tests:

  • Some that run on excerpt constants in the code
  • Some that run on text files and apply given methods
    For example:
    src/test/resources/out/file1.methodX.methodY.sample would trigger a test asserting that its content is the same as what methodY(methodX({content of src/test/resources/in/file1.sample}) outputs (don't do reflection there are 3 buttons)
@DyspC
Copy link
Owner Author

DyspC commented Sep 23, 2020

Note to myself

  • Create CONTRIBUTING.md
  • Write README.md wtf is that thing supposed to do

@NyorJa
Copy link
Contributor

NyorJa commented Oct 1, 2020

Can I give this a try? Thanks

@DyspC
Copy link
Owner Author

DyspC commented Oct 1, 2020

Yeah sure,

If you want me to provide you with some before/after test data don't hesitate and ask me

@NyorJa
Copy link
Contributor

NyorJa commented Oct 1, 2020

that would be great!thanks again

@DyspC
Copy link
Owner Author

DyspC commented Oct 4, 2020

Hi, I've put some test files on the 'testing' branch

DyspC added a commit that referenced this issue Oct 21, 2020
Add some unit test issue #1
@DyspC DyspC closed this as completed Oct 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants