annotate tests/test_data_provider.py @ 380:f33712c4cfab

routing: Fix bugs with matching URLs with correct route but missing metadata. When matching a route like `/foo/%slug%` against an URL like `/foo`, the route will (correctly) return a match, but it will be completely missing the `slug` metadata, resulting in problems elsewhere. This change makes it so that any missing route metadata will be filled in with an empty string. And because this means generated URLs may differ from the incoming URL when using trailing slashes (`/foo/` _vs._ `/foo`), we make the assert in the chef server handle those discrepancies.
author Ludovic Chabant <ludovic@chabant.com>
date Sun, 10 May 2015 00:34:21 -0700
parents 4b1019bb2533
children f987b29d6fab
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
369
4b1019bb2533 serve: Giant refactor to change how we handle data when serving pages.
Ludovic Chabant <ludovic@chabant.com>
parents: 321
diff changeset
1 from piecrust.rendering import QualifiedPage, PageRenderingContext, render_page
321
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
2 from .mockutil import mock_fs, mock_fs_scope
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
3
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
4
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
5 def test_blog_provider():
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
6 fs = (mock_fs()
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
7 .withPage('posts/2015-03-01_one.md',
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
8 {'title': 'One', 'category': 'Foo'})
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
9 .withPage('posts/2015-03-02_two.md',
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
10 {'title': 'Two', 'category': 'Foo'})
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
11 .withPage('posts/2015-03-03_three.md',
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
12 {'title': 'Three', 'category': 'Bar'})
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
13 .withPage('pages/categories.md',
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
14 {'format': 'none', 'layout': 'none'},
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
15 "{%for c in blog.categories%}\n"
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
16 "{{c.name}} ({{c.post_count}})\n"
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
17 "{%endfor%}\n"))
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
18 with mock_fs_scope(fs):
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
19 app = fs.getApp()
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
20 page = app.getSource('pages').getPage({'slug': 'categories'})
369
4b1019bb2533 serve: Giant refactor to change how we handle data when serving pages.
Ludovic Chabant <ludovic@chabant.com>
parents: 321
diff changeset
21 route = app.getRoute('pages', None)
4b1019bb2533 serve: Giant refactor to change how we handle data when serving pages.
Ludovic Chabant <ludovic@chabant.com>
parents: 321
diff changeset
22 route_metadata = {'slug': 'categories'}
4b1019bb2533 serve: Giant refactor to change how we handle data when serving pages.
Ludovic Chabant <ludovic@chabant.com>
parents: 321
diff changeset
23 qp = QualifiedPage(page, route, route_metadata)
4b1019bb2533 serve: Giant refactor to change how we handle data when serving pages.
Ludovic Chabant <ludovic@chabant.com>
parents: 321
diff changeset
24 ctx = PageRenderingContext(qp)
321
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
25 rp = render_page(ctx)
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
26 expected = "\nBar (1)\n\nFoo (2)\n"
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
27 assert rp.content == expected
59d65654184e tests: Add a blog data provider test.
Ludovic Chabant <ludovic@chabant.com>
parents:
diff changeset
28