Integrate code review comments from Fred.

Author: davidfstr

splunklib/client.py

@@ -2138,7 +2138,7 @@ def _get_kind_list(self, subpath=None):
         content = _load_atom_entries(response)
         for entry in content:
             this_subpath = subpath + [entry.title]
-            # The "all" endpoint is not a real input collection.
+            # The "all" endpoint doesn't work yet.
             # The "tcp/ssl" endpoint is not a real input collection.
             if entry.title == 'all' or this_subpath == ['tcp','ssl']:
                 continue

tests/test_service.py

@@ -109,12 +109,7 @@ def test_splunk_version(self):
 
     def test_query_without_login(self):
         service = Service()
-        try:
-            service.splunk_version
-            self.fail('Expected AuthenticationError.')
-        except AuthenticationError:
-            # Good
-            pass
+        self.assertRaises(AuthenticationError, lambda: service.splunk_version)
 
 class TestSettings(testlib.SDKTestCase):
     def test_read_settings(self):