shimming-toolbox / shimming-toolbox / 617 / 1
90%
master: 90%

DEFAULT BRANCH: master
Ran
Files 29
Run time 2s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 90.157% (+0.2%) from 89.947%
617.1

push

travis-ci-com

GitHub 
Check installation and versions of 3rd party binaries (#176)

* Add CLI to check 3rd-party binary installation

Resolves #161

**Why this change was necessary**
It's more useful to the user to have a separate script that they can
run to check their installation without running the visually
overwhelming testing suite.

**What this change does**
Creates 2 helper functions using the GNU `which` tool to verify that
the `prelude` and `dcm2niic` binaries are both installed and available
in the PATH. Since these helper functions only return the resulting
exit code of the `which` calls, a wrapper function is exposed as a
CLI, with informative text for a human user.

**Any side-effects?**
Note that this duplicates code in conftest.py, which will be
refactored in a subsequent commit to rely on the helper functions
added in this commit.

* Refactor conftest to use check_env helpers

Related to #161

**Why this change was necessary**
conftest.py had some functionality duplicated in check_env.py, so the
duplicated code was factored out.

**What this change does**
Refactors conftest dependency checks to rely on imported check_env.py.

**Any side-effects?**
conftest now depends on the check_env helper functions for verifying
the prelude and dcm2niix installations.

* Add helper function to check dcm2niix version

Resolves #161

**Why this change was necessary**
Part of the issue spec is to also check the dependency version if it's
installed.

**What this change does**
Adds function that captures output of `dcm2niix --version` and is
called if `dcm2niix` has been verified to be installed and in the
PATH.

Note that `dcm2niix --version` returns a nonzero exit code. This
helper's approach will be invalid if this behaviour in dcm2niix is
fixed.

**Any side-effects?**
None.

* Add helper function to check prelude version

Resolves #161

**Why this change was necessary**
Part of the issue spec is to also check t... (continued)

916 of 1016 relevant lines covered (90.16%)

0.9 hits per line

Source Files on job 617.1