Add test for creating product with attributes

It was broken without anyone noticing, as reported in #1451. My work in
PR #1441 magically fixed it, but we should still test it.

Fixes #1451.

Also sneakily fixes the wrong URL in the release notes for the previous
commit.

docs/source/releases/v0.8.rst

@@ -242,7 +242,7 @@ Minor changes
   ``JQuery UI`` as it's in use in the frontend.
 
 .. _django-tables2: http://django-tables2.readthedocs.org/en/latest/
-.. _bootstrap-datetimepicker: http://jdewit.github.io/bootstrap-timepicker/
+.. _bootstrap-datetimepicker: http://www.malot.fr/bootstrap-datetimepicker/
 
 .. _incompatible_changes_in_0.8:
 

tests/functional/dashboard/catalogue_tests.py

@@ -2,7 +2,7 @@ from oscar.core.loading import get_model
 from django.core.urlresolvers import reverse
 
 from oscar.test.testcases import WebTestCase, add_permissions
-from oscar.test.factories import create_product, create_stockrecord
+from oscar.test.factories import create_product, create_stockrecord, ProductAttributeFactory
 
 from django_dynamic_fixture import G
 
@@ -90,6 +90,22 @@ class TestAStaffUser(WebTestCase):
             if product.has_stockrecords:
                 self.fail('Product has stock records but should not')
 
+    def test_can_create_product_with_required_attributes(self):
+        category = G(Category)
+        attribute = ProductAttributeFactory(required=True)
+        product_class = attribute.product_class
+        page = self.get(reverse('dashboard:catalogue-product-create',
+                                args=(product_class.slug,)))
+        form = page.form
+        form['upc'] = '123456'
+        form['title'] = 'new product'
+        form['attr_weight'] = '5'
+        form['productcategory_set-0-category'] = category.id
+
+        page = form.submit()
+
+        self.assertEqual(Product.objects.count(), 1)
+
     def test_can_delete_an_individual_product(self):
         product = create_product()
         stockrecord = create_stockrecord(product, partner_users=[self.user, ])
@@ -174,12 +190,17 @@ class TestANonStaffUser(TestAStaffUser):
         assert product1 in products_on_page
         assert product2 not in products_on_page
 
+    # Tests below can't work because they don't create a stockrecord
+
     def test_can_create_a_product_without_stockrecord(self):
         pass
 
     def test_can_update_a_product_without_stockrecord(self):
         pass
 
+    def test_can_create_product_with_required_attributes(self):
+        pass
+
     def test_can_submit_an_invalid_product_update_and_returns_to_update_page(self):
         pass