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

Add test case that ensures that examples are triggering the desired rule #27

Conversation

macisamuele
Copy link
Collaborator

While working on #22 I felt like having examples is great.
But not being sure about the fact examples are still triggering the desired rule is not great.

In order to address this I've added a new test test_ensure_that_examples_are_covering_the_correct_rule that essentially runs the tool over the examples files and ensures that the correct reporting is thrown.

@macisamuele macisamuele added the enhancement New feature or request label Feb 13, 2020
@macisamuele macisamuele requested a review from sjaensch February 13, 2020 17:40
@codecov
Copy link

codecov bot commented Feb 13, 2020

Codecov Report

Merging #27 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #27   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          60     60           
  Lines        1891   1898    +7     
  Branches      121    121           
=====================================
+ Hits         1891   1898    +7
Impacted Files Coverage Δ
tests/docs_test.py 100% <100%> (ø) ⬆️

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 580b6c2...758243b. Read the comment docs.

@macisamuele macisamuele merged commit 6c4ce48 into Yelp:master Feb 14, 2020
@macisamuele macisamuele deleted the maci-add-test-to-ensure-that-tester-are-testing-the-correct-rule branch February 14, 2020 20:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants