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

Tests !!!!!!!!!!! #46

Open
spMohanty opened this issue Aug 29, 2018 · 2 comments
Open

Tests !!!!!!!!!!! #46

spMohanty opened this issue Aug 29, 2018 · 2 comments
Labels
help wanted Extra attention is needed high priority

Comments

@spMohanty
Copy link
Member

Currently we have absolutely no tests !!

And we definitely do need good test coverage as the code base starts accepting contributions from multiple users.

This issue could use a little bit of love from the community !
(And this could be a great way for someone to understand the internals of the library too !)

@spMohanty spMohanty added help wanted Extra attention is needed high priority labels Aug 29, 2018
@EC2EZ4RD
Copy link

I am willing to make some contributions, so how and what should I do?

@idvhfd
Copy link
Collaborator

idvhfd commented Sep 4, 2018

Hey @EC2EZ4RD,

Thanks for taking the time. What I believe Mohanty means is that we are very happy with -any- test coverage submissions from the community. Therefore, you should be able to push whatever you wish through. We have 0.00f tests!

That being said, a good starter test should be to run the experiments by stubbing out the step phase and other open ends. This will help tremendously in ensuring future releases do not break people's ability to run the internals of the mod.

Running and testing each environment should be a next test priority. Ideally, we'd be able to recursively find and test all environments listed in the envs directory.

Sometimes on Win10, the game spazzes out when you run a map and focus on another window while it loads, causing the mod to presumably run in the background, but the window becomes impossible to maximize, therefore running a client and ensuring that it behaves as expected (i.e. correctly implements the given resolution, environment and agent parameters for example) would be another great contribution.

Everything needs to be automatically tested!

I'll put down a "tests" folder in the base directory on Master so you can all follow that pattern. I'll also try to follow up with some coverage later this week or the next if time allows.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed high priority
Projects
None yet
Development

No branches or pull requests

3 participants