3
63%
master: 63%

Ran 21 Dec 2015 05:34PM UTC

Files 2

Run time 1s

Badge

Embed ▾

Committed 21 Dec 2015 05:29PM UTC coverage: 72.222% (-0.5%) from 72.68%

Job # 158.3

Build Type

push

travis-ci

Committed by

Ichimonji10

Commit Message

Use urljoin in `pulp_smash.utils`

Fix #58:

> Many places in the tests combine URLs and paths with simple string addition.
> This has the potential to produce incorrect results. […] Pulp should produce
> paths that behave correctly with urljoin, and we've had problems in the past
> with developers making "relative paths" that have a leading slash.

Using urljoin instead of plain string concatenation lets Pulp Smash test the
paths returned by Pulp. If Pulp returns a relative path with a leading slash,
plain string concatenation will produce a path containing a (typically harmless
but incorrect) `//`, but urljoin will produce a just-plain-wrong URL.

Test results both before and after change:

    ============  =============================
    Pulp Version  Test Suite Results
    ============  =============================
    2.7           OK (skipped=13)
    dev (2.8)     FAILED (errors=2, skipped=13)
    ============  =============================

Command used:

    PULP_SMASH_CONFIG_FILE=$PS_CONFIG \
    python -m unittest2 discover pulp_smash.tests

Run Details

143 of 198 relevant lines covered (72.22%)

0.72 hits per line

PulpQE / pulp-smash / 158 / 3
63%
master: 63%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 158.3

PulpQE / pulp-smash / 158 / 3 63% master: 63%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 158.3

PulpQE / pulp-smash / 158 / 3
63%
master: 63%

README BADGES
x