refactor: 根据 sonarlint 提示修改异味代码

Author: leafcoder

bgesdk/client.py

@@ -41,6 +41,8 @@
 
 endpoints = [v['endpoint'] for v in constants.ENDPOINTS]
 
+ACCESS_TOKEN_API = '/oauth2/access_token'
+
 
 def progress_callback(bytes_consumed, total_bytes):
     sys.stdout.write(
@@ -65,7 +67,7 @@ def alive(self):
         self.endpoint, max_retries=max_retries, verbose=verbose)
     try:
         request.get('/ping', timeout=timeout)
-    except Exception as e:
+    except Exception:
         return False
     return True
 
@@ -151,7 +153,7 @@ def exchange_authorization_code(self, code, redirect_uri):
         request = HTTPRequest(
             self.endpoint, max_retries=max_retries, verbose=verbose)
         result = request.post(
-            '/oauth2/access_token', data=data, timeout=timeout)
+            ACCESS_TOKEN_API, data=data, timeout=timeout)
         return models.AuthorizationCodeToken(self, result)
 
     def exchange_refresh_token(self, refresh_token):
@@ -175,7 +177,7 @@ def exchange_refresh_token(self, refresh_token):
         request = HTTPRequest(
             self.endpoint, max_retries=max_retries, verbose=verbose)
         result = request.post(
-            '/oauth2/access_token', data=data, timeout=timeout)
+            ACCESS_TOKEN_API, data=data, timeout=timeout)
         return models.AuthorizationCodeToken(self, result)
 
     def get_credentials_token(self):
@@ -195,7 +197,7 @@ def get_credentials_token(self):
         request = HTTPRequest(
             self.endpoint, max_retries=max_retries, verbose=verbose)
         result = request.post(
-            '/oauth2/access_token', data=data, timeout=timeout)
+            ACCESS_TOKEN_API, data=data, timeout=timeout)
         return models.ClientCredentialsToken(self, result)
 
     def get_api(self, access_token):

bgesdk/management/command.py

@@ -20,7 +20,7 @@ def __init__(self, parser, command_name=None, **kwargs):
         self.add_arguments(parser)
 
     def add_arguments(self, parser):
-        pass
+        """添加自定义命令参数"""
 
     @abstractmethod
     def handler(self, args):

bgesdk/management/commands/api/commands/aggregate_omics_data.py

@@ -6,7 +6,10 @@
 from bgesdk.management import constants
 from bgesdk.management.command import BaseCommand
 from bgesdk.management.utils import (
-    get_active_project, config_get, read_config, output
+    get_active_project,
+    config_get,
+    read_config,
+    output
 )
 from bgesdk.models import ModelEncoder
 from bgesdk.version import __version__

bgesdk/management/commands/api/commands/download.py

@@ -6,7 +6,10 @@
 from bgesdk.management import constants
 from bgesdk.management.command import BaseCommand
 from bgesdk.management.utils import (
-    get_active_project, config_get, read_config, output
+    get_active_project,
+    config_get,
+    read_config,
+    output
 )
 from bgesdk.version import __version__
 

bgesdk/management/commands/api/commands/get_data_items.py

@@ -6,7 +6,10 @@
 from bgesdk.management import constants
 from bgesdk.management.command import BaseCommand
 from bgesdk.management.utils import (
-    get_active_project, config_get, read_config, output
+    get_active_project,
+    config_get,
+    read_config,
+    output
 )
 from bgesdk.models import ModelEncoder
 from bgesdk.version import __version__

bgesdk/management/commands/api/commands/get_func_abundance.py

@@ -6,7 +6,10 @@
 from bgesdk.management import constants
 from bgesdk.management.command import BaseCommand
 from bgesdk.management.utils import (
-    get_active_project, config_get, read_config, output
+    get_active_project,
+    config_get,
+    read_config,
+    output
 )
 from bgesdk.models import ModelEncoder
 from bgesdk.version import __version__

bgesdk/management/commands/api/commands/get_gene_abundance.py

@@ -6,7 +6,10 @@
 from bgesdk.management import constants
 from bgesdk.management.command import BaseCommand
 from bgesdk.management.utils import (
-    get_active_project, config_get, read_config, output
+    get_active_project,
+    config_get,
+    read_config,
+    output
 )
 from bgesdk.models import ModelEncoder
 from bgesdk.version import __version__

bgesdk/management/commands/api/commands/get_range_stream.py

@@ -6,7 +6,10 @@
 from bgesdk.management import constants
 from bgesdk.management.command import BaseCommand
 from bgesdk.management.utils import (
-    get_active_project, config_get, read_config, output
+    get_active_project,
+    config_get,
+    read_config,
+    output
 )
 from bgesdk.models import ModelEncoder
 from bgesdk.version import __version__
@@ -36,12 +39,12 @@ def add_arguments(self, parser):
         parser.add_argument(
             '--start_time',
             type=str,
-            help='数据元编号。'
+            help='起始时间。'
         )
         parser.add_argument(
             '--end_time',
             type=str,
-            help='数据元编号。'
+            help='终止时间。'
         )
         parser.add_argument(
             '--sort_direction',

bgesdk/management/commands/api/commands/get_samples.py

@@ -6,7 +6,10 @@
 from bgesdk.management import constants
 from bgesdk.management.command import BaseCommand
 from bgesdk.management.utils import (
-    get_active_project, config_get, read_config, output
+    get_active_project,
+    config_get,
+    read_config,
+    output
 )
 from bgesdk.models import ModelEncoder
 from bgesdk.version import __version__

bgesdk/management/commands/api/commands/get_taxon_abundance.py

@@ -6,7 +6,10 @@
 from bgesdk.management import constants
 from bgesdk.management.command import BaseCommand
 from bgesdk.management.utils import (
-    get_active_project, config_get, read_config, output
+    get_active_project,
+    config_get,
+    read_config,
+    output
 )
 from bgesdk.models import ModelEncoder
 from bgesdk.version import __version__