2
63%
master: 63%

Ran 22 Jul 2016 12:45PM UTC

Files 6

Run time 0s

Badge

Embed ▾

Committed 22 Jul 2016 12:26PM UTC coverage: 70.543%. Remained the same

Job # 876.2

Build Type

push

travis-ci

Committed by

Ichimonji10

Commit Message

Refactor newly-added DRPM upload tests

Do at least the following:

* Change `pulp_smash.constants.DRPM` so that it includes the full path
  to the DRPM file relative to `pulp_smash.constants.DRPM_FEED_URL`.
  This removes several magic strings from the codebase, and should make
  it easier to change the value of `DRPM` in the furure.
* Tweak the style of multi-line imports: make imports alphabetic, and
  place a trailing comma at the end of every line. This makes it easier
  to scan through imports, and it makes future diffs smaller.
* Rewrite docstrings. Move general test case descriptions into the test
  case docstrings, and move information about specific tests and
  assertions into test case methods. Drop duplicate URLs.
* Make code more compact. Drop extraneous whitespace, drop some
  single-use variables, etc.
* Change the structure of the CLI `UploadDrpmTestCase`. Instead of
  following the common pattern of placing the majority of business logic
  in `setUpClass` and `tearDownClass`, and assertion-specific logic in
  the test methods, move all test case logic into a single test method.
  This allows an excessive number of class variables to be dropped, and
  it allows the cleanup logic to be made much more resilient. It also
  allows an unnecessary upload and import to be dropped, which should
  lower test execution time.

Run Details

364 of 516 relevant lines covered (70.54%)

0.71 hits per line

PulpQE / pulp-smash / 876 / 2
63%
master: 63%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 876.2

PulpQE / pulp-smash / 876 / 2 63% master: 63%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 876.2

PulpQE / pulp-smash / 876 / 2
63%
master: 63%

README BADGES
x