Change server_info endpoint error message to be verbose (tableau#439)

jacobj10 · t8y8 · commit ea437a203dbb · 2019-05-29T13:22:36.000-07:00
First contribution from @jacobj10, fix a bug introduced from a requests upgrade, and some error handling improvements for a swallowed exception
diff --git a/tableauserverclient/server/endpoint/server_info_endpoint.py b/tableauserverclient/server/endpoint/server_info_endpoint.py
@@ -1,5 +1,5 @@
 from .endpoint import Endpoint, api
-from .exceptions import ServerResponseError, ServerInfoEndpointNotFoundError
+from .exceptions import ServerResponseError, ServerInfoEndpointNotFoundError, EndpointUnavailableError
 from ...models import ServerInfoItem
 import logging
 
@@ -19,6 +19,8 @@ def get(self):
         except ServerResponseError as e:
             if e.code == "404003":
                 raise ServerInfoEndpointNotFoundError
+            if e.code == "404001":
+                raise EndpointUnavailableError
 
         server_info = ServerInfoItem.from_response(server_response.content, self.parent_srv.namespace)
         return server_info
diff --git a/test/test_sort.py b/test/test_sort.py
@@ -58,7 +58,7 @@ def test_filter_in(self):
                                                        auth_token='j80k54ll2lfMZ0tv97mlPvvSCRyD0DOM',
                                                        content_type='text/xml')
 
-            self.assertEqual(resp.request.query, 'pagenumber=13&pagesize=13&filter=tags:in:[stocks,market]')
+            self.assertEqual(resp.request.query, 'pagenumber=13&pagesize=13&filter=tags:in:%5bstocks,market%5d')
 
     def test_sort_asc(self):
         with requests_mock.mock() as m:
