[py] Reverse validation logic

Author: cgoldberg

py/selenium/webdriver/chrome/service.py

@@ -60,6 +60,6 @@ def service_args(self) -> Sequence[str]:
 
     @service_args.setter
     def service_args(self, value: Sequence[str]):
-        if not isinstance(value, Sequence) or isinstance(value, str):
+        if isinstance(value, str) or not isinstance(value, Sequence):
             raise TypeError("service_args must be a sequence")
         self._service_args = list(value)

py/selenium/webdriver/chromium/service.py

@@ -74,6 +74,6 @@ def service_args(self) -> Sequence[str]:
 
     @service_args.setter
     def service_args(self, value: Sequence[str]):
-        if not isinstance(value, Sequence) or isinstance(value, str):
+        if isinstance(value, str) or not isinstance(value, Sequence):
             raise TypeError("service_args must be a sequence")
         self._service_args = list(value)

py/selenium/webdriver/edge/service.py

@@ -63,6 +63,6 @@ def service_args(self) -> Sequence[str]:
 
     @service_args.setter
     def service_args(self, value: Sequence[str]):
-        if not isinstance(value, Sequence) or isinstance(value, str):
+        if isinstance(value, str) or not isinstance(value, Sequence):
             raise TypeError("service_args must be a sequence")
         self._service_args = list(value)

py/selenium/webdriver/firefox/service.py

@@ -70,6 +70,6 @@ def service_args(self) -> Sequence[str]:
 
     @service_args.setter
     def service_args(self, value: Sequence[str]):
-        if not isinstance(value, Sequence) or isinstance(value, str):
+        if isinstance(value, str) or not isinstance(value, Sequence):
             raise TypeError("service_args must be a sequence")
         self._service_args = list(value)

py/selenium/webdriver/ie/service.py

@@ -73,6 +73,6 @@ def service_args(self) -> Sequence[str]:
 
     @service_args.setter
     def service_args(self, value: Sequence[str]):
-        if not isinstance(value, Sequence) or isinstance(value, str):
+        if isinstance(value, str) or not isinstance(value, Sequence):
             raise TypeError("service_args must be a sequence")
         self._service_args = list(value)

py/selenium/webdriver/safari/service.py

@@ -84,6 +84,6 @@ def service_args(self) -> Sequence[str]:
 
     @service_args.setter
     def service_args(self, value: Sequence[str]):
-        if not isinstance(value, Sequence) or isinstance(value, str):
+        if isinstance(value, str) or not isinstance(value, Sequence):
             raise TypeError("service_args must be a sequence")
         self._service_args = list(value)

py/selenium/webdriver/webkitgtk/service.py

@@ -71,6 +71,6 @@ def service_args(self) -> Sequence[str]:
 
     @service_args.setter
     def service_args(self, value: Sequence[str]):
-        if not isinstance(value, Sequence) or isinstance(value, str):
+        if isinstance(value, str) or not isinstance(value, Sequence):
             raise TypeError("service_args must be a sequence")
         self._service_args = list(value)

py/selenium/webdriver/wpewebkit/service.py

@@ -65,6 +65,6 @@ def service_args(self) -> Sequence[str]:
 
     @service_args.setter
     def service_args(self, value: Sequence[str]):
-        if not isinstance(value, Sequence) or isinstance(value, str):
+        if isinstance(value, str) or not isinstance(value, Sequence):
             raise TypeError("service_args must be a sequence")
         self._service_args = list(value)