418
63%

Ran 07 Apr 2016 08:11PM UTC

Jobs 4

Files 6

Run time 2min

Badge

Embed ▾

pending completion

Build # 418

Build Type

Pull #186

travis-ci

Committed by

Ichimonji10

Commit Message

Fix a partially broken test

From a comment in the source code:

> When executing commands over SSH, in a non-login shell, and as a non-root
> user, the PATH environment variable is quite short. For example:
>
>     /usr/lib64/qt-3.3/bin:/usr/local/bin:/usr/bin

In cases where these conditions hold true, such as when Jenkins logs
into a Nodepool test system, several private functions that work with
`/usr/sbin/getenforce` will malfunction. Make these functions more
resilient by hard-coding in references to `/usr/sbin/getenforce`. This
approach is inflexible and inelegant. However:

> We're doing this it's simple (see _has_getenforce()), because Pulp is
> available on a limited number of platforms, and because we may move to
> an SSH client that allows for shell expansion.

Are there other solutions available? Sure! However:

> We cannot execute `PATH=${PATH}:/usr/sbin which getenforce` because
> Plumbum does a good job of preventing shell expansions. See:
> https://github.com/PulpQE/pulp-smash/issues/89

Pull Request Pull Request #186: Fix a partially broken test

Run Details

302 of 411 relevant lines covered (73.48%)

2.94 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	418.1	07 Apr 2016 08:11PM UTC	6	73.48	Travis Job 418.1
2	418.2	07 Apr 2016 08:12PM UTC	6	73.48	Travis Job 418.2
3	418.3	07 Apr 2016 08:13PM UTC	6	73.48	Travis Job 418.3
4	418.4	07 Apr 2016 08:13PM UTC	6	73.48	Travis Job 418.4

Source Files on build 418

Coverage	∆	File	Lines	Relevant	Covered	Missed	Hits/Line

PulpQE / pulp-smash / 418
63%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 418

PulpQE / pulp-smash / 418 63%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 418

PulpQE / pulp-smash / 418
63%

README BADGES
x