Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Delta Between Two Patch Sets: tests/test_xtm_translations_utils.py

Issue 29906560: Issue 7042 - [XTM Integration] Solve UnicodeDecodeErrors occuring (Closed)
Left Patch Set: Created Oct. 10, 2018, 1:53 p.m.
Right Patch Set: Fixed flake8 docstring error Created Oct. 16, 2018, 12:19 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « tests/test_site/pages/translate-unicode.md ('k') | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 # This file is part of the Adblock Plus web scripts, 1 # This file is part of the Adblock Plus web scripts,
2 # Copyright (C) 2006-present eyeo GmbH 2 # Copyright (C) 2006-present eyeo GmbH
3 # 3 #
4 # Adblock Plus is free software: you can redistribute it and/or modify 4 # Adblock Plus is free software: you can redistribute it and/or modify
5 # it under the terms of the GNU General Public License version 3 as 5 # it under the terms of the GNU General Public License version 3 as
6 # published by the Free Software Foundation. 6 # published by the Free Software Foundation.
7 # 7 #
8 # Adblock Plus is distributed in the hope that it will be useful, 8 # Adblock Plus is distributed in the hope that it will be useful,
9 # but WITHOUT ANY WARRANTY; without even the implied warranty of 9 # but WITHOUT ANY WARRANTY; without even the implied warranty of
10 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the 10 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
11 # GNU General Public License for more details. 11 # GNU General Public License for more details.
12 # 12 #
13 # You should have received a copy of the GNU General Public License 13 # You should have received a copy of the GNU General Public License
14 # along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>. 14 # along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
15
15 from __future__ import unicode_literals 16 from __future__ import unicode_literals
Vasily Kuznetsov 2018/10/10 14:01:26 Nit: empty line after the top comment
Tudor Avram 2018/10/11 13:58:28 Done.
16 17
17 import json 18 import json
18 import os 19 import os
19 20
20 import pytest 21 import pytest
21 22
22 from cms.translations.xtm import utils 23 from cms.translations.xtm import utils
23 import cms.translations.xtm.constants as const 24 import cms.translations.xtm.constants as const
24 from cms.sources import FileSource 25 from cms.sources import FileSource
25 from cms.translations.xtm.xtm_api import XTMCloudAPI 26 from cms.translations.xtm.xtm_api import XTMCloudAPI
(...skipping 189 matching lines...) Expand 10 before | Expand all | Expand 10 after
215 @pytest.mark.parametrize('path', ['de/test.json', 'de/dir1/dir2/test.json']) 216 @pytest.mark.parametrize('path', ['de/test.json', 'de/dir1/dir2/test.json'])
216 def test_write_data(toydir, path): 217 def test_write_data(toydir, path):
217 """Test if writing data to files works as expected.""" 218 """Test if writing data to files works as expected."""
218 data = bytes(json.dumps({'a': 'b'})) 219 data = bytes(json.dumps({'a': 'b'}))
219 220
220 utils.write_to_file(data, str(toydir.join(path))) 221 utils.write_to_file(data, str(toydir.join(path)))
221 222
222 assert toydir.join(path).read('rb') == data 223 assert toydir.join(path).read('rb') == data
223 224
224 225
225 def test_extract_unicode_strings(tmpsite_with_unicode): 226 def test_extract_unicode_strings(temp_site):
226 """Test correct extraction of unicode strings for translation.""" 227 """Test correct extraction of unicode strings for translation."""
227 with FileSource(tmpsite_with_unicode) as fs: 228 with FileSource(temp_site) as fs:
228 strings = utils.extract_strings(fs) 229 strings = utils.extract_strings(fs)
229 230
230 assert '\u1234' in strings['unicode']['simple']['message'] 231 assert '\u0376' in strings['translate-unicode']['simple']['message']
LEFTRIGHT

Powered by Google App Engine
This is Rietveld