Rework customer functional tests

* Tests now split into more coherent modules
* Added IDs to forms to help WebTest
* Some refactoring along the way

oscar/apps/customer/app.py

@@ -1,11 +1,11 @@
 from django.conf.urls.defaults import patterns, url
 from django.contrib.auth.decorators import login_required
 
-from oscar.apps.customer.views import AccountSummaryView, OrderHistoryView, \
-    OrderDetailView, OrderLineView, AddressListView, AddressCreateView, \
-    AddressUpdateView, AddressDeleteView, EmailHistoryView, EmailDetailView, \
-    AccountAuthView, AnonymousOrderDetailView, ChangePasswordView, ProfileUpdateView, \
-    AccountRegistrationView, LogoutView
+from oscar.apps.customer.views import (AccountSummaryView, OrderHistoryView,
+    OrderDetailView, OrderLineView, AddressListView, AddressCreateView,
+    AddressUpdateView, AddressDeleteView, EmailHistoryView, EmailDetailView,
+    AccountAuthView, AnonymousOrderDetailView, ChangePasswordView,
+    ProfileUpdateView, AccountRegistrationView, LogoutView)
 from oscar.core.application import Application
 
 
@@ -30,7 +30,8 @@ class CustomerApplication(Application):
 
     def get_urls(self):
         urlpatterns = patterns('',
-            url(r'^$', login_required(self.summary_view.as_view()), name='summary'),
+            url(r'^$', login_required(self.summary_view.as_view()),
+                name='summary'),
             url(r'^login/$', self.login_view.as_view(), name='login'),
             url(r'^logout/$', self.logout_view.as_view(), name='logout'),
             url(r'^register/$', self.register_view.as_view(), name='register'),
@@ -38,24 +39,37 @@ class CustomerApplication(Application):
                 name='change-password'),
 
             # Profile
-            url(r'^profile/$', login_required(self.profile_update_view.as_view()), name='profile-update'),
-            url(r'^orders/$', login_required(self.order_history_view.as_view()), name='order-list'),
+            url(r'^profile/$',
+                login_required(self.profile_update_view.as_view()),
+                name='profile-update'),
+            url(r'^orders/$',
+                login_required(self.order_history_view.as_view()),
+                name='order-list'),
             url(r'^order-status/(?P<order_number>[\w-]*)/(?P<hash>\w+)/$',
                 self.anon_order_detail_view.as_view(), name='anon-order'),
