Fix lint issues

lsst · Mar 11, 2024 · d029068 · d029068
1 parent fd8cd5a
commit d029068
Show file tree

Hide file tree

Showing 7 changed files with 4 additions and 13 deletions.
diff --git a/python/lsst/dm/rucio/register/app_parser.py b/python/lsst/dm/rucio/register/app_parser.py
@@ -122,7 +122,7 @@ def __init__(self, argv):
         else:
             loglevel = args.loglevel
 
-        logging.basicConfig(level=logging.INFO, format=(_FORMAT), datefmt="%Y-%m-%d %H:%M:%S")
+        logging.basicConfig(level=loglevel, format=(_FORMAT), datefmt="%Y-%m-%d %H:%M:%S")
 
         self.butler_repo = args.repo
         self.collections = args.collections

diff --git a/python/lsst/dm/rucio/register/dataset_map.py b/python/lsst/dm/rucio/register/dataset_map.py
@@ -19,8 +19,6 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
-import sys
-
 import yaml
 
 

diff --git a/python/lsst/dm/rucio/register/rucio_interface.py b/python/lsst/dm/rucio/register/rucio_interface.py
@@ -21,17 +21,13 @@
 
 import hashlib
 import logging
-import os
 import random
-import re
 import time
 import zlib
 
 import lsst.daf.butler
 import rucio.common.exception
-from lsst.dm.rucio.register.dataset_map import DatasetMap
 from lsst.dm.rucio.register.resource_bundle import ResourceBundle
-from lsst.resources import ResourcePath
 from rucio.client.didclient import DIDClient
 from rucio.client.replicaclient import ReplicaClient
 
@@ -41,7 +37,8 @@
 
 
 class RucioInterface:
-    """Add files as replicas in Rucio, along with metadata, and attach them to datasets.
+    """Add files as replicas in Rucio, along with metadata,
+    and attach them to datasets.
 
     Parameters
     ----------

diff --git a/python/lsst/dm/rucio/register/rucio_register_config.py b/python/lsst/dm/rucio/register/rucio_register_config.py
@@ -19,7 +19,6 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
-import os
 
 import yaml
 

diff --git a/tests/test_app_parser.py b/tests/test_app_parser.py
@@ -21,7 +21,6 @@
 
 
 import logging
-import os
 import unittest
 
 import lsst.utils.tests
@@ -61,6 +60,7 @@ def testMissingArguments(self):
             ap = AppParser(argv)
         except SystemExit:
             pass
+        self.assertIsNotNone(ap)
 
 
 class MemoryTester(lsst.utils.tests.MemoryTestCase):

diff --git a/tests/test_interface.py b/tests/test_interface.py
@@ -24,14 +24,12 @@
 import os
 import shutil
 import tempfile
-import unittest
 from unittest.mock import MagicMock
 
 import lsst.utils.tests
 from lsst.daf.butler import Butler, DatasetRef, FileDataset
 from lsst.daf.butler.registry import DatasetTypeError, MissingCollectionError
 from lsst.dm.rucio.register.rucio_interface import RucioInterface
-from lsst.dm.rucio.register.rucio_register_config import RucioRegisterConfig
 from lsst.pipe.base import Instrument
 from lsst.resources import ResourcePath
 

diff --git a/tests/test_resource_bundle.py b/tests/test_resource_bundle.py
@@ -20,7 +20,6 @@
 # along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
 
-import os
 import unittest
 
 import lsst.utils.tests