fix some variables in the task bundling related tests

maproulette · Mar 11, 2024 · 4b90bc7 · 4b90bc7
1 parent f394514
commit 4b90bc7
Showing 1 changed file with 11 additions and 11 deletions.
diff --git a/test/org/maproulette/framework/service/TaskBundleServiceSpec.scala b/test/org/maproulette/framework/service/TaskBundleServiceSpec.scala
@@ -35,7 +35,7 @@ class TaskBundleServiceSpec(implicit val application: Application) extends Frame
         )
 
       val response =
-        this.service.createTaskBundle(User.superUser, "my bundle", Some(0),  List(task1.id, task2.id))
+        this.service.createTaskBundle(User.superUser, "my bundle", Some(task1.id), List(task1.id, task2.id))
       response.taskIds.length mustEqual 2
 
       // tasks.bundle_id is NOT set until setTaskStatus is called!!!
@@ -51,14 +51,14 @@ class TaskBundleServiceSpec(implicit val application: Application) extends Frame
 
       // Cannot create a bundle with tasks already assigned
       intercept[InvalidException] {
-        this.service.createTaskBundle(User.superUser, "my bundle again", Some(0), List(task1.id, task2.id))
+        this.service.createTaskBundle(User.superUser, "my bundle again", Some(task1.id), List(task1.id, task2.id))
       }
     }
 
     "not create a task Bundle with no tasks" taggedAs (TaskTag) in {
       // Cannot create a bundle with no tasks
       intercept[InvalidException] {
-        this.service.createTaskBundle(User.superUser, "my bundle again", Some(0),  List())
+        this.service.createTaskBundle(User.superUser, "my bundle again", Some(0), List())
       }
     }
 
@@ -93,7 +93,7 @@ class TaskBundleServiceSpec(implicit val application: Application) extends Frame
 
       // Cannot create a bundle with tasks from different challenges
       intercept[InvalidException] {
-        this.service.createTaskBundle(User.superUser, "bad bundle", Some(0), List(task1.id, task2.id))
+        this.service.createTaskBundle(User.superUser, "bad bundle", Some(task1.id), List(task1.id, task2.id))
       }
     }
 
@@ -110,7 +110,7 @@ class TaskBundleServiceSpec(implicit val application: Application) extends Frame
         )
 
       val bundle =
-        this.service.createTaskBundle(User.superUser, "my bundle for get", Some(0),  List(task1.id, task2.id))
+        this.service.createTaskBundle(User.superUser, "my bundle for get", Some(task1.id), List(task1.id, task2.id))
 
       val response = this.service.getTaskBundle(User.superUser, bundle.bundleId)
       response.bundleId mustEqual bundle.bundleId
@@ -130,7 +130,7 @@ class TaskBundleServiceSpec(implicit val application: Application) extends Frame
         )
 
       val bundle = this.service
-        .createTaskBundle(User.superUser, "my bundle for delete", Some(0),  List(task1.id, task2.id))
+        .createTaskBundle(User.superUser, "my bundle for delete", Some(task1.id), List(task1.id, task2.id))
 
       // tasks.bundle_id is NOT set until setTaskStatus is called
       taskDAL.setTaskStatus(
@@ -159,7 +159,7 @@ class TaskBundleServiceSpec(implicit val application: Application) extends Frame
         )
 
       val bundle = this.service
-        .createTaskBundle(User.superUser, "my bundle for delete", Some(0),  List(task1.id, task2.id))
+        .createTaskBundle(User.superUser, "my bundle for delete", Some(task1.id), List(task1.id, task2.id))
 
       // tasks.bundle_id is NOT set until setTaskStatus is called
       taskDAL.setTaskStatus(
@@ -193,7 +193,7 @@ class TaskBundleServiceSpec(implicit val application: Application) extends Frame
         )
 
       val bundle = this.service
-        .createTaskBundle(User.superUser, "my bundle for unbundle", Some(0),  List(task1.id, task2.id))
+        .createTaskBundle(User.superUser, "my bundle for unbundle", Some(task1.id), List(task1.id, task2.id))
 
       // tasks.bundle_id is NOT set until setTaskStatus is called
       taskDAL.setTaskStatus(
@@ -204,7 +204,7 @@ class TaskBundleServiceSpec(implicit val application: Application) extends Frame
         primaryTaskId = Some(task1.id)
       )
 
-      this.service.unbundleTasks(User.superUser, bundle.bundleId, List(task1.id), List(task2.id))()
+      this.service.unbundleTasks(User.superUser, bundle.bundleId, List(task2.id), List(task1.id))()
       val response = this.service.getTaskBundle(User.superUser, bundle.bundleId)
       response.taskIds.length mustEqual 1
       response.taskIds.head mustEqual task1.id
@@ -223,7 +223,7 @@ class TaskBundleServiceSpec(implicit val application: Application) extends Frame
         )
 
       val bundle = this.service
-        .createTaskBundle(User.superUser, "my bundle for unbundle", Some(0), List(task1.id, task2.id))
+        .createTaskBundle(User.superUser, "my bundle for unbundle", Some(task1.id), List(task1.id, task2.id))
 
       // tasks.bundle_id is NOT set until setTaskStatus is called
       taskDAL.setTaskStatus(
@@ -241,7 +241,7 @@ class TaskBundleServiceSpec(implicit val application: Application) extends Frame
 
       // Random user is not allowed to delete this bundle
       an[IllegalAccessException] should be thrownBy
-        this.service.unbundleTasks(randomUser, bundle.bundleId, List(task1.id), List(task2.id))()
+        this.service.unbundleTasks(randomUser, bundle.bundleId, List(task2.id), List(task1.id))()
     }
 
   }