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.

Sign up for GitHub

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

Jump to bottom

Test coverage #17

Open
4 of 8 tasks
Evalle opened this issue Oct 27, 2018 · 5 comments
Open
4 of 8 tasks

Test coverage #17

Evalle opened this issue Oct 27, 2018 · 5 comments
Assignees
@Daxaker @aberkromb
Labels
Testing

Comments

@Evalle
Copy link
Owner

Evalle commented Oct 27, 2018 •
edited by aberkromb
Loading

Write untitests for basic functionality. For more info see here, and here
So far, we need to test:

  • config.py
  • hostbuilder.py
  • randomvideo.py
  • server.py
  • youtube_service.py
  • quotes.py
  • bm_cat_bot.py
  • cache.py
@Evalle Evalle changed the title Write some unittestst Write some unittests Oct 27, 2018
@Evalle Evalle added the Testing label Oct 27, 2018
@Evalle
Copy link
Owner Author

Evalle commented Oct 27, 2018 •
edited
Loading

I'm gonna write a test for quotes.py

@Daxaker
Copy link
Collaborator

Daxaker commented Oct 27, 2018

I'll take hostbuilder.py

@aberkromb
Copy link
Collaborator

I'm gonna write a test for quotes.py

can i take quotes.py?

@Evalle
Copy link
Owner Author

Evalle commented Oct 29, 2018

yep, it's all yours

@Daxaker Daxaker mentioned this issue Oct 29, 2018
Merged
@Daxaker
Copy link
Collaborator

Daxaker commented Oct 31, 2018

There is no need of testing config.py it's only a bunch of global variables

@Evalle Evalle changed the title Write some unittests Test coverage Nov 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Daxaker Daxaker

@aberkromb aberkromb

Labels
Testing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Evalle @Daxaker @aberkromb