10156
70%
master: 97%

Ran 11 Feb 2016 12:54PM UTC

Jobs 1

Files 93

Run time 3s

Badge

Embed ▾

pending completion

Build # 10156

Build Type

push

travis-ci

Committed by

seanh

Commit Message

Break up views_test.py into classes

This has several advantages:

- Make this very long code file easier to navigate, having the class lines and
  indentation as markers for when the tests for one class begins and another
  ends, the structure of the test module mirroring that of the real module.

- We can put @pytest.mark.usefixtures() on the classes, instead of repeating it
  on every single test function.

- views.py has recently been refactored so that multiple classes each have a
  method named get() and a method named post(). Before this commit the test
  functions hadn't been updated with these new method names, and so were
  incorrectly named.

  Now that we have multiple get() methods (for example) in different classes,
  test_get_*() will no longer do as a test function name. It would have to be
  test_FooClass_get_*(). The class names in this module are sometimes quite
  long leading to a long prefix on each test function name. With test classes
  the test methods can just be called test_get_*() (plus four characters
  of indentation).

- All in all I think the code is much less noisy, more readable, now.

- Since there's only one blank line required between methods, but two between
  functions, we save a lot of lines!

Run Details

2611 of 3738 relevant lines covered (69.85%)

0.7 hits per line

Uncovered Existing Lines

Lines	Coverage	∆	File
12	100.0		h/auth.py
18	100.0		h/app.py

Jobs

ID	Job ID	Ran	Files	Coverage
1	10156.1	11 Feb 2016 12:54PM UTC	0	69.85	Travis Job 10156.1

Source Files on build 10156

Detailed source file information is not available for this build.

hypothesis / h / 10156
70%
master: 97%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build 10156

hypothesis / h / 10156 70% master: 97%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build 10156

hypothesis / h / 10156
70%
master: 97%

README BADGES
x