Cleaning up tests so travis-ci can automate them all

Author: Neil Grey

.travis.yml

@@ -4,11 +4,11 @@ python:
   - "2.7"
 # command to install dependencies
 install:
-  - pip install nose --use-mirrors
+  - pip install nose
 before_script:
   - cp ./tests/example_config ./tests/config
 # command to run tests
-script: nosetests tests.tests_unit tests.test_client
+script: nosetests -w tests/
 notifications:
   email:
     - api@shotgunsoftware.com

shotgun_api3/shotgun.py

@@ -409,7 +409,7 @@ def __init__(self,
         else:
             self.__ca_certs = os.environ.get('SHOTGUN_API_CACERTS')
 
-        self.base_url = (base_url or "").lower()
+        self.base_url = (base_url or "")
         self.config.scheme, self.config.server, api_base, _, _ = \
             urlparse.urlsplit(self.base_url)
         if self.config.scheme not in ("http", "https"):

tests/base.py

@@ -1,4 +1,5 @@
 """Base class for Shotgun API tests."""
+import os
 import re
 import unittest
 from ConfigParser import ConfigParser
@@ -300,13 +301,13 @@ def __init__(self):
         self.shot_code      = None
         self.task_content   = None
 
-
     def read_config(self, config_path):
         config_parser = ConfigParser()
         config_parser.read(config_path)
         for section in config_parser.sections():
             for option in config_parser.options(section):
-                value = config_parser.get(section, option)
+                value = (os.environ.get('SG_{0}'.format(str(option).upper())) or
+                        config_parser.get(section, option))
                 setattr(self, option, value)
 
 

tests/test_client.py

@@ -240,7 +240,7 @@ def test_rpc_error(self):
 
         try:
             self.sg.info()
-        except api.Fault, e:
+        except api.Fault as e:
             self.assertEqual("Go BANG", str(e))
 
     def test_call_rpc(self):