Unit tests

Author: pierotofy

app/tests/test_api.py

@@ -222,11 +222,18 @@ def test_projects_and_tasks(self):
         # Task should have failed to be restarted
         self.assertTrue("has no processing node" in task.last_error)
 
-        # Cannot cancel, restart or delete a task for which we don't have permission
-        for action in ['cancel', 'remove', 'restart']:
+        # Cannot cancel, restart, delete, compact a task for which we don't have permission
+        for action in ['cancel', 'remove', 'restart', 'compact']:
             res = client.post('/api/projects/{}/tasks/{}/{}/'.format(other_project.id, other_task.id, action))
             self.assertEqual(res.status_code, status.HTTP_404_NOT_FOUND)
 
+        # Can compact
+        self.assertFalse(task.compacted)
+        res = client.post('/api/projects/{}/tasks/{}/compact/'.format(project.id, task.id))
+        self.assertEqual(res.status_code, status.HTTP_200_OK)
+        task.refresh_from_db()
+        self.assertTrue(task.compacted)
+
         # Can delete
         res = client.post('/api/projects/{}/tasks/{}/remove/'.format(project.id, task.id))
         self.assertTrue(res.data["success"])

app/tests/test_api_task.py

@@ -963,7 +963,6 @@ def connTimeout(*args, **kwargs):
             task.refresh_from_db()
             self.assertTrue(task.last_error is None)
 
-
             # Reassigning the task to another project should move its assets
             self.assertTrue(os.path.exists(full_task_directory_path(task.id, project.id)))
             self.assertTrue(len(task.scan_images()) == 2)
@@ -974,6 +973,23 @@ def connTimeout(*args, **kwargs):
             self.assertFalse(os.path.exists(full_task_directory_path(task.id, project.id)))
             self.assertTrue(os.path.exists(full_task_directory_path(task.id, other_project.id)))
 
+            # Move back
+            task.project = project
+            task.save()
+            task.refresh_from_db()
+
+            # Compacting the task should remove the images
+            # but not the assets
+            self.assertTrue(len(os.listdir(task.assets_path())) > 0)
+            self.assertEqual(len(task.scan_images()), 2)
+
+            res = client.post("/api/projects/{}/tasks/{}/compact/".format(project.id, task.id))
+            self.assertEqual(res.status_code, status.HTTP_200_OK)
+            task.refresh_from_db()
+
+            self.assertTrue(len(os.listdir(task.assets_path())) > 0)
+            self.assertEqual(len(task.scan_images()), 0)            
+
         # Restart node-odm as to not generate orthophotos
         testWatch.clear()
         with start_processing_node(["--test_skip_orthophotos"]):

app/tests/test_app.py

@@ -101,6 +101,15 @@ def test_views(self):
         self.assertEqual(res.content.decode("utf-8").count('href="/processingnode/'), 3)
         self.assertTemplateUsed(res, 'app/dashboard.html')
 
+        # There should be an onboarding message
+        self.assertTrue('To create a map, press the' in res.content.decode("utf-8"))
+
+        # But not if we disable it
+        settings.DASHBOARD_ONBOARDING = False
+        res = c.get('/dashboard/', follow=True)
+        self.assertFalse('To create a map, press the' in res.content.decode("utf-8"))
+        settings.DASHBOARD_ONBOARDING  = True        
+
         # The API should return 3 nodes
         res = c.get('/api/processingnodes/')
         self.assertEqual(len(res.data), 3)