Merge pull request #4367 from opendatazurich/4346-missing-legacy-routes

[#4346] Missing legacy routes
ckan · Aug 3, 2018 · 5710d3f · 5710d3f
2 parents 7a9b596 + cd6f643
commit 5710d3f
Show file tree

Hide file tree

Showing 2 changed files with 33 additions and 0 deletions.
diff --git a/ckan/lib/helpers.py b/ckan/lib/helpers.py
@@ -69,6 +69,7 @@
     'home': 'home.index',
     'about': 'home.about',
     'search': 'dataset.search',
+    'group_index': 'group.index',
     'organizations_index': 'organization.index'
 }
 

diff --git a/ckan/tests/lib/test_helpers.py b/ckan/tests/lib/test_helpers.py
@@ -532,6 +532,38 @@ def test_non_string(self):
             u'2018-01-05 10:48:23.463511')
 
 
+class TestBuildNavMain(object):
+    def test_flask_routes(self):
+        menu = (
+            ('home.index', 'Home'),
+            ('dataset.search', 'Datasets'),
+            ('organization.index', 'Organizations'),
+            ('group.index', 'Groups'),
+            ('home.about', 'About')
+        )
+        eq_(h.build_nav_main(*menu), (
+            '<li><a href="/">Home</a></li>'
+            '<li><a href="/dataset/">Datasets</a></li>'
+            '<li><a href="/organization/">Organizations</a></li>'
+            '<li><a href="/group/">Groups</a></li>'
+            '<li><a href="/about">About</a></li>'))
+
+    def test_legacy_pylon_routes(self):
+        menu = (
+            ('home', 'Home'),
+            ('search', 'Datasets'),
+            ('organizations_index', 'Organizations'),
+            ('group_index', 'Groups'),
+            ('about', 'About')
+        )
+        eq_(h.build_nav_main(*menu), (
+            '<li><a href="/">Home</a></li>'
+            '<li><a href="/dataset/">Datasets</a></li>'
+            '<li><a href="/organization/">Organizations</a></li>'
+            '<li><a href="/group/">Groups</a></li>'
+            '<li><a href="/about">About</a></li>'))
+
+
 class TestHelperException(helpers.FunctionalTestBase):
 
     @raises(ckan.exceptions.HelperError)