diff --git a/pkgs/development/python-modules/approvaltests/default.nix b/pkgs/development/python-modules/approvaltests/default.nix
index f55ca7e29670..070479b69377 100644
--- a/pkgs/development/python-modules/approvaltests/default.nix
+++ b/pkgs/development/python-modules/approvaltests/default.nix
@@ -1,36 +1,33 @@
 { lib
-, buildPythonPackage
-, fetchFromGitHub
-, pythonOlder
-
-# propagates
 , allpairspy
 , approval-utilities
 , beautifulsoup4
+, buildPythonPackage
 , empty-files
+, fetchFromGitHub
+, mock
 , mrjob
+, numpy
 , pyperclip
 , pytest
-, typing-extensions
-
-# tests
-, numpy
 , pytestCheckHook
+, pythonOlder
+, testfixtures
+, typing-extensions
 }:
 
 buildPythonPackage rec {
-  version = "8.3.1";
   pname = "approvaltests";
+  version = "8.4.1";
   format = "setuptools";
 
   disabled = pythonOlder "3.7";
 
-  # no tests included in PyPI tarball
   src = fetchFromGitHub {
     owner = "approvals";
     repo = "ApprovalTests.Python";
     rev = "refs/tags/v${version}";
-    hash = "sha256-FyYT+w4CX+CdUg0uGwyjw98H8Z+HMVecgMBW/ytrtFU=";
+    hash = "sha256-BuFiNZZ7228CKJ97mVK2S8Siqe040EYV5pElVtwuVf4=";
   };
 
   propagatedBuildInputs = [
@@ -41,16 +38,18 @@ buildPythonPackage rec {
     mrjob
     pyperclip
     pytest
+    testfixtures
     typing-extensions
   ];
 
   nativeCheckInputs = [
+    mock
     numpy
     pytestCheckHook
   ];
 
   disabledTests = [
-    # tests expects paths below ApprovalTests.Python directory
+    # Tests expects paths below ApprovalTests.Python directory
     "test_received_filename"
     "test_pytest_namer"
   ];
@@ -63,7 +62,8 @@ buildPythonPackage rec {
   meta = with lib; {
     description = "Assertion/verification library to aid testing";
     homepage = "https://github.com/approvals/ApprovalTests.Python";
+    changelog = "https://github.com/approvals/ApprovalTests.Python/releases/tag/v${version}";
     license = licenses.asl20;
-    maintainers = [ maintainers.marsam ];
+    maintainers = with maintainers; [ marsam ];
   };
 }