Updated test cases, version and refactored some comments for PEP8 com…

…pliancy

HISTORY.rst

@@ -3,6 +3,10 @@
 Release History
 ---------------
 
+0.1.9 (2018-02-22)
+++++++++++++++++++
+- Use find_library to load libiperf (Thanks to @Austinpayne). This should allow iperf3 lib to run on Mac OS X.
+
 0.1.8 (2018-01-24)
 ++++++++++++++++++
 - Fixed segmentation fault on several Linux distro's (Thanks to @illu89)

docs/source/conf.py

@@ -66,9 +66,9 @@
 # built documents.
 #
 # The short X.Y version.
-version = '0.1.8'
+version = '0.1.9'
 # The full version, including alpha/beta/rc tags.
-release = '0.1.8'
+release = '0.1.9'
 
 # The language for content autogenerated by Sphinx. Refer to documentation
 # for a list of supported languages.

iperf3/iperf3.py

@@ -31,7 +31,7 @@ class provides common settings for the :class:`Client` and :class:`Server`
     from Queue import Queue  # Python2 compatibility
 
 
-__version__ = '0.1.8'
+__version__ = '0.1.9'
 
 
 MAX_UDP_BULKSIZE = (65535 - 8 - 20)
@@ -802,24 +802,33 @@ def __init__(self, result):
 
             # TCP specific test results
             if self.protocol == 'TCP':
-                self.sent_bytes = self.json['end']['sum_sent']['bytes']
-                self.sent_bps = self.json['end']['sum_sent']['bits_per_second']
-                self.received_bytes = self.json['end']['sum_received']['bytes']
-                self.received_bps = self.json['end']['sum_received']['bits_per_second']
-
-                #  Bits are measured in 10**3 terms, Bytes are measured in 2**10 terms
-                self.sent_kbps = self.sent_bps / 1000           # Kilobits per second
-                self.sent_Mbps = self.sent_kbps / 1000          # Megabits per second
-                self.sent_kB_s = self.sent_bps / (8 * 1024)     # kiloBytes per second
-                self.sent_MB_s = self.sent_kB_s / 1024          # MegaBytes per second
-
-                self.received_kbps = self.received_bps / 1000   # Kilobits per second
-                self.received_Mbps = self.received_kbps / 1000  # Megabits per second
-                self.received_kB_s = self.received_bps / (8* 1024)     # kiloBytes per second
-                self.received_MB_s = self.received_kB_s / 1024     # MegaBytes per second
+                sent_json = self.json['end']['sum_sent']
+                self.sent_bytes = sent_json['bytes']
+                self.sent_bps = sent_json['bits_per_second']
+
+                recv_json = self.json['end']['sum_received']
+                self.received_bytes = recv_json['bytes']
+                self.received_bps = recv_json['bits_per_second']
+
+                # Bits are measured in 10**3 terms
+                # Bytes are measured in 2**10 terms
+                # kbps = Kilobits per second
+                # Mbps = Megabits per second
+                # kB_s = kiloBytes per second
+                # MB_s = MegaBytes per second
+
+                self.sent_kbps = self.sent_bps / 1000
+                self.sent_Mbps = self.sent_kbps / 1000
+                self.sent_kB_s = self.sent_bps / (8 * 1024)
+                self.sent_MB_s = self.sent_kB_s / 1024
+
+                self.received_kbps = self.received_bps / 1000
+                self.received_Mbps = self.received_kbps / 1000
+                self.received_kB_s = self.received_bps / (8 * 1024)
+                self.received_MB_s = self.received_kB_s / 1024
 
                 # retransmits only returned from client
-                self.retransmits = self.json['end']['sum_sent'].get('retransmits')
+                self.retransmits = sent_json.get('retransmits')
 
             # UDP specific test results
             elif self.protocol == 'UDP':

tests/test_iperf3.py

@@ -9,10 +9,6 @@ def isclose(a, b, rel_tol=1e-09, abs_tol=0.0):
 
 class TestPyPerf:
 
-    def test_unavailable_library(self):
-        with pytest.raises(OSError):
-            client = iperf3.Client(lib_name='bla')
-
     def test_init_client(self):
         client = iperf3.Client()
         assert client._test