PEP8

backdrop/core/parse_excel.py

@@ -24,7 +24,8 @@ def _extract_values(row, book):
 
 def _extract_cell_value(cell, book):
     if cell.ctype == 3:
-        return utc(datetime.datetime(*xlrd.xldate_as_tuple(cell.value, book.datemode)))
+        time_tuple = xlrd.xldate_as_tuple(cell.value, book.datemode)
+        return utc(datetime.datetime(*time_tuple))
     elif cell.ctype == 5:
         raise ParseError("Error encountered in cell")
-    return cell.value
+    return cell.value

tests/core/test_parse_excel.py

@@ -17,7 +17,6 @@ def test_parse_an_xlsx_file(self):
                         {"name": "Pawel", "age": 27, "nationality": "Polish"},
                         {"name": "Max", "age": 35, "nationality": "Italian"}))
 
-
     def test_parse_xlsx_dates(self):
         assert_that(self._parse_excel("dates.xlsx"),
                     only_contains(
@@ -31,4 +30,4 @@ def test_parse_xls_file(self):
                         {"date": d_tz(2013, 12, 4), "name": "test2", "number": 34}))
 
     def test_parse_xlsx_with_error(self):
-        self.assertRaises(ParseError, self._parse_excel, "error.xlsx")
+        self.assertRaises(ParseError, self._parse_excel, "error.xlsx")

tests/support/test_helpers.py

@@ -71,4 +71,4 @@ def d(year, month, day, hour=0, minute=0, second=0):
 
 
 def fixture_path(name):
-    return os.path.abspath(os.path.join(os.path.dirname(__file__), '..', '..', 'features', 'fixtures', name))
+    return os.path.abspath(os.path.join(os.path.dirname(__file__), '..', '..', 'features', 'fixtures', name))