From 7908d87902d0e89faa674eb9383c6ecb00e10f6f Mon Sep 17 00:00:00 2001
From: Deena Wang <thedeenawang@gmail.com>
Date: Thu, 31 Jan 2013 15:58:14 -0500
Subject: [PATCH] more tests for views.py

---
 .../contentstore/tests/test_views.py          | 32 ++++++++++++++++---
 1 file changed, 28 insertions(+), 4 deletions(-)

diff --git a/cms/djangoapps/contentstore/tests/test_views.py b/cms/djangoapps/contentstore/tests/test_views.py
index 9fa16ad4b2..d5a15a10e3 100644
--- a/cms/djangoapps/contentstore/tests/test_views.py
+++ b/cms/djangoapps/contentstore/tests/test_views.py
@@ -6,6 +6,7 @@ import unittest
 from nose.tools import set_trace
 from nose.plugins.skip import SkipTest
 from collections import defaultdict
+import re
 
 from django.http import Http404, HttpResponse, HttpRequest, HttpResponseRedirect, HttpResponseBadRequest
 from django.conf import settings
@@ -24,6 +25,7 @@ from xmodule.modulestore import Location
 from xmodule.x_module import ModuleSystem
 from xmodule.error_module import ErrorModule
 from contentstore.utils import get_course_for_item
+from xmodule.templates import update_templates
 
 class Stub():
     pass
@@ -50,12 +52,14 @@ TEST_DATA_XML_MODULESTORE = xml_store_config(TEST_DATA_DIR)
 
 class ViewsTestCase(TestCase):
     def setUp(self):
-        #modulestore().collection.drop()
+        self._MODULESTORES = {}
+        modulestore().collection.drop()
+        update_templates()
         self.location = ['i4x', 'edX', 'toy', 'chapter', 'Overview']
         self.location_2 = ['i4x', 'edX', 'full', 'course', '6.002_Spring_2012']
         self.location_3 = ['i4x', 'MITx', '999', 'course', 'Robot_Super_Course']
         # empty Modulestore
-        self._MODULESTORES = {}
+        
         self.course_id = 'edX/toy/2012_Fall'
         self.course_id_2 = 'edx/full/6.002_Spring_2012'
         # is a CourseDescriptor object?
@@ -210,6 +214,7 @@ class ViewsTestCase(TestCase):
         self.assertEquals(s.children[0].n, 1)
 
     def test_get_module_previews(self):
+        raise SkipTest
         # needs a working render_to_string
         raise SkipTest
         self.request = RequestFactory().get('foo')
@@ -218,6 +223,7 @@ class ViewsTestCase(TestCase):
         print views.get_module_previews(self.request, self.course)
 
     def test_delete_item(self):
+        raise SkipTest
         # If user doesn't have permission, redirect
         self.no_permit_user = MagicMock(is_staff = False, is_active = False)
         self.no_permit_user.is_authenticated.return_value = True
@@ -297,7 +303,7 @@ class ViewsTestCase(TestCase):
         self.assertIsInstance(views.save_item(self.request_3), HttpResponse)
         self.assertEquals(modulestore().get_item(self.item_2.location.dict()).definition['data'],
                           {u'foo': u'bar'})
-        # Test metadata, which is a dictionary?
+        # Test updating metadata
         self.request_4 = RequestFactory().post(self.item_2.location.url())
         self.request_4.POST = self.request.POST.copy()
         self.request_4.POST.update({'id':self.item_2.location.url(),
@@ -306,7 +312,25 @@ class ViewsTestCase(TestCase):
         self.assertIsInstance(views.save_item(self.request_4), HttpResponse)
         self.assertEquals(modulestore().get_item(self.item_2.location.dict()).metadata['foo'],
                           'bar')
-        
+
+    def test_clone_item(self):
+        # Test that user with no permissions gets redirected
+        self.no_permit_user = MagicMock(is_staff = False, is_active = False)
+        self.no_permit_user.is_authenticated.return_value = True
+        self.request = RequestFactory().post(self.item.location.url())
+        self.request.POST = self.request.POST.copy()
+        self.request.POST.update({'id':self.item.location.url(),
+                                  'parent_location':self.course.location.url(),
+                                  'template':self.location_3,
+                                  'display_name':'bar'})
+        self.request.user = self.no_permit_user
+        self.assertRaises(PermissionDenied, views.clone_item, self.request)
+        self.permit_user = MagicMock(is_staff = True, is_active = True)
+        self.permit_user.is_authenticated.return_value = True
+        self.request.user = self.permit_user
+        response = views.clone_item(self.request)
+        self.assertIsInstance(response, HttpResponse)
+        self.assertRegexpMatches(response.content, '{"id": "i4x://MITx/999/course/')
 
 def f(x):
     x.n += 1