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 and minor improvements to ciphers module #235

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

sarathsoma
Copy link

@sarathsoma sarathsoma commented Nov 23, 2018

Short description of what this resolves:

Minor tweaks to how yoda exits during invalid inputs for ciphers module.
adds tests for ciphers module.

Changes proposed in this pull request:

  • Exit with status != 0 during invalid inputs
  • Tests to ciphers module [ ceaser, atbash and rot13 ]

Status (please check what you already did):

  • made PR ready for code review
  • added some tests for the functionality

@sarathsoma
Copy link
Author

@manparvesh Does this look good ? Can i work on adding more ciphers and tests ?

@manparvesh
Copy link
Member

@sarathsoma the tests are failing. Please fix that before adding more ciphers

@sarathsoma
Copy link
Author

@manparvesh Ran the tests on the dev environment. Everything looks okay. No tests failed. Any leads on this ?

@manparvesh
Copy link
Member

@sarathsoma I reran the build and found that it was okay for python 2 but not for 3. For Python 3 it probably went into some infinite loop. Please take a look:
https://travis-ci.org/yoda-pa/yoda/builds/458736952

I am guessing you have python 2 in your machine, please try on python 3 as well

@codecov-io
Copy link

codecov-io commented Nov 27, 2018

Codecov Report

Merging #235 into master will increase coverage by 2.82%.
The diff coverage is 93.18%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #235      +/-   ##
==========================================
+ Coverage   76.78%   79.61%   +2.82%     
==========================================
  Files          60       64       +4     
  Lines        3442     3522      +80     
==========================================
+ Hits         2643     2804     +161     
+ Misses        799      718      -81
Impacted Files Coverage Δ
modules/ciphers/atbash.py 100% <100%> (+70%) ⬆️
modules/ciphers/caesar.py 95% <100%> (+74.48%) ⬆️
modules/ciphers/rot13.py 94.28% <100%> (+70.75%) ⬆️
modules/dev.py 88.05% <50%> (+4.01%) ⬆️
tests/ciphers/test_rot13.py 94.44% <94.44%> (ø)
tests/ciphers/test_atbash.py 94.44% <94.44%> (ø)
tests/ciphers/test_input.py 94.44% <94.44%> (ø)
tests/ciphers/test_ceaser.py 95.45% <95.45%> (ø)
tests/dev/test_horoscope.py 92% <0%> (-4%) ⬇️
modules/gif.py 70% <0%> (-3.34%) ⬇️
... and 11 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f9e5b6b...8d17f4f. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants