Add tests for "oscar_fork_statics" management command

src/oscar/management/commands/oscar_fork_statics.py

@@ -36,10 +36,10 @@ class Command(BaseCommand):
 
         source = os.path.realpath(
             os.path.join(os.path.dirname(__file__), '../../static'))
-        print("Copying Oscar's static files to %s" % (destination,))
+        self.stdout.write("Copying Oscar's static files to %s" % destination)
         shutil.copytree(source, destination)
 
         # Check if this new folder is in STATICFILES_DIRS
         if destination not in settings.STATICFILES_DIRS:
-            print(("You need to add %s to STATICFILES_DIRS in order for your "
-                   "local overrides to be picked up") % destination)
+            self.stdout.write("You need to add %s to STATICFILES_DIRS in order for your local overrides to be picked "
+                              "up" % destination)

tests/integration/management_commands/test_oscar_fork_statics.py

@@ -0,0 +1,89 @@
+import io
+import os
+import pathlib
+import tempfile
+
+from django.core.management import call_command
+from django.core.management.base import CommandError
+from django.test import TestCase
+
+import oscar
+from tests import _site
+
+
+class OscarForkStaticsTestCase(TestCase):
+
+    def setUp(self):
+        # Create dummy Oscar static-files directory (this exists in released
+        # Oscar packages)
+        self.oscar_static_dir_path = pathlib.Path(os.path.dirname(oscar.__file__), 'static')
+        self.oscar_static_dir_path.mkdir()
+        self.oscar_static_file_path = pathlib.Path(self.oscar_static_dir_path, 'name.css')
+        self.oscar_static_file_path.touch(exist_ok=False)
+
+        # Change to the project's base directory (where the static-files
+        # directory should be copied to)
+        self.project_base_dir_path = pathlib.Path(os.path.dirname(_site.__file__))
+        os.chdir(self.project_base_dir_path)
+
+    def tearDown(self):
+        # Delete dummy Oscar static-files directory
+        self.oscar_static_file_path.unlink()
+        self.oscar_static_dir_path.rmdir()
+
+    def test_command_with_already_existing_directory(self):
+        project_static_dir_path = pathlib.Path(self.project_base_dir_path, 'static')
+
+        project_static_dir_path.mkdir()
+        with self.assertRaises(CommandError) as e:
+            call_command('oscar_fork_statics')
+        self.assertEqual(e.exception.args[0], "The folder %s already exists - aborting!" % project_static_dir_path)
+
+        project_static_dir_path.rmdir()
+
+    def test_command_with_default_target_path(self):
+        project_static_dir_path = pathlib.Path(self.project_base_dir_path, 'static')
+        project_static_file_path = pathlib.Path(project_static_dir_path, 'name.css')
+
+        out = io.StringIO()
+        call_command('oscar_fork_statics', stdout=out)
+        self.assertTrue(project_static_file_path.exists())
+        messages = out.getvalue().split('\n')
+        self.assertEqual(messages[0], "Copying Oscar's static files to %s" % project_static_dir_path)
+        self.assertEqual(messages[1], "You need to add %s to STATICFILES_DIRS in order for your local overrides to be "
+                                      "picked up" % project_static_dir_path)
+
+        project_static_file_path.unlink()
+        project_static_dir_path.rmdir()
+
+    def test_command_with_relative_target_path(self):
+        project_static_dir_path = pathlib.Path(self.project_base_dir_path, 'relative/dir')
+        project_static_file_path = pathlib.Path(project_static_dir_path, 'name.css')
+
+        out = io.StringIO()
+        call_command('oscar_fork_statics', target_path='relative/dir', stdout=out)
+        self.assertTrue(project_static_file_path.exists())
+        messages = out.getvalue().split('\n')
+        self.assertEqual(messages[0], "Copying Oscar's static files to %s" % project_static_dir_path)
+        self.assertEqual(messages[1], "You need to add %s to STATICFILES_DIRS in order for your local overrides to be "
+                                      "picked up" % project_static_dir_path)
+
+        project_static_file_path.unlink()
+        project_static_dir_path.rmdir()
+        project_static_dir_path.parent.rmdir()
+
+    def test_command_with_absolute_target_path(self):
+        project_static_dir_path = pathlib.Path(tempfile.mkdtemp(), 'static')
+        project_static_file_path = pathlib.Path(project_static_dir_path, 'name.css')
+
+        out = io.StringIO()
+        call_command('oscar_fork_statics', target_path=str(project_static_dir_path), stdout=out)
+        self.assertTrue(project_static_file_path.exists())
+        messages = out.getvalue().split('\n')
+        self.assertEqual(messages[0], "Copying Oscar's static files to %s" % project_static_dir_path)
+        self.assertEqual(messages[1], "You need to add %s to STATICFILES_DIRS in order for your local overrides to be "
+                                      "picked up" % project_static_dir_path)
+
+        project_static_file_path.unlink()
+        project_static_dir_path.rmdir()
+        project_static_dir_path.parent.rmdir()