Merge pull request #7 from RachelTucker/pythonsdk-49

Added tests and fixed bug

Author: GraciesPadre

ds3/ds3network.py

@@ -312,7 +312,7 @@ def remove_http_from_endpoint(self, endpoint):
     if not endpoint.startswith('http'):
       return endpoint
     index = endpoint.find('://')
-    if endpoint >= 0:
+    if index >= 0:
       return endpoint[index+3:]
     return endpoint
 

tests/clientTests.py

@@ -1314,4 +1314,21 @@ def testGetBigFile(self):
         f.close()
         os.close(fd)
         os.remove(tempname)
-        
+
+
+class Ds3NetworkTestCase(Ds3TestCase):
+    def testEncodingQueryParamsWithSymbols(self):
+        expected = "/test/path?key=value%3Bwith%20symbols"
+        query_params = {"key": "value;with symbols"}
+
+        result = self.client.net_client.build_path("/test/path", query_params)
+
+        self.assertEqual(result, expected)
+
+    def testEncodingPathWithObjectNameWithPlus(self):
+        expected = "/test/path/name%2Bwith%2Bplus"
+        obj_name = "name+with+plus"
+
+        result = self.client.net_client.build_path("/test/path/" + obj_name)
+
+        self.assertEqual(result, expected)