Fix a hidden but kind of big bug that has been hiding for a while.

gwax · Jul 1, 2016 · 591ef22 · 591ef22
1 parent d31fd10
commit 591ef22
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 3 deletions.
diff --git a/README.rst b/README.rst
@@ -149,6 +149,12 @@ Acknowledgments
 Changelog
 =========
 
+1.3.1
+=====
+
+- Fix bug where were were never actually reading set names from xlsx
+  files.
+
 1.3.0
 -----
 

diff --git a/mtg_ssm/serialization/xlsx.py b/mtg_ssm/serialization/xlsx.py
@@ -198,7 +198,7 @@ def counts_from_sheet(sheet):
     header = [cell.value for cell in next(rows)]
     for row in rows:
         row_values = [cell.value for cell in row]
-        yield dict(zip(header, row_values))
+        yield dict(zip(header, row_values), set=sheet.title)
 
 
 class XlsxDialect(interface.SerializationDialect):

diff --git a/mtg_ssm/version.py b/mtg_ssm/version.py
@@ -1,2 +1,2 @@
 """Version information."""
-__version__ = '1.3.0'
+__version__ = '1.3.1'
diff --git a/tests/serialization/test_xlsx.py b/tests/serialization/test_xlsx.py
@@ -157,7 +157,7 @@ def test_counts_from_sheet():
     sheet.append([1, 'B', '=5+7'])
     rows = xlsx.counts_from_sheet(sheet)
     assert list(rows) == [
-        {'A': 1, 'B': 'B', 'C': '=5+7'},
+        {'set': 'Sheet', 'A': 1, 'B': 'B', 'C': '=5+7'},
     ]