Author: batiste.bieler
Date: Mon Jul 13 02:44:51 2009
New Revision: 605

Modified:
    trunk/pages/admin/views.py
    trunk/pages/media/pages/javascript/pages_list.js
    trunk/pages/tests/pages_tests.py

Log:
Add a bunch of not very bright tests

Modified: trunk/pages/admin/views.py
==============================================================================
--- trunk/pages/admin/views.py  (original)
+++ trunk/pages/admin/views.py  Mon Jul 13 02:44:51 2009
@@ -60,19 +60,9 @@
  get_content = staff_member_required(get_content)
  get_content = auto_render(get_content)

-def valid_targets_list(request, page_id):
-    """A list of valid targets to move a page"""
-    if not settings.PAGE_PERMISSION:
-        perms = "All"
-    else:
-        from pages.models import PagePermission
-        perms = PagePermission.objects.get_page_id_list(request.user)
-    query = Page.objects.valid_targets(page_id, request, perms)
-    return HttpResponse(",".join([str(p.id) for p in query]))
-valid_targets_list = staff_member_required(valid_targets_list)
-
  def sub_menu(request, page_id):
-    """Render the children of the requested page"""
+    """Render the children of the requested page with the sub_menu
+    template."""
      page = Page.objects.get(id=page_id)
      pages = page.children.all()
      has_permission = page.has_page_permission(request)

Modified: trunk/pages/media/pages/javascript/pages_list.js
==============================================================================
--- trunk/pages/media/pages/javascript/pages_list.js    (original)
+++ trunk/pages/media/pages/javascript/pages_list.js    Mon Jul 13 02:44:51  
2009
@@ -161,6 +161,7 @@
                          // Update the move and add links of the inserted  
rows
                          if (action == 'move') {
                               
$('#page-row-'+selected_page).addClass('selected').add(get_children(selected_page)).addClass('highlighted');
+                            // this could become quite slow with a lot of  
pages
                              $('tr:not(.highlighted)',  
changelist).addClass('insertable');
                          } else if (action == 'add') {
                               
$('#page-row-'+selected_page).addClass('highlighted insertable');

Modified: trunk/pages/tests/pages_tests.py
==============================================================================
--- trunk/pages/tests/pages_tests.py    (original)
+++ trunk/pages/tests/pages_tests.py    Mon Jul 13 02:44:51 2009
@@ -91,6 +91,7 @@
          page_data = self.get_new_page_data()
          page_data['status'] = Page.PUBLISHED
          page_data['slug'] = 'test-page-2'
+        page_data['template'] = 'pages/index.html'
          response = c.post('/admin/pages/page/add/', page_data)
          self.assertRedirects(response, '/admin/pages/page/')

@@ -613,3 +614,16 @@
          url = '/admin/pages/page/%d/modify-content/title/en-us/' % page.id
          response = c.post(url, {'content': 'test content'})
          self.assertEqual(page.title(), 'test content')
+
+        # TODO: realy test these methods
+        url = '/admin/pages/page/%d/traduction/en-us/' % page.id
+        response = c.get(url)
+        self.assertEqual(response.status_code, 200)
+
+        url = '/admin/pages/page/%d/sub-menu/' % page.id
+        response = c.get(url)
+        self.assertEqual(response.status_code, 200)
+
+        url = '/admin/pages/page/%d/get-content/1/' % page.id
+        response = c.get(url)
+        self.assertEqual(response.status_code, 200)
\ No newline at end of file

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"pinax-updates" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/pinax-updates?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to