Add tests to the project

[oneofmanylukes]

Maybe some tests would be a good idea, because then you can see if your changes break anything or have other unexpected effects.

[Aleksbgbg]

If the codebase is to be tested, it must first be testable. For the codebase to be testable, we must refactor as per issue #4.

[oneofmanylukes]

Sounds as it would be a good idea to refactor then.

[andrewyancey]

It’s already started it just needs to be merged into dev

[andrewyancey]

Should we add a separate vs project for the test classes?

[oneofmanylukes]

Isn't that the way it's usually done? As far as I can tell

[andrewyancey]

Yes but I believe it's also possible to do it in the same project. Just curious if you guys wanted the extra overhead of a second project.

Alternatively, I think we could have a testing branch so that only the people who want to run the tests have to pull them.

[Aleksbgbg]

Testing branch is a fair point, but it will require more synchronisation.

I don't think the second project adds any significant overhead - in fact, it prevents us from compiling test code into the release build.

[oneofmanylukes]

I would also go with a second project for tests

[andrewyancey]

Alright let's go with the second project then.