Removing RF 2.9 support

aaltat · aaltat · commit bcdd57a2f3bc · 2019-02-12T09:44:25.000+02:00
diff --git a/.travis.yml b/.travis.yml
@@ -51,7 +51,7 @@ matrix:
       env:
         - BROWSER=chrome
         - SELENIUM=3.141.0
-        - ROBOTFRAMEWORK=2.9.2
+        - ROBOTFRAMEWORK=3.0.4
         - ROBOT_OPTIONS=--dotted
         - INTERPRETER=python2
 before_script:
diff --git a/src/SeleniumLibrary/utils/types.py b/src/SeleniumLibrary/utils/types.py
@@ -15,15 +15,11 @@
 # limitations under the License.
 
 # Originally based on  Robot Framework 3.0.2 robot.utils.robottypes and
-# robot.utils.PY3 Can be removed when library minimum required Robot Framework
+# Can be removed when library minimum required Robot Framework
 # version is greater than 3.0.2. Then Robot Framework is_truthy should
 # also support string NONE as Python False.
 from robot.utils import is_string
-try:
-    from robot.utils import PY3
-except ImportError:
-    import sys
-    PY3 = sys.version_info[0] == 3
+from robot.utils import PY3
 
 
 def is_truthy(item):
diff --git a/utest/test/utils/test_types.py b/utest/test/utils/test_types.py
@@ -1,19 +1,6 @@
 import unittest
 
-from SeleniumLibrary.utils import is_string, is_truthy, is_falsy, is_noney
-
-
-class IsstringTests(unittest.TestCase):
-
-    def test_is_string(self):
-        strings = ['1', 'foo', ' ', u'', '']
-        for item in strings:
-            self.assertTrue(is_string(item))
-
-    def test_is_not_string(self):
-        strings = [1, 2.345, None, False]
-        for item in strings:
-            self.assertFalse(is_string(item))
+from SeleniumLibrary.utils import is_truthy, is_falsy, is_noney
 
 
 class IsTruthyFalsyNoneyTests(unittest.TestCase):
