add travis yml, increase test coverage

Author: radzhome

.travis.yml

@@ -0,0 +1,13 @@
+language: python
+sudo: false
+python:
+  - "2.7"
+  - "3.5"
+
+# command to install dependencies
+install:
+  - pip install .
+  - pip install -r requirements.txt
+
+# command to run tests
+script: nosetests

CHANGES.rst

@@ -8,6 +8,8 @@ Change Log
 * Added [Validation]AvailabilityAndCommitmentService_v4.wsdl for AVC service. (radzhome)
 * Added examples and unit tests for AVC service.
 * Refactored examples and documentation. (radzhome)
+* A quick PEP8 pass using pycharm on most of the codebase (radzhome)
+* Add travis yml (radzhome)
 
 
 2.0.0

fedex/config.py

@@ -70,5 +70,5 @@ def __init__(self, key, password, account_number=None, meter_number=None, freigh
         if wsdl_path is None:
             self.wsdl_path = os.path.join(os.path.dirname(os.path.abspath(__file__)),
                                           'wsdl')
-        else:
+        else:  # pragma: no cover
             self.wsdl_path = wsdl_path

fedex/services/package_movement.py

@@ -57,7 +57,7 @@ def _check_response_for_request_errors(self):
         this WSDL.
         """
         if self.response.HighestSeverity == "ERROR":
-            for notification in self.response.Notifications:
+            for notification in self.response.Notifications:  # pragma: no cover
                 if notification.Severity == "ERROR":
                     if "Postal Code Not Found" in notification.Message:
                         raise FedexPostalCodeNotFound(notification.Code,

fedex/services/ship_service.py

@@ -210,6 +210,6 @@ def _assemble_and_send_request(self):
             ClientDetail=self.ClientDetail,
             TransactionDetail=self.TransactionDetail,
             Version=self.VersionId,
-            ShipTimestamp=datetime.now(),
+            ShipTimestamp=datetime.datetime.now(),
             TrackingId=self.TrackingId,
             DeletionControl=self.DeletionControlType)

fedex/services/track_service.py

@@ -82,7 +82,7 @@ def _check_response_for_request_errors(self):
         Checks the response to see if there were any errors specific to
         this WSDL.
         """
-        if self.response.HighestSeverity == "ERROR":
+        if self.response.HighestSeverity == "ERROR":  # pragma: no cover
             for notification in self.response.Notifications:
                 if notification.Severity == "ERROR":
                     if "Invalid tracking number" in notification.Message:

requirements.txt

@@ -0,0 +1 @@
+suds-jurko

tests/test_ship_service.py

@@ -68,6 +68,7 @@ def test_create_delete_shipment(self):
         package1.Weight = package1_weight
         shipment.add_package(package1)
 
+        shipment.send_validation_request()
         shipment.send_request()
 
         assert shipment.response