Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix autocomplete terms unicode decode utf-8 error #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rimbendhaou
Copy link

Using : Python 2.7 and Windows OS
error:
File "U:\wd\dar-dev\ant-ws\build\opt\nose2_html_report-0.6.0\Lib\site-packages\nose2_html_report\html_report.py", line 109, in afterSummaryReport
'autocomplete_terms': json.dumps(self.generate_search_terms()),
File "D:\Program Files (x86)\Python27\lib\json_init
.py", line 244, in dumps
return _default_encoder.encode(obj)
File "D:\Program Files (x86)\Python27\lib\json\encoder.py", line 207, in encode
chunks = self.iterencode(o, _one_Generating HTML reportshot=True)
File "D:\Program Files (x86)\Python27\lib\json\encoder.py", line 270, in iterencode
return _iterencode(o, 0)
UnicodeDecodeError: 'utf8' codec can't decode byte 0xea in position 1: invalid continuation byte

Using : Python 2.7 and Windows OS
error:
File "U:\wd\dar-dev\ant-ws\build\opt\nose2_html_report-0.6.0\Lib\site-packages\nose2_html_report\html_report.py", line 109, in afterSummaryReport
       'autocomplete_terms': json.dumps(self._generate_search_terms()),
        File "D:\Program Files (x86)\Python27\lib\json\__init__.py", line 244, in dumps
         return _default_encoder.encode(obj)
      File "D:\Program Files (x86)\Python27\lib\json\encoder.py", line 207, in encode
        chunks = self.iterencode(o, _one_Generating HTML reportshot=True)
        File "D:\Program Files (x86)\Python27\lib\json\encoder.py", line 270, in iterencode
         return _iterencode(o, 0)
      UnicodeDecodeError: 'utf8' codec can't decode byte 0xea in position 1: invalid continuation byte
@coveralls
Copy link

Coverage Status

Coverage remained the same at 87.248% when pulling 1acef8c on rimbendhaou:fix_autocomplete_terms_unicode into e8c89be on mgrijalva:master.

@mgrijalva
Copy link
Owner

This looks reasonable to me. Would you mind adding a quick test? Since it seems you have a reproducible case already.

@rimbendhaou
Copy link
Author

This looks reasonable to me. Would you mind adding a quick test? Since it seems you have a reproducible case already.

It doesnt need a new test case. but travis is configured to run tests on linux environment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants