Merge pull request #118355 from dotlambda/fava-pytestCheckHook

fava: modernize expression

pkgs/applications/office/fava/default.nix

@@ -1,41 +1,44 @@
-{ lib, python3, beancount }:
+{ lib, python3 }:
 
-let
+python3.pkgs.buildPythonApplication rec {
-  inherit (python3.pkgs) buildPythonApplication fetchPypi;
-in
-buildPythonApplication rec {
   pname = "fava";
   version = "1.18";
 
-  src = fetchPypi {
+  src = python3.pkgs.fetchPypi {
     inherit pname version;
     sha256 = "21336b695708497e6f00cab77135b174c51feb2713b657e0e208282960885bf5";
   };
 
-  checkInputs = [ python3.pkgs.pytest ];
   nativeBuildInputs = with python3.pkgs; [ setuptools-scm ];
-  propagatedBuildInputs = with python3.pkgs;
-    [
-      Babel
-      cheroot
-      flaskbabel
-      flask
-      jinja2
-      beancount
-      click
-      markdown2
-      ply
-      simplejson
-      werkzeug
-      jaraco_functools
-    ];
 
-  # CLI test expects fava on $PATH.  Not sure why static_url fails.
+  propagatedBuildInputs = with python3.pkgs; [
-  # the entry_slices and render_entries requires other files to pass
+    Babel
-  checkPhase = ''
+    cheroot
-    py.test tests -k 'not cli and not static_url and not entry_slice and not render_entries'
+    flaskbabel
+    flask
+    jinja2
+    beancount
+    click
+    markdown2
+    ply
+    simplejson
+    werkzeug
+    jaraco_functools
+  ];
+
+  checkInputs = with python3.pkgs; [
+    pytestCheckHook
+  ];
+
+  preCheck = ''
+    export HOME=$TEMPDIR
   '';
 
+  disabledTests = [
+    # runs fava in debug mode, which tries to interpret bash wrapper as Python
+    "test_cli"
+  ];
+
   meta = {
     homepage = "https://beancount.github.io/fava";
     description = "Web interface for beancount";