Skip to content

Commit

Permalink
Merge pull request ssrikanta#14 from ssrikanta/python3_fix
Browse files Browse the repository at this point in the history
Fixes issue observed with report update in python3.6
  • Loading branch information
ssrikanta authored Feb 1, 2017
2 parents cf6978e + aeba805 commit 1cbc436
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 4 deletions.
7 changes: 6 additions & 1 deletion CHANGES.rst
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,12 @@ Release Notes

* Added support to include markers in the excel report

**1.2.1 (2017-01-3)**
**1.2.1 (2017-01-31)**

* Fix for the issue obserevd on python3.6
* Fix for unicode related issues in python3.6


**1.2.2 (2017-02-01)**

* Fixes issue with report update method in python3.6
4 changes: 2 additions & 2 deletions pytest_excel/pytest_excel.py
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ def create_sheet(self, column_heading):

def update_worksheet(self):
for data in self.results:
for key, value in data.iteritems():
for key, value in data.items():
self.wsheet.cell(row=self.rc, column=list(data).index(key) + 1).value = value
self.rc = self.rc + 1

Expand Down Expand Up @@ -187,7 +187,7 @@ def pytest_runtest_makereport(self, item, call):
report = outcome.get_result()
report.test_doc = item.obj.__doc__
test_marker = []
for k, v in item.keywords.iteritems():
for k, v in item.keywords.items():
if isinstance(v, MarkInfo):
test_marker.append(k)
report.test_marker = ', '.join(test_marker)
Expand Down
2 changes: 1 addition & 1 deletion version.txt
Original file line number Diff line number Diff line change
@@ -1 +1 @@
1.2.1
1.2.2

0 comments on commit 1cbc436

Please sign in to comment.