Fixed some Flake8 errors

Author: pavelvazquez

src/omnipy/data/serializer.py

@@ -124,7 +124,7 @@ def _to_data_from_data_if_direct(dataset, serializer: Serializer):
                 try:
                     new_dataset = func(dataset, serializer)
                     return new_dataset, serializer
-                except (TypeError, ValueError, ValidationError, AssertionError) as e:
+                except (TypeError, ValueError, ValidationError, AssertionError):
                     pass
 
         return None, None

src/omnipy/modules/fairtracks/functions.py

@@ -23,12 +23,9 @@ def encode_api(endpoint='experiments', id=None, limit=None, format='json', frame
 
 # ['projects', 'cases', 'files', 'annotations'], starting_point='0', size='25'
 def gdc_api(object_type='projects', starting_point=None, size=None):
-    api_url = GDC_BASE_URL + object_type + '/' + '?' + \
-              '&'.join(
-                  (['from=' + starting_point] if starting_point else [])
-                  + (['size=' + size] if size else [])
-                  + (['expand=' + 'project'] if object_type == 'cases' else [])
-              )
+    api_url = GDC_BASE_URL + object_type + '/' + '?' + '&'.join(
+        (['from=' + starting_point] if starting_point else []) + (['size=' + size] if size else [])
+        + ([+ 'project'] if object_type == 'cases' else []))
     print(api_url)
     response = requests.get(api_url)
     if response.status_code != 200:

tests/integration/reused/engine/conftest.py

@@ -33,8 +33,11 @@ def all_job_classes(
     dag_flow_template_cls: Type[IsDagFlowTemplate],
     func_flow_template_cls: Type[IsFuncFlowTemplate],
 ):
-    return job_type, task_template_cls, linear_flow_template_cls, \
-           dag_flow_template_cls, func_flow_template_cls
+    return (job_type,
+            task_template_cls,
+            linear_flow_template_cls,
+            dag_flow_template_cls,
+            func_flow_template_cls)
 
 
 @pc.fixture(scope='function', name='plain_engine')
@@ -65,7 +68,11 @@ def all_func_types_real_jobs_all_engines_real_reg(
     engine_decorator: Optional[Callable[[IsEngine], IsEngine]],
     registry: Optional[IsRunStateRegistry],
 ):
-    job_type, task_template_cls, linear_flow_template_cls, dag_flow_template_cls, func_flow_template_cls = job_classes
+    (job_type,
+     task_template_cls,
+     linear_flow_template_cls,
+     dag_flow_template_cls,
+     func_flow_template_cls) = job_classes
 
     # TODO: Fix job_type comparisons everywhere (bug due to pytest.fixture?)
     if job_type.value == JobType.linear_flow.value:

tests/util/test_dynamic_mixin.py

@@ -1,4 +1,4 @@
-from abc import ABCMeta
+# from abc import ABCMeta
 from inspect import Parameter, signature
 from typing import Dict, Generic, get_args, Optional, Tuple, Type, TypeVar
 
@@ -114,7 +114,7 @@ def override(self):
 
 class MockKwArgAccessOrigClsMemberStateMixin:
     def __init__(self, *, my_kwarg):
-        self._value = f'my_kwarg: {my_kwarg}' if self.kwargs.get('verbose') == True else my_kwarg
+        self._value = f'my_kwarg: {my_kwarg}' if self.kwargs.get('verbose', False) else my_kwarg
 
     def new_method(self):
         return self._value
@@ -182,7 +182,7 @@ def _assert_args_and_kwargs(mock_obj: object,
 
     if mock_cls:
         assert tuple(signature(mock_cls.__init__).parameters.keys()) == \
-           tuple(['self'] + non_self_param_keys)
+               tuple(['self'] + non_self_param_keys)
 
 
 def test_new_method_no_state_mixin(mock_plain_cls):
@@ -513,7 +513,7 @@ def override(self):
 
 def test_predefined_init_kwargs_progressive_multiple_state_mixins_same_default(
         mock_predefined_init_kwargs_cls):
-    MockPredefinedInitKwargsCls = mock_predefined_init_kwargs_cls  #noqa
+    MockPredefinedInitKwargsCls = mock_predefined_init_kwargs_cls  # noqa
 
     MockPredefinedInitKwargsCls.accept_mixin(MockKwArgStateMixin)
 
@@ -561,7 +561,7 @@ def test_predefined_init_kwargs_progressive_multiple_state_mixins_same_default(
 
 def test_predefined_init_kwargs_progressive_multiple_state_mixins_different_withmixin_classes(
         mock_predefined_init_kwargs_cls):
-    MockPredefinedInitKwargsCls = mock_predefined_init_kwargs_cls  #noqa
+    MockPredefinedInitKwargsCls = mock_predefined_init_kwargs_cls  # noqa
 
     MockPredefinedInitKwargsCls.accept_mixin(MockKwArgStateMixin)
 
@@ -602,7 +602,7 @@ def test_predefined_init_kwargs_progressive_multiple_state_mixins_different_with
 
 
 def test_nested_mixins(mock_predefined_init_kwargs_cls, mock_plain_cls):
-    MockPredefinedInitKwargsCls = mock_predefined_init_kwargs_cls  #noqa
+    MockPredefinedInitKwargsCls = mock_predefined_init_kwargs_cls  # noqa
     MockPlainCls = mock_plain_cls  # noqa
 
     MockPredefinedInitKwargsCls.accept_mixin(MockKwArgStateMixin)
@@ -625,7 +625,7 @@ def test_nested_mixins(mock_predefined_init_kwargs_cls, mock_plain_cls):
 
 
 def test_nested_mixins_static_outer_inheritance(mock_predefined_init_kwargs_cls):
-    MockPredefinedInitKwargsCls = mock_predefined_init_kwargs_cls  #noqa
+    MockPredefinedInitKwargsCls = mock_predefined_init_kwargs_cls  # noqa
     MockPredefinedInitKwargsCls.accept_mixin(MockKwArgStateMixin)
 
     class MockMockPlainCls(MockPredefinedInitKwargsCls):
@@ -654,7 +654,7 @@ def override(self):
 
 
 def test_nested_mixins_double_static_outer_inheritance(mock_predefined_init_kwargs_cls):
-    MockPredefinedInitKwargsCls = mock_predefined_init_kwargs_cls  #noqa
+    MockPredefinedInitKwargsCls = mock_predefined_init_kwargs_cls  # noqa
     MockPredefinedInitKwargsCls.accept_mixin(MockKwArgStateMixin)
 
     class MockMockPlainCls(MockPredefinedInitKwargsCls):
@@ -687,7 +687,7 @@ class MockMockMockPlainCls(MockMockPlainCls):
 
 def test_nested_mixins_static_outer_inheritance_from_generic(
         mock_predefined_init_kwargs_generic_cls):
-    MockPredefinedInitKwargsGenericCls = mock_predefined_init_kwargs_generic_cls  #noqa
+    MockPredefinedInitKwargsGenericCls = mock_predefined_init_kwargs_generic_cls  # noqa
     MockPredefinedInitKwargsGenericCls.accept_mixin(MockKwArgStateMixin)
 
     class MockMockPlainCls(MockPredefinedInitKwargsGenericCls[int]):