New Issue

Add Unit Tests #20

Closed

opened 2 years ago by tdeeb · 5 comments

tdeeb commented 2 years ago (Migrated from github.com)

Owner

Create unit tests using NUnit. Some tests could include:

Parser output using many different sample inputs.
Adding/removing macros, memes, etc.

It would be better to have a separate project for the tests so the code doesn't get muddled.

lucasortizny commented 2 years ago (Migrated from github.com)

Owner

Would this be something we want after a database implementation? (Update: my bad this shouldn't matter since it isn't an integration test)

tdeeb commented 2 years ago (Migrated from github.com)

Owner

Not necessarily, but it may be preferable. Having the database implemented would let us test the database itself.

tdeeb commented 2 years ago (Migrated from github.com)

Owner

Some of the code is hard to test due to tight coupling. For example, Parser.ParseInputs is coupled to reading bot data. I'll start on improving this.

tdeeb commented 2 years ago (Migrated from github.com)

Owner

I made the necessary changes to test the Parser and added a project for conducting tests. I'll double-check everything, merge it in, and add documentation on how to run the tests later.

tdeeb commented 2 years ago (Migrated from github.com)

Owner

The unit test project and several Parser tests merged into develop with 9078a9e9ec.

enhancement good first issue

No Milestone

No project

No Assignees

1 Participants

Notifications

Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.