forked from mirrors/nixpkgs
nbstripout: fix tests
Building of nbstripout has failed after the version was bumped to 0.3.1 because: the test files were missing from the new release tarball. This adds tests by using the release tarball from GitHub instead of PyPI. However, these tests fail because of a bug in one test. Thus, a patch is used to fix the tests.
This commit is contained in:
parent
56fb68dcef
commit
caf8794910
|
@ -1,4 +1,4 @@
|
||||||
{lib, python2Packages, git, mercurial, coreutils}:
|
{lib, python2Packages, fetchFromGitHub, fetchurl, git, mercurial, coreutils}:
|
||||||
|
|
||||||
with python2Packages;
|
with python2Packages;
|
||||||
buildPythonApplication rec {
|
buildPythonApplication rec {
|
||||||
|
@ -12,11 +12,30 @@ buildPythonApplication rec {
|
||||||
buildInputs = [ pytest pytest-flake8 pytest-cram git pytestrunner ];
|
buildInputs = [ pytest pytest-flake8 pytest-cram git pytestrunner ];
|
||||||
propagatedBuildInputs = [ ipython nbformat ];
|
propagatedBuildInputs = [ ipython nbformat ];
|
||||||
|
|
||||||
src = fetchPypi {
|
# PyPI source is currently missing tests. Thus, use GitHub instead.
|
||||||
inherit pname version;
|
# See: https://github.com/kynan/nbstripout/issues/73
|
||||||
sha256 = "b997c99b8bbb865988202d2f005cdaabb2598b07dad891c302a147a5871a4a95";
|
# Use PyPI again after it has been fixed in a release.
|
||||||
|
src = fetchFromGitHub {
|
||||||
|
owner = "kynan";
|
||||||
|
repo = pname;
|
||||||
|
rev = version;
|
||||||
|
sha256 = "1jifqmszjzyaqzaw2ir83k5fdb04iyxdad4lclawpb42hbink9ws";
|
||||||
};
|
};
|
||||||
|
|
||||||
|
patches = [
|
||||||
|
(
|
||||||
|
# Fix git diff tests by using --no-index.
|
||||||
|
# See: https://github.com/kynan/nbstripout/issues/74
|
||||||
|
#
|
||||||
|
# Remove this patch once the pull request has been merged and a new
|
||||||
|
# release made.
|
||||||
|
fetchurl {
|
||||||
|
url = "https://github.com/jluttine/nbstripout/commit/03e28424fb788dd09a95e99814977b0d0846c0b4.patch";
|
||||||
|
sha256 = "09myfb77a2wh8lqqs9fcpam97vmaw8b7zbq8n5gwn6d80zbl7dn0";
|
||||||
|
}
|
||||||
|
)
|
||||||
|
];
|
||||||
|
|
||||||
# for some reason, darwin uses /bin/sh echo native instead of echo binary, so
|
# for some reason, darwin uses /bin/sh echo native instead of echo binary, so
|
||||||
# force using the echo binary
|
# force using the echo binary
|
||||||
postPatch = ''
|
postPatch = ''
|
||||||
|
|
Loading…
Reference in a new issue