Merge "Ensure all tox is passing"

software/software/exceptions.py

@@ -96,14 +96,17 @@ class StateValidationFailure(SoftwareError):
     """State Validation Failure"""
     pass
 
+
 class DeployDoNotExist(SoftwareError):
     """Deploy Do Not Exist"""
     pass
 
+
 class DeployAlreadyExist(SoftwareError):
     """Deploy Already Exist"""
     pass
 
+
 class ReleaseVersionDoNotExist(SoftwareError):
     """Release Version Do Not Exist"""
     pass

software/software/software_entities.py

@@ -6,7 +6,8 @@ SPDX-License-Identifier: Apache-2.0
 """
 
 import logging
-from abc import ABC, abstractmethod
+from abc import ABC
+from abc import abstractmethod
 from enum import Enum
 from typing import List
 
@@ -227,7 +228,6 @@ class DeployHosts(ABC):
         pass
 
 
-
 class DeployHandler(Deploy):
     def __init__(self):
         super().__init__()

software/software/utils.py

@@ -292,7 +292,7 @@ def check_state(state, states):
     :param state: String value.
     :param states: An Enum object.
     """
-    if not state in states.__members__:
+    if state not in states.__members__:
         msg = "State %s not in valid states: %s" % (state, list(states.__members__.keys()))
         LOG.exception(msg)
         raise StateValidationFailure(msg)