Merge pull request #186413 from onny/rnginline

pkgs/development/python-modules/rnginline/default.nix

@@ -1,4 +1,13 @@
-{ lib, fetchPypi, buildPythonPackage, lxml, docopt, six, pytestCheckHook, mock }:
+{ lib
+, fetchPypi
+, buildPythonPackage
+, lxml
+, docopt
+, six
+, pytestCheckHook
+, mock
+, fetchpatch
+}:
 
 buildPythonPackage rec {
   pname = "rnginline";
@@ -9,20 +18,33 @@ buildPythonPackage rec {
     sha256 = "sha256-j4W4zwHA4yA6iAFVa/LDKp00eeCX3PbmWkjd2LSUGfk=";
   };
 
-  propagatedBuildInputs = [ lxml docopt six ];
-
-  checkInputs = [ pytestCheckHook mock ];
-
-  # Those tests does not succeed, a test dependency is likely missing but nothing is specified upstream
-  disabledTestPaths = [
-    "rnginline/test/test_cmdline.py"
-    "rnginline/test/test_rnginline.py"
+  patches = [
+    # Fix failing tests. Should be included in releases after 0.0.2
+    # https://github.com/h4l/rnginline/issues/3
+    (fetchpatch {
+      url = "https://github.com/h4l/rnginline/commit/b1d1c8cda2a17d46627309950f2442021749c07e.patch";
+      sha256 = "sha256-XbisEwun2wPOp7eqW2YDVdayJ4sjAMG/ezFwgoCKe9o=";
+      name = "fix_tests_failing_collect.patch";
+    })
   ];
 
-  meta = {
+  propagatedBuildInputs = [
+    docopt
+    lxml
+    six
+  ];
+
+  checkInputs = [
+    mock
+    pytestCheckHook
+  ];
+
+  pythonImportsCheck = [ "rnginline" ];
+
+  meta = with lib; {
     description = "A Python library and command-line tool for loading multi-file RELAX NG schemas from arbitary URLs, and flattening them into a single RELAX NG schema";
     homepage = "https://github.com/h4l/rnginline";
-    license = lib.licenses.asl20;
-    maintainers = [ lib.maintainers.lesuisse ];
+    license = licenses.asl20;
+    maintainers = with maintainers; [ lesuisse ];
   };
 }