More coverage for _config.py [skip ci]

src/gevent/_config.py

@@ -49,7 +49,10 @@ class SettingType(type):
             def getter(self):
                 return self.settings[setting_name].get()
 
-            def setter(self, value):
+            def setter(self, value): # pragma: no cover
+                # The setter should never be hit, Config has a
+                # __setattr__ that would override. But for the sake
+                # of consistency we provide one.
                 self.settings[setting_name].set(value)
 
             prop = property(getter, setter, doc=new_class.__doc__)
@@ -250,7 +253,7 @@ class ImportableSetting(object):
             if package_path:
                 try:
                     sys.path.remove(package_path)
-                except ValueError:
+                except ValueError: # pragma: no cover
                     pass
 
     shortname_map = {}

src/greentest/test___config.py

@@ -100,6 +100,10 @@ class TestConfig(unittest.TestCase):
         self.assertEqual(sorted(_config.config.settings),
                          sorted(dir(_config.config)))
 
+    def test_getattr(self):
+        # Bypass the property that might be set here
+        self.assertIsNotNone(_config.config.__getattr__('resolver'))
+
     def test__getattr__invalid(self):
         with self.assertRaises(AttributeError):
             getattr(_config.config, 'no_such_setting')
@@ -135,5 +139,9 @@ class TestImportableSetting(unittest.TestCase):
         self.assertEqual(w[0].category, DeprecationWarning)
         self.assertIn('Absolute paths', str(w[0].message))
 
+    def test_non_string(self):
+        i = _config.ImportableSetting()
+        self.assertIs(i._import(self), self)
+
 if __name__ == '__main__':
     unittest.main()