From bd2aeb4883176554214bdf2af404b88eb09d83fa Mon Sep 17 00:00:00 2001 From: aszlig Date: Sat, 2 Sep 2017 01:50:36 +0200 Subject: [PATCH] beets-alternatives: Disable test suite for now Regression introduced by 94351197cd40d7e2d22e8a971e888b8333764cb5. Running the tests results in the following traceback: ... File ".../unittest/loader.py", line 91, in loadTestsFromName module = __import__('.'.join(parts_copy)) File ".../test/regrtest.py", line 184, in for module in sys.modules.itervalues(): RuntimeError: dictionary changed size during iteration The reason for this is that the test directory itself is called "test" and the package including regrtest.py is also called "test", so the loader tries to load tests from its own implementation. We could fix this by changing PYTHONPATH and/or making the test directory a proper package, but we'd still have failing tests because beets itself is required to run the tests. However for now I'm just removing the unit_tests kwarg in setup.py so that we have the same behaviour as before the initially mentioned commit. Signed-off-by: aszlig --- pkgs/tools/audio/beets/alternatives-plugin.nix | 1 + 1 file changed, 1 insertion(+) diff --git a/pkgs/tools/audio/beets/alternatives-plugin.nix b/pkgs/tools/audio/beets/alternatives-plugin.nix index 116281e0a052..35562a85a4d3 100644 --- a/pkgs/tools/audio/beets/alternatives-plugin.nix +++ b/pkgs/tools/audio/beets/alternatives-plugin.nix @@ -13,6 +13,7 @@ pythonPackages.buildPythonApplication rec { postPatch = '' sed -i -e '/install_requires/,/\]/{/beets/d}' setup.py + sed -i -e '/test_suite/d' setup.py ''; propagatedBuildInputs = with pythonPackages; [ futures ];