Added 'view on site' link to catalogue list page

oscar/templates/dashboard/catalogue/product_list.html

@@ -82,6 +82,7 @@ Catalogue management | {{ block.super }}
 		{% endwith %}
 		<td>
 			<a class="btn btn-info" href="{% url dashboard:catalogue-product product.id %}">Edit</a>
+			<a class="btn btn-success" href="{{ product.get_absolute_id }}">View on site</a>
 		</td>
     </tr>
     {% endfor %}

tests/functional/dashboard/catalogue_tests.py

@@ -7,10 +7,10 @@ from oscar.apps.dashboard.catalogue.forms import CategoryForm
 from oscar.apps.catalogue.categories import create_from_breadcrumbs
 
 
-class ViewTests(ClientTestCase):
+class TestCatalogueViews(ClientTestCase):
     is_staff = True
 
-    def test_pages_exist(self):
+    def test_exist(self):
         urls = [reverse('dashboard:catalogue-product-list'),
                 reverse('dashboard:catalogue-category-list'),
                 reverse('dashboard:stock-alert-list'),
@@ -30,7 +30,7 @@ def create_test_category_tree():
     create_from_breadcrumbs(trail)
 
 
-class CategoryFormTests(TestCase):
+class TestCategoryForm(TestCase):
 
     def setUp(self):
         Category.objects.all().delete()

tests/functional/dashboard_tests.py

@@ -7,18 +7,18 @@ from oscar.test import ClientTestCase
 from oscar.test.helpers import create_order
 
 
-class AnonymousUserTests(ClientTestCase):
+class TestDashboardIndexForAnonUser(ClientTestCase):
     is_anonymous = True
 
-    def test_login_form_is_displayed_for_anon_user(self):
+    def test_is_not_available(self):
         response = self.client.get(reverse('dashboard:index'), follow=True)
         self.assertContains(response, 'login-username', status_code=200)
 
 
-class DashboardViewTests(ClientTestCase):
+class TestDashboardIndexForStaffUser(ClientTestCase):
     is_staff = True
 
-    def test_dashboard_index_is_for_staff_only(self):
+    def test_is_available(self):
         urls = ('dashboard:index',
                 'dashboard:order-list',
                 'dashboard:users-index',)
@@ -26,7 +26,7 @@ class DashboardViewTests(ClientTestCase):
             response = self.client.get(reverse(name))
             self.assertTrue('Password' not in response.content)
 
-    def test_dashboard_hourly_report_with_no_orders(self):
+    def test_includes_hourly_report_with_no_orders(self):
         report = IndexView().get_hourly_report()
         self.assertItemsEqual(report, ['order_total_hourly', 'max_revenue',
                                        'y_range'])
@@ -34,7 +34,7 @@ class DashboardViewTests(ClientTestCase):
         self.assertEquals(len(report['y_range']), 0)
         self.assertEquals(report['max_revenue'], 0)
 
-    def test_dashboard_hourly_report_with_orders(self):
+    def test_includes_hourly_report_with_orders(self):
         create_order(total_incl_tax=D('34.05'), total_excl_tax=D('34.05'))
         create_order(total_incl_tax=D('21.90'), total_excl_tax=D('21.90'))
         report = IndexView().get_hourly_report()
@@ -43,7 +43,7 @@ class DashboardViewTests(ClientTestCase):
         self.assertEquals(len(report['y_range']), 11)
         self.assertEquals(report['max_revenue'], D('60'))
 
-    def test_dashboard_index_has_stats_vars_in_context(self):
+    def test_has_stats_vars_in_context(self):
         response = self.client.get(reverse('dashboard:index'))
 
         self.assertInContext(response, 'total_orders_last_day')