style: pep8, pyflakes

samael500 · Oct 24, 2014 · 84f5745 · 84f5745
1 parent 6dc5198
commit 84f5745
Show file tree

Hide file tree

Showing 5 changed files with 17 additions and 14 deletions.
diff --git a/sevboats/sb.py b/sevboats/sb.py
@@ -1,10 +1,10 @@
-from src.engine import *
-from src.engine import _update_ais_data, _update_ais_lastpos
-from src.coordinates import Coordinates
+# from src.engine import *
+# from src.engine import _update_ais_data, _update_ais_lastpos
+# from src.coordinates import Coordinates
 
-initial()
-_update_ais_data()
-_update_ais_lastpos()
+# initial()
+# _update_ais_data()
+# _update_ais_lastpos()
 
-SlL.fleet().print_ships(True)
-print 'on routes: ', check_ships_on_routes()
+# SlL.fleet().print_ships(True)
+# print 'on routes: ', check_ships_on_routes()
diff --git a/sevboats/src/engine.py b/sevboats/src/engine.py
@@ -78,9 +78,10 @@ def _check_ships_on_routes():
                 potential += 1
 
     if not moved_on_routes and (potential >= Ship.CRITICAL_MINIMUM):
-         return potential
+        return potential
     return moved_on_routes
 
+
 def send_fleet_message():
     """ Check fleet status on ais and return message to send to twitter """
     _update_ais_data()

diff --git a/sevboats/src/ships.py b/sevboats/src/ships.py
@@ -123,7 +123,7 @@ def update(self, data):
 
     def update_status(self):
         """ Change ship status """
-        status_old = self.status
+        status_old = self.status  # noqa
         status_new = None
         # check different situation
         if self.coordinates is None:

diff --git a/sevboats/tests/scrapper_test.py b/sevboats/tests/scrapper_test.py
@@ -16,10 +16,13 @@ def setUp(self):
         # change timelimit and scrapp urls
         self.scrapper.timeout = 0
         file_url = r'file:///'
-        self.scrapper.ships_list_url = file_url + os.path.join(BASE_DIR, 'sevboats', 'tests', 'test_data', 'list-page:{page}.html')
+        self.scrapper.ships_list_url = file_url + os.path.join(
+            BASE_DIR, 'sevboats', 'tests', 'test_data', 'list-page:{page}.html')
         self.scrapper.ship_info_url = file_url + os.path.join(BASE_DIR, 'sevboats', 'tests', 'test_data', 'list.html')
-        self.scrapper.ship_lastpos_url = file_url + os.path.join(BASE_DIR, 'sevboats', 'tests', 'test_data', 'lastpos.html')
-        self.scrapper.none_url = file_url + os.path.join(BASE_DIR, 'sevboats', 'tests', 'test_data', '{types}_none.html')
+        self.scrapper.ship_lastpos_url = file_url + os.path.join(
+            BASE_DIR, 'sevboats', 'tests', 'test_data', 'lastpos.html')
+        self.scrapper.none_url = file_url + os.path.join(
+            BASE_DIR, 'sevboats', 'tests', 'test_data', '{types}_none.html')
         self.mmsi_list = (
             '272083500', '272083600', '272083700', '272083800', '272092200', '272093800', '272093900',
             '272094100', '272094200', '272094300', '272094700', '272100500', '272105400', '272105500',

diff --git a/sevboats/tests/ship_test.py b/sevboats/tests/ship_test.py
@@ -50,4 +50,3 @@ def test_ship_str(self):
         """ Check str and unicode """
         self.assertEquals('URAN: OFFLINE', str(self.ship))
         self.assertEquals(u'Уран: 272105400 - OFFLINE', unicode(self.ship))
-