-            url(r'^orders/(?P<order_number>[\w-]*)/$', login_required(self.order_detail_view.as_view()),
+            url(r'^orders/(?P<order_number>[\w-]*)/$',
+                login_required(self.order_detail_view.as_view()),
                 name='order'),
             url(r'^orders/(?P<order_number>[\w-]*)/(?P<line_id>\d+)$',
-                login_required(self.order_line_view.as_view()), name='order-line'),
-            url(r'^addresses/$', login_required(self.address_list_view.as_view()),
+                login_required(self.order_line_view.as_view()),
+                name='order-line'),
+            url(r'^addresses/$',
+                login_required(self.address_list_view.as_view()),
                 name='address-list'),
-            url(r'^addresses/add/$', login_required(self.address_create_view.as_view()),
+            url(r'^addresses/add/$',
+                login_required(self.address_create_view.as_view()),
                 name='address-create'),
-            url(r'^addresses/(?P<pk>\d+)/$', login_required(self.address_update_view.as_view()),
+            url(r'^addresses/(?P<pk>\d+)/$',
+                login_required(self.address_update_view.as_view()),
                 name='address-detail'),
-            url(r'^addresses/(?P<pk>\d+)/delete/$', login_required(self.address_delete_view.as_view()),
+            url(r'^addresses/(?P<pk>\d+)/delete/$',
+                login_required(self.address_delete_view.as_view()),
                 name='address-delete'),
-            url(r'^emails/$', login_required(self.email_list_view.as_view()), name='email-list'),
-            url(r'^emails/(?P<email_id>\d+)/$', login_required(self.email_detail_view.as_view()),
+            url(r'^emails/$',
+                login_required(self.email_list_view.as_view()),
+                name='email-list'),
+            url(r'^emails/(?P<email_id>\d+)/$',
+                login_required(self.email_detail_view.as_view()),
                 name='email-detail'),
             )
         return self.post_process_urls(urlpatterns)

oscar/apps/customer/history_helpers.py

@@ -12,9 +12,9 @@ MAX_PRODUCTS = settings.OSCAR_RECENTLY_VIEWED_PRODUCTS
 # Helpers
 
 def get_recently_viewed_product_ids(request):
-    u"""
-    Returns the list of ids of the last products browsed by the user
-    
+    """
+    Returns IDs of the last products browsed by the user
+
     Limited to the max number defined in settings.py
     under OSCAR_RECENTLY_VIEWED_PRODUCTS.
     """
@@ -38,7 +38,7 @@ def _update_recently_viewed_products(product, request, response):
     product_ids.append(product.id)
     if (len(product_ids) > MAX_PRODUCTS):
         product_ids = product_ids[len(product_ids)-MAX_PRODUCTS:]
-    response.set_cookie('oscar_recently_viewed_products', 
+    response.set_cookie('oscar_recently_viewed_products',
                         _get_json_string_from_list(product_ids),
                         httponly=True)
 
@@ -57,7 +57,7 @@ def _get_json_string_from_list(list):
 def receive_product_view(sender, product, user, request, response, **kwargs):
     """
     Receiver to handle viewing single product pages
-    
+
     Requires the request and response objects due to dependence on cookies
     """
     return _update_recently_viewed_products(product, request, response)

oscar/apps/customer/models.py

@@ -6,4 +6,7 @@ class Email(AbstractEmail):
 
 
 class CommunicationEventType(AbstractCommunicationEventType):
-    pass
+    pass
+
+
+from oscar.apps.customer.history_helpers import *

oscar/templates/oscar/customer/change-password.html

@@ -17,25 +17,24 @@
 </ul>
 {% endblock %}
 
-
 {% block title %}
 {% trans 'Change password' %} | {{ block.super }}
 {% endblock %}
 
 {% block header %}
 <div class="page-header">
-    <h2>{% trans 'Change password' %}</h2>
+    <h1>{% trans 'Change password' %}</h1>
 </div>
 {% endblock header %}
 
 
 {% block content %}
 
-<form action="." method="post">
+<form id="change_password_form" action="." method="post">
     {% csrf_token %}
     {% include 'partials/form_fields.html' with form=form %}
     <div class="form-actions">
-        <button type="submit" class="btn btn-primary">{% trans 'Update password' %}</button>
+        <button type="submit" class="btn btn-large btn-primary">{% trans 'Update password' %}</button>
         {% trans 'or' %} <a href="{% url customer:summary %}">{% trans 'cancel' %}</a>.
     </div>
 </form>

oscar/templates/oscar/customer/order-history.html

@@ -51,7 +51,7 @@
         <td>{{ order.date_placed }}</td>
         <td>
             <a class="btn btn-info" href="{% url customer:order order.number %}">{% trans 'View' %}</a>
-            <form action="{% url customer:order order.number %}" method="post">
+			<form id="order_form_{{ order.id }}" action="{% url customer:order order.number %}" method="post">
                 {% csrf_token %}
                 <input type="hidden" name="order_id" value="{{ order.id }}" />
                 <input type="hidden" name="action" value="reorder" />

oscar/templates/oscar/customer/order.html

@@ -30,7 +30,6 @@
 </div>
 {% endblock header %}
 
-
 {% block content %}
 
 <div class="tabbable dashboard">
@@ -100,7 +99,7 @@
                     <td>{{ line.line_price_before_discounts_excl_tax|currency }}</td>
                     <td>{{ line.line_price_before_discounts_incl_tax|currency }}</td>
                     <td>
-                        <form action="{% url customer:order-line order.number line.id %}" method="POST">
+						<form id="line_form_{{ line.id }}" action="{% url customer:order-line order.number line.id %}" method="POST">
                             {% csrf_token %}
                             <input type="hidden" name="action" value="reorder" />
                             <button class="btn btn-success" type="submit">{% trans 'Re-order' %}</button>

oscar/templates/oscar/customer/profile-form.html

@@ -15,7 +15,6 @@
 </ul>
 {% endblock %}
 
-
 {% block title %}
 {% trans 'Edit profile' %} | {{ block.super }}
 {% endblock %}
@@ -26,16 +25,13 @@
 </div>
 {% endblock header %}
 
-
 {% block content %}
-
-<form action="." method="post">
-    {% csrf_token %}
-    {% include 'partials/form_fields.html' with form=form %}
-    <div class="form-actions">
-        <button type="submit" class="btn btn-primary">{% trans 'Save' %}</button>
-        {% trans 'or' %} <a href="{% url customer:summary %}">{% trans 'cancel' %}</a>.
-    </div>
+<form id="profile_form" action="." method="post">
+	{% csrf_token %}
+	{% include 'partials/form_fields.html' with form=form %}
+	<div class="form-actions">
+		<button type="submit" class="btn btn-primary btn-large">{% trans 'Save' %}</button>
+		{% trans 'or' %} <a href="{% url customer:summary %}">{% trans 'cancel' %}</a>.
+	</div>
 </form>
-
 {% endblock content %}

oscar/test/helpers.py

@@ -78,8 +78,8 @@ def create_order(number=None, basket=None, user=None, shipping_address=None, shi
             billing_address=billing_address,
             total_incl_tax=total_incl_tax,
             total_excl_tax=total_excl_tax,
-            **kwargs
-            )
+            **kwargs)
+    basket.set_as_submitted()
     return order
 
 

tests/functional/customer/auth_tests.py

@@ -1,19 +1,20 @@
 import re
 
 from django.contrib.auth import models
-from django.core.urlresolvers import reverse
+from django.contrib.auth.models import User
 from django.core import mail
-
+from django.core.urlresolvers import reverse
 from django_webtest import WebTest
 
+from oscar.test import WebTestCase
+
 
 class TestAUserWhoseForgottenHerPassword(WebTest):
 
     def test_can_reset_her_password(self):
         username, email, password = 'lucy', 'lucy@example.com', 'password'
         models.User.objects.create_user(
-            username, email, password
-        )
+            username, email, password)
 
         # Fill in password reset form
         page = self.app.get(reverse('password-reset'))
@@ -47,3 +48,26 @@ class TestAUserWhoseForgottenHerPassword(WebTest):
         form['login-password'] = 'monkey'
         response = form.submit('login_submit')
         self.assertEqual(302, response.status_code)
+
+
+class TestAnAnonymousUser(WebTestCase):
+
+    def test_can_login(self):
+        email, password = 'd@d.com', 'mypassword'
+        User.objects.create_user('_', email, password)
+
+        url = reverse('customer:login')
+        form = self.app.get(url).forms['login_form']
+        form['login-username'] = email
+        form['login-password'] = password
+        response = form.submit('login_submit')
+        self.assertRedirectsTo(response, 'customer:summary')
+
+    def test_can_register(self):
+        url = reverse('customer:register')
+        form = self.app.get(url).forms['register_form']
+        form['registration-email'] = 'terry@boom.com'
+        form['registration-password1'] = 'hedgehog'
+        form['registration-password2'] = 'hedgehog'
+        response = form.submit()
+        self.assertRedirectsTo(response, 'customer:summary')

tests/functional/customer/history_tests.py

@@ -0,0 +1,53 @@
+import re
+
+from django.contrib.auth import models
+from django.test.client import Client
+from django.contrib.auth.models import User
+from django.core import mail
+from django.core.urlresolvers import reverse
+from django.test import TestCase
+from django_webtest import WebTest
+
+from oscar.test.helpers import create_product
+from oscar.test import WebTestCase
+from oscar.apps.customer.history_helpers import get_recently_viewed_product_ids
+from django.http import HttpRequest
+
+
+class HistoryHelpersTest(TestCase):
+
+    def setUp(self):
+        self.client = Client()
+        self.product = create_product()
+
+    def test_viewing_product_creates_cookie(self):
+        response = self.client.get(self.product.get_absolute_url())
+        self.assertTrue('oscar_recently_viewed_products' in response.cookies)
+
+    def test_id_gets_added_to_cookie(self):
+        response = self.client.get(self.product.get_absolute_url())
+        request = HttpRequest()
+        request.COOKIES['oscar_recently_viewed_products'] = response.cookies['oscar_recently_viewed_products'].value
+        self.assertTrue(self.product.id in get_recently_viewed_product_ids(request))
+
+
+class TestAUserWhoLogsOut(TestCase):
+    username = 'customer'
+    password = 'cheeseshop'
+    email = 'customer@example.com'
+
+    def setUp(self):
+        self.client = Client()
+        self.product = create_product()
+        User.objects.create_user(username=self.username,
+                                 email=self.email, password=self.password)
+        self.client.login(username=self.username, password=self.password)
+
+    def test_has_their_cookies_deleted_on_logout(self):
+        response = self.client.get(self.product.get_absolute_url())
+        self.assertTrue('oscar_recently_viewed_products' in response.cookies)
+
+        response = self.client.get(reverse('customer:logout'))
+        self.assertTrue(('oscar_recently_viewed_products' not in response.cookies)
+                        or not
+                        self.client.cookies['oscar_recently_viewed_products'].coded_value)

tests/functional/customer/order_status_tests.py

@@ -0,0 +1,31 @@
+import httplib
+
+from django.core.urlresolvers import reverse
+
+from oscar.test.helpers import create_order
+from oscar.test import WebTestCase
+
+
+class TestAnAnonymousUser(WebTestCase):
+
+    def test_gets_a_404_when_requesting_an_unknown_order(self):
+        path = reverse('customer:anon-order', kwargs={'order_number': 1000,
+                                                      'hash': '1231231232'})
+        response = self.app.get(path, status="*")
+        self.assertEqual(httplib.NOT_FOUND, response.status_code)
+
+    def test_can_see_order_status(self):
+        order = create_order()
+        path = reverse('customer:anon-order',
+                       kwargs={'order_number': order.number,
+                               'hash': order.verification_hash()})
+        response = self.app.get(path)
+        self.assertEqual(httplib.OK, response.status_code)
+
+    def test_gets_404_when_using_incorrect_hash(self):
+        order = create_order()
+        path = reverse('customer:anon-order',
+                       kwargs={'order_number': order.number,
+                               'hash': 'bad'})
+        response = self.app.get(path, status="*")
+        self.assertEqual(httplib.NOT_FOUND, response.status_code)

tests/functional/customer/profile_tests.py

@@ -0,0 +1,142 @@
+import httplib
+from mock import patch
+from decimal import Decimal as D
+
+from django.contrib.auth.models import User
+from django.core.urlresolvers import reverse
+from django.test import TestCase
+from django.test.client import Client
+
+from oscar.test.helpers import create_product, create_order
+from oscar.test import ClientTestCase, WebTestCase
+from oscar.apps.basket.models import Basket
+
+
+class TestASignedInUser(WebTestCase):
+    email = 'customer@example.com'
+    password = 'cheeseshop'
+
+    def setUp(self):
+        self.user = User.objects.create_user('_', self.email, self.password)
+        self.order = create_order(user=self.user)
+
+    def tearDown(self):
+        Basket.objects.all().delete()
+
+    def test_can_view_their_profile(self):
+        response = self.app.get(reverse('customer:summary'),
+                                user=self.user)
+        self.assertEqual(200, response.status_code)
+        self.assertTrue(self.email in response.content)
+
+    def test_can_update_their_email_address_and_name(self):
+        profile_form_page = self.app.get(reverse('customer:profile-update'),
+                                user=self.user)
+        self.assertEqual(200, profile_form_page.status_code)
+        form = profile_form_page.forms['profile_form']
+        form['email'] = 'new@example.com'
+        form['first_name'] = 'Barry'
+        form['last_name'] = 'Chuckle'
+        response = form.submit()
+        self.assertRedirects(response, reverse('customer:summary'))
+
+        user = User.objects.get(id=self.user.id)
+        self.assertEqual('new@example.com', user.email)
+        self.assertEqual('Barry', user.first_name)
+        self.assertEqual('Chuckle', user.last_name)
+
+    def test_can_change_their_password(self):
+        password_form_page = self.app.get(reverse('customer:change-password'),
+                                          user=self.user)
+        self.assertEqual(200, password_form_page.status_code)
+        form = password_form_page.forms['change_password_form']
+        form['old_password'] = self.password
+        form['new_password1'] = 'bubblesgopop'
+        form['new_password2'] = 'bubblesgopop'
+        response = form.submit()
+        self.assertRedirects(response, reverse('customer:summary'))
+
+    def test_can_reorder_a_previous_order(self):
+        order_history_page = self.app.get(reverse('customer:order-list'),
+                                          user=self.user)
+        form = order_history_page.forms['order_form_%d' % self.order.id]
+        form.submit()
+
+        basket = Basket.open.get(owner=self.user)
+        self.assertEqual(1, basket.all_lines().count())
+
+    def test_can_reorder_a_previous_order_line(self):
+        order_history_page = self.app.get(
+            reverse('customer:order',
+                    kwargs={'order_number': self.order.number}),
+            user=self.user)
+        line = self.order.lines.all()[0]
+        form = order_history_page.forms['line_form_%d' % line.id]
+        form.submit()
+
+        basket = Basket.open.get(owner=self.user)
+        self.assertEqual(1, basket.all_lines().count())
+
+    def test_cannot_reorder_an_out_of_stock_product(self):
+        product = self.order.lines.all()[0].product
+        product.stockrecord.num_in_stock = 0
+        product.stockrecord.save()
+
+        order_history_page = self.app.get(reverse('customer:order-list'),
+                                          user=self.user)
+        form = order_history_page.forms['order_form_%d' % self.order.id]
+        form.submit()
+
+        basket = Basket.open.get(owner=self.user)
+        self.assertEqual(0, basket.all_lines().count())
+
+
+class TestReorderingOrderLines(ClientTestCase):
+    # TODO - rework this as a webtest
+
+    @patch('django.conf.settings.OSCAR_MAX_BASKET_QUANTITY_THRESHOLD', 1)
+    def test_cannot_reorder_when_basket_maximum_exceeded(self):
+        order = create_order(user=self.user)
+        line = order.lines.all()[0]
+
+        Basket.objects.all().delete()
+        #add a product
+        product = create_product(price=D('12.00'))
+        self.client.post(reverse('basket:add'), {'product_id': product.id,
+                                                 'quantity': 1})
+
+
+        basket = Basket.objects.all()[0]
+        self.assertEquals(len(basket.all_lines()), 1)
+
+        #try to reorder a product
+        self.client.post(reverse('customer:order',
+                                            args=(order.number,)),
+                                    {'order_id': order.pk,
+                                     'action': 'reorder'})
+
+        self.assertEqual(len(basket.all_lines()), 1)
+        self.assertNotEqual(line.product.pk, product.pk)
+
+    @patch('django.conf.settings.OSCAR_MAX_BASKET_QUANTITY_THRESHOLD', 1)
+    def test_cannot_reorder_line_when_basket_maximum_exceeded(self):
+        order = create_order(user=self.user)
+        line = order.lines.all()[0]
+
+        Basket.objects.all().delete()
+        #add a product
+        product = create_product(price=D('12.00'))
+        self.client.post(reverse('basket:add'), {'product_id': product.id,
+                                                 'quantity': 1})
+
+        basket = Basket.objects.all()[0]
+        self.assertEquals(len(basket.all_lines()), 1)
+
+        self.client.post(reverse('customer:order-line',
+                                            args=(order.number, line.pk)),
+                                    {'action': 'reorder'})
+
+        self.assertEquals(len(basket.all_lines()), 1)
+        self.assertNotEqual(line.product.pk, product.pk)
+
+

tests/functional/customer_tests.py

@@ -1,260 +0,0 @@
-import httplib
-from mock import patch
-from decimal import Decimal as D
-
-from django.contrib.auth.models import User
-from django.core.urlresolvers import reverse
-from django.http import HttpRequest
-from django.test import TestCase
-from django.test.client import Client
-
-from oscar.apps.customer.history_helpers import get_recently_viewed_product_ids
-from oscar.test.helpers import create_product, create_order
-from oscar.test import ClientTestCase, WebTestCase
-from oscar.apps.basket.models import Basket
-
-
-class HistoryHelpersTest(TestCase):
-
-    def setUp(self):
-        self.client = Client()
-        self.product = create_product()
-
-    def test_viewing_product_creates_cookie(self):
-        response = self.client.get(self.product.get_absolute_url())
-        self.assertTrue('oscar_recently_viewed_products' in response.cookies)
-
-    def test_id_gets_added_to_cookie(self):
-        response = self.client.get(self.product.get_absolute_url())
-        request = HttpRequest()
-        request.COOKIES['oscar_recently_viewed_products'] = response.cookies['oscar_recently_viewed_products'].value
-        self.assertTrue(self.product.id in get_recently_viewed_product_ids(request))
-
-
-class AnonOrderDetail(TestCase):
-
-    def setUp(self):
-        self.client = Client()
-
-    def test_404_received_for_unknown_order(self):
-        response = self.client.get(reverse('customer:anon-order', kwargs={'order_number': 1000,
-            'hash': '1231231232'}))
-        self.assertEqual(httplib.NOT_FOUND, response.status_code)
-
-    def test_200_received_for_order_with_correct_hash(self):
-        order = create_order()
-        response = self.client.get(reverse('customer:anon-order', kwargs={'order_number': order.number,
-            'hash': order.verification_hash()}))
-        self.assertEqual(httplib.OK, response.status_code)
-
-    def test_404_received_for_order_with_incorrect_hash(self):
-        order = create_order()
-        response = self.client.get(reverse('customer:anon-order', kwargs={'order_number': order.number,
-            'hash': 'bad'}))
-        self.assertEqual(httplib.NOT_FOUND, response.status_code)
-
-
-class EditProfileTests(TestCase):
-    username = 'customer'
-    password = 'cheeseshop'
-    email = 'customer@example.com'
-
-    def setUp(self):
-        User.objects.create_user(username=self.username,
-                                 email=self.email, password=self.password)
-        is_successful = self.client.login(username=self.username,
-                                          password=self.password)
-        if not is_successful:
-            self.fail("Unable to login as %s" % self.username)
-
-    def tearDown(self):
-        User.objects.all().delete()
-
-    def test_update_profile_page_for_smoke(self):
-        url = reverse('customer:profile-update')
-        response = self.client.get(url)
-        self.assertEqual(200, response.status_code)
-        self.assertTrue('form' in response.context)
-
-
-class AuthTestCase(TestCase):
-    username = 'customer'
-    password = 'cheeseshop'
-    email = 'customer@example.com'
-
-    def setUp(self):
-        self.client = Client()
-        self.product = create_product()
-        User.objects.create_user(username=self.username,
-                                 email=self.email, password=self.password)
-        self.client.login(username=self.username, password=self.password)
-
-    def test_cookies_deleted_on_logout(self):
-        response = self.client.get(self.product.get_absolute_url())
-        self.assertTrue('oscar_recently_viewed_products' in response.cookies)
-
-        response = self.client.get(reverse('customer:logout'))
-        self.assertTrue(('oscar_recently_viewed_products' not in response.cookies)
-                        or not
-                        self.client.cookies['oscar_recently_viewed_products'].coded_value)
-
-
-class AuthStaffRedirectTests(TestCase):
-    username = 'staff'
-    password = 'cheeseshop'
-    email = 'staff@example.com'
-
-    def test_staff_member_login_for_dashboard(self):
-        """
-        Test if a staff member that is not yet logged in and trying to access the
-        dashboard is redirected to the Oscar login page (instead of the ``admin``
-        login page). Also test that the redirect after successful login will
-        be the originally requested page.
-        """
-        self.client = Client()
-        user = User.objects.create_user(username=self.username,
-                                    email=self.email, password=self.password)
-        user.is_staff = True
-        user.save()
-
-        response = self.client.get(reverse('dashboard:index'), follow=True)
-        self.assertContains(response, "login-username", status_code=200)
-        self.assertEquals(response.context['next'], reverse('dashboard:index'))
-
-
-class ReorderTests(ClientTestCase):
-
-    def test_can_reorder(self):
-        order = create_order(user=self.user)
-        Basket.objects.all().delete()
-
-        self.client.post(reverse('customer:order',
-                                            args=(order.number,)),
-                                    {'order_id': order.pk,
-                                     'action': 'reorder'})
-
-        basket = Basket.objects.all()[0]
-        self.assertEquals(len(basket.all_lines()), 1)
-
-    def test_can_reorder_line(self):
-        order = create_order(user=self.user)
-        line = order.lines.all()[0]
-        Basket.objects.all().delete()
-
-        self.client.post(reverse('customer:order-line',
-                                            args=(order.number, line.pk)),
-                                    {'action': 'reorder'})
-
-        basket = Basket.objects.all()[0]
-        self.assertEquals(len(basket.all_lines()), 1)
-
-    def test_cannot_reorder_out_of_stock_product(self):
-        order = create_order(user=self.user)
-
-        product = order.lines.all()[0].product
-        product.stockrecord.num_in_stock = 0
-        product.stockrecord.save()
-
-        Basket.objects.all().delete()
-
-        self.client.post(reverse('customer:order',
-                                            args=(order.number,)),
-                                    {'order_id': order.pk,
-                                     'action': 'reorder'})
-
-        basket = Basket.objects.all()[0]
-        self.assertEquals(len(basket.all_lines()), 0)
-
-    def test_cannot_reorder_out_of_stock_line(self):
-        order = create_order(user=self.user)
-        line = order.lines.all()[0]
-
-        product = line.product
-        product.stockrecord.num_in_stock = 0
-        product.stockrecord.save()
-
-        Basket.objects.all().delete()
-
-        self.client.post(reverse('customer:order-line',
-                                            args=(order.number, line.pk)),
-                                    {'action': 'reorder'})
-
-        basket = Basket.objects.all()[0]
-        self.assertEquals(len(basket.all_lines()), 0)
-
-    @patch('django.conf.settings.OSCAR_MAX_BASKET_QUANTITY_THRESHOLD', 1)
-    def test_cannot_reorder_when_basket_maximum_exceeded(self):
-        order = create_order(user=self.user)
-        line = order.lines.all()[0]
-
-        Basket.objects.all().delete()
-        #add a product
-        product = create_product(price=D('12.00'))
-        self.client.post(reverse('basket:add'), {'product_id': product.id,
-                                                 'quantity': 1})
-
-
-        basket = Basket.objects.all()[0]
-        self.assertEquals(len(basket.all_lines()), 1)
-
-        #try to reorder a product
-        self.client.post(reverse('customer:order',
-                                            args=(order.number,)),
-                                    {'order_id': order.pk,
-                                     'action': 'reorder'})
-
-        self.assertEqual(len(basket.all_lines()), 1)
-        self.assertNotEqual(line.product.pk, product.pk)
-
-    @patch('django.conf.settings.OSCAR_MAX_BASKET_QUANTITY_THRESHOLD', 1)
-    def test_cannot_reorder_line_when_basket_maximum_exceeded(self):
-        order = create_order(user=self.user)
-        line = order.lines.all()[0]
-
-        Basket.objects.all().delete()
-        #add a product
-        product = create_product(price=D('12.00'))
-        self.client.post(reverse('basket:add'), {'product_id': product.id,
-                                                 'quantity': 1})
-
-        basket = Basket.objects.all()[0]
-        self.assertEquals(len(basket.all_lines()), 1)
-
-        self.client.post(reverse('customer:order-line',
-                                            args=(order.number, line.pk)),
-                                    {'action': 'reorder'})
-
-        self.assertEquals(len(basket.all_lines()), 1)
-        self.assertNotEqual(line.product.pk, product.pk)
-
-
-class TestAnAnonymousUser(WebTestCase):
-
-    def test_can_login(self):
-        email, password = 'd@d.com', 'mypassword'
-        User.objects.create_user('_', email, password)
-
-        url = reverse('customer:login')
-        form = self.app.get(url).forms['login_form']
-        form['login-username'] = email
-        form['login-password'] = password
-        response = form.submit('login_submit')
-        self.assertRedirectsTo(response, 'customer:summary')
-
-    def test_can_register(self):
-        url = reverse('customer:register')
-        form = self.app.get(url).forms['register_form']
-        form['registration-email'] = 'terry@boom.com'
-        form['registration-password1'] = 'hedgehog'
-        form['registration-password2'] = 'hedgehog'
-        response = form.submit()
-        self.assertRedirectsTo(response, 'customer:summary')
-
-
-class TestASignedInUser(WebTestCase):
-
-    def test_can_view_their_profile(self):
-        email, password = 'd@d.com', 'mypassword'
-        user = User.objects.create_user('_', email, password)
-        url = reverse('customer:summary')
-        self.app.get(url, user=user)