update comments

wtsi-npg · Jan 25, 2024 · c138254 · c138254
1 parent a61a36c
commit c138254
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 8 deletions.
diff --git a/src/npg_irods/illumina.py b/src/npg_irods/illumina.py
@@ -1,6 +1,6 @@
 # -*- coding: utf-8 -*-
 #
-# Copyright © 2023 Genome Research Ltd. All rights reserved.
+# Copyright © 2023, 2024 Genome Research Ltd. All rights reserved.
 #
 # This program is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -234,7 +234,7 @@ def empty_acl(*args):
         for fc in flowcells:
             secondary_metadata.extend(sample_fn(fc.sample))
             secondary_metadata.extend(study_fn(fc.study))
-            acl.extend(acl_fn(fc.sample, fc.study, c.subset, zone=zone))
+            acl.extend(acl_fn(fc.sample, fc.study, subset=c.subset, zone=zone))
 
     # Remove duplicates
     secondary_metadata = sorted(set(secondary_metadata))

diff --git a/src/npg_irods/metadata/lims.py b/src/npg_irods/metadata/lims.py
@@ -1,6 +1,6 @@
 # -*- coding: utf-8 -*-
 #
-# Copyright © 2021, 2022, 2023 Genome Research Ltd. All rights reserved.
+# Copyright © 2021, 2022, 2023, 2024 Genome Research Ltd. All rights reserved.
 #
 # This program is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -187,10 +187,10 @@ def make_sample_acl(
     Note that this function does not check that the sample is in the study.
 
     Args:
-        subset: Subset of sequence reads
         sample: A sample, which will be used to confirm consent, which modifies the
                 ACL.
         study: A study, which will provide permissions for the ACL.
+        subset: Subset of sequence reads.
         zone: The iRODS zone.
 
     Returns:

diff --git a/tests/conftest.py b/tests/conftest.py
@@ -1,6 +1,6 @@
 # -*- coding: utf-8 -*-
 #
-# Copyright © 2020, 2022, 2023 Genome Research Ltd. All rights reserved.
+# Copyright © 2020, 2022, 2023, 2024 Genome Research Ltd. All rights reserved.
 #
 # This program is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by

diff --git a/tests/test_illumina.py b/tests/test_illumina.py
@@ -1,6 +1,6 @@
 # -*- coding: utf-8 -*-
 #
-# Copyright © 2023 Genome Research Ltd. All rights reserved.
+# Copyright © 2023, 2024 Genome Research Ltd. All rights reserved.
 #
 # This program is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -412,7 +412,7 @@ def test_updates_changed_study_permissions(
 
     @m.context("When data are multiplexed")
     @m.context("When data contain a human subset")
-    @m.it("Update managed access permissions to restricted human access group")
+    @m.it("Updates managed access permissions to restricted human access group")
     def test_updates_human_permissions_mx(
         self, illumina_synthetic_irods, illumina_synthetic_mlwh
     ):

diff --git a/tests/test_pacbio.py b/tests/test_pacbio.py
@@ -1,6 +1,6 @@
 # -*- coding: utf-8 -*-
 #
-# Copyright © 2023 Genome Research Ltd. All rights reserved.
+# Copyright © 2023, 2024 Genome Research Ltd. All rights reserved.
 #
 # This program is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by