JakeHillion/nixpkgs
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

beets: Re-enable tests

Browse Source 
The reason why the completion tests didn't pass was because we had it
already disabled in 2acc258dff.

Meanwhile, beetbox/beets@a07cb83 has moved the file from
test/test_completion.sh to test/rsrc/test_completion.sh.

So this has silently re-enabled the completion tests, which we need to
investigate on our side why they failed in the first place.

Signed-off-by: aszlig <aszlig@redmoonstudios.org>
This commit is contained in:
aszlig 2016-02-10 02:41:15 +01:00 committed by Robin Gloster
parent 85c0d55d1d
commit 2d25ab3a03
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
pkgs/tools/audio/beets/default.nix
View File

@ -121,7 +121,7 @@ in buildPythonPackage rec {
postPatch = '' postPatch = ''
sed -i -e '/assertIn.*item.*path/d' test/test_info.py sed -i -e '/assertIn.*item.*path/d' test/test_info.py
echo echo completion tests passed > test/test_completion.sh echo echo completion tests passed > test/rsrc/test_completion.sh
sed -i -e '/^BASH_COMPLETION_PATHS *=/,/^])$/ { sed -i -e '/^BASH_COMPLETION_PATHS *=/,/^])$/ {
/^])$/i u"${completion}" /^])$/i u"${completion}"
@ -139,7 +139,7 @@ in buildPythonPackage rec {
test/test_replaygain.py test/test_replaygain.py
''; '';
doCheck = false; # TODO, see https://github.com/beetbox/beets/issues/1876#issuecomment-182010438 doCheck = true;
preCheck = '' preCheck = ''
(${concatMapStrings (s: "echo \"${s}\";") allPlugins}) \ (${concatMapStrings (s: "echo \"${s}\";") allPlugins}) \