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

Side by Side Diff: tests/test_generation_exceptional_cases.py

Issue 29887585: Issue #5352 - generate_static_pages cannot deal with directories being turned into regular pages (Closed)
Patch Set: Addressed comments from Patch Set #2 Created Oct. 4, 2018, 6:31 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « tests/expected_output/en/sitemap ('k') | tests/test_site/pages/foo/bar.md » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 # This file is part of the Adblock Plus web scripts,
2 # Copyright (C) 2006-present eyeo GmbH
3 #
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
6 # published by the Free Software Foundation.
7 #
8 # Adblock Plus is distributed in the hope that it will be useful,
9 # but WITHOUT ANY WARRANTY; without even the implied warranty of
10 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
11 # GNU General Public License for more details.
12 #
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/>.
15
16 import os
17
18 import pytest
19
20 from cms.bin.generate_static_pages import generate_pages
21
22
23 @pytest.fixture
24 def target_dir_with_file(tmpdir):
25 target_dir = tmpdir.mkdir('out_file').strpath
26 os.mkdir(os.path.join(target_dir, 'en'))
27
28 with open(os.path.join(target_dir, 'en', 'foo'), 'w') as f:
29 f.write('test\n')
30
31 yield target_dir
32
33
34 @pytest.fixture
35 def target_dir_with_dir(tmpdir):
36 target_dir = tmpdir.mkdir('out_dir').strpath
37 os.makedirs(os.path.join(target_dir, 'en', 'translate'))
38
39 yield target_dir
40
41
42 @pytest.fixture
43 def target_dir_with_fifo(tmpdir):
44 target_dir = tmpdir.mkdir('out_dir').strpath
45 os.mkdir(os.path.join(target_dir, 'en'))
46 os.mkfifo(os.path.join(target_dir, 'en', 'translate'))
47
48 yield target_dir
49
50
51 def test_generate_dir_instead_of_file(temp_site, target_dir_with_file):
52 """Case where a file from previous version becomes a directory."""
53 generate_pages(str(temp_site), str(target_dir_with_file))
54
55 assert os.path.isdir(os.path.join(target_dir_with_file, 'en', 'foo'))
56
57
58 def test_generate_file_instead_of_dir(temp_site, target_dir_with_dir):
59 """Case where a directory from previous version becomes a file."""
60 generate_pages(str(temp_site), str(target_dir_with_dir))
61
62 assert os.path.isfile(os.path.join(target_dir_with_dir, 'en', 'translate'))
63
64
65 @pytest.mark.script_launch_mode('subprocess')
66 def test_generate_fifo_instead_of_file(temp_site, target_dir_with_fifo,
67 script_runner):
68 cmd = ['python', '-m', 'cms.bin.generate_static_pages', str(temp_site),
69 str(target_dir_with_fifo)]
70
71 ret = script_runner.run(*cmd)
72
73 assert not ret.success
74 assert 'It is neither a file, nor a directory!' in ret.stderr
OLDNEW
« no previous file with comments | « tests/expected_output/en/sitemap ('k') | tests/test_site/pages/foo/bar.md » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld