Merge pull request #2 from BenjamenMeyer/bugfix_fix-pep8

Bugfix fix pep8

Author: BenjamenMeyer

.coveragerc

@@ -0,0 +1,12 @@
+[report]
+fail_under = 100
+exclude_lines =
+    # Have to re-enable the standard pragma
+    pragma: no cover
+
+    # Don't complain if tests don't hit defensive assertion code:
+    raise NotImplementedError
+
+[run]
+omit = *tests*
+branch = True

.travis.yml

@@ -0,0 +1,27 @@
+language: python
+matrix:
+    include:
+        - python: "2.7"
+          env: TEST_SUITE=suite_2_7 TOX_ENV=py27
+        - python: "pypy"
+          env: TEST_SUITE=suite_pypy TOX_ENV=pypy
+        - python: "3.4"
+          env: TEST_SUITE=suite_3_4 TOX_ENV=py34
+        - python: "3.5"
+          env: TEST_SUITE=suite_3_5 TOX_ENV=py35
+        - python: "3.4"
+          env: TEST_SUITE=suite_3_4 TOX_ENV=pep8
+
+before_install:
+  - sudo apt-get install build-essential python-dev
+
+install: pip install tox setuptools virtualenv coveralls --upgrade
+
+script: tox -v -e $TOX_ENV -- -v
+
+after_success: coveralls
+
+notifications:
+  email:
+    - bm_witness@yahoo.com
+

pyinstance/__init__.py

@@ -66,9 +66,12 @@ def __del__(self):
         if self.__session in PyInstance.pyinstances:
             # Decrement the reference counter
             PyInstance.pyinstances[self.__session][PyInstance.__COUNTER] = (
-                PyInstance.pyinstances[self.__session][PyInstance.__COUNTER] - 1
+                PyInstance.pyinstances[
+                    self.__session][PyInstance.__COUNTER] - 1
             )
 
             # If zero, clear out the session from the instances dict
-            if not PyInstance.pyinstances[self.__session][PyInstance.__COUNTER]:
+            if not (
+                PyInstance.pyinstances[self.__session][PyInstance.__COUNTER]
+            ):
                 del PyInstance.pyinstances[self.__session]

tests/requires.txt

@@ -1,2 +1,3 @@
 nose2
+nose2-cov
 six

tests/test_derived.py

@@ -60,6 +60,7 @@ def test_derived_multiple(self):
                 pyinstance.PyInstance.pyinstances[k]['instance'],
                 DerivedInstance
             )
+            v.__del__()
 
     def test_derived_repeats(self):
         session_names = [
@@ -75,7 +76,7 @@ def test_derived_repeats(self):
             id(v)
             for k, v in six.iteritems(sessions)
         ]
-        
+
         self.assertEqual(
             len(session_names),
             len(ids)

tests/test_direct.py

@@ -52,6 +52,7 @@ def test_direct_multiple(self):
                 pyinstance.PyInstance.pyinstances[k]['instance'],
                 pyinstance.PyInstance
             )
+            v.__del__()
 
     def test_direct_repeats(self):
         session_names = [
@@ -89,3 +90,6 @@ def test_direct_repeats(self):
                 pyinstance.PyInstance.pyinstances[s]['instance'],
                 pyinstance.PyInstance
             )
+
+        for k, v in six.iteritems(sessions):
+            v.__del__()

unittest.cfg

@@ -0,0 +1,6 @@
+[coverage]
+always-on = True
+coverage = pyinstance
+coverage-report = html
+                  term-missing
+				  term