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.

Create unit tests using [NUnit](https://nunit.org/). 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)

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.

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.

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.

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.

The unit test project and several Parser tests merged into develop with https://github.com/teamradish/TRTwitchPlaysBot/commit/9078a9e9ec40189d05f442aa342281e11618f1ae.
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.