run collab tests in enforced liveCollab = true flag env

Author: MichaelBuessemeyer

frontend/javascripts/test/sagas/proofreading/proofreading_mesh_interaction.spec.ts

@@ -32,13 +32,17 @@ import type { MinCutTargetEdge } from "admin/rest_api";
 import _ from "lodash";
 import { delay } from "typed-redux-saga";
 import { updateUserSettingAction } from "viewer/model/actions/settings_actions";
+import { WkDevFlags } from "viewer/api/wk_dev";
 
 describe("Proofreading (with mesh actions)", () => {
+  const initialLiveCollab = WkDevFlags.liveCollab;
   beforeEach<WebknossosTestContext>(async (context) => {
+    WkDevFlags.liveCollab = true;
     await setupWebknossosForTesting(context, "hybrid");
   });
 
   afterEach<WebknossosTestContext>(async (context) => {
+    WkDevFlags.liveCollab = initialLiveCollab;
     context.tearDownPullQueues();
     // Saving after each test and checking that the root saga didn't crash,
     expect(hasRootSagaCrashed()).toBe(false);

frontend/javascripts/test/sagas/proofreading/proofreading_multi_user.spec.ts

@@ -24,13 +24,17 @@ import {
   mockInitialBucketAndAgglomerateData,
 } from "./proofreading_test_utils";
 import { setOthersMayEditForAnnotationAction } from "viewer/model/actions/annotation_actions";
+import { WkDevFlags } from "viewer/api/wk_dev";
 
 describe("Proofreading (Multi User)", () => {
+  const initialLiveCollab = WkDevFlags.liveCollab;
   beforeEach<WebknossosTestContext>(async (context) => {
+    WkDevFlags.liveCollab = true;
     await setupWebknossosForTesting(context, "hybrid");
   });
 
   afterEach<WebknossosTestContext>(async (context) => {
+    WkDevFlags.liveCollab = initialLiveCollab;
     context.tearDownPullQueues();
     // Saving after each test and checking that the root saga didn't crash,
     expect(hasRootSagaCrashed()).toBe(false);

frontend/javascripts/test/sagas/proofreading/proofreading_poll_only.spec.ts

@@ -18,14 +18,18 @@ import {
 } from "./proofreading_test_utils";
 import { AnnotationTool } from "viewer/model/accessors/tool_accessor";
 import { setOthersMayEditForAnnotationAction } from "viewer/model/actions/annotation_actions";
+import { WkDevFlags } from "viewer/api/wk_dev";
 
 describe("Proofreading (Poll only)", () => {
+  const initialLiveCollab = WkDevFlags.liveCollab;
   beforeEach<WebknossosTestContext>(async (context) => {
+    WkDevFlags.liveCollab = true;
     await setupWebknossosForTesting(context, "hybrid");
     //Store.dispatch(setIsUpdatingAnnotationCurrentlyAllowedAction(false));
   });
 
   afterEach<WebknossosTestContext>(async (context) => {
+    WkDevFlags.liveCollab = initialLiveCollab;
     context.tearDownPullQueues();
     expect(hasRootSagaCrashed()).toBe(false);
   });
@@ -430,3 +434,5 @@ describe("Proofreading (Poll only)", () => {
     await task.toPromise();
   }, 8000);
 });
+
+// TODOM: Test where OthersMayEdit = true is not needed

frontend/javascripts/test/sagas/proofreading/proofreading_skeleton_interaction.spec.ts

@@ -27,6 +27,7 @@ import { deleteEdgeAction, mergeTreesAction } from "viewer/model/actions/skeleto
 import { minCutAgglomerateAction } from "viewer/model/actions/proofread_actions";
 import type { MinCutTargetEdge } from "admin/rest_api";
 import _ from "lodash";
+import { WkDevFlags } from "viewer/api/wk_dev";
 
 function* performMergeTreesProofreading(
   context: WebknossosTestContext,
@@ -235,13 +236,15 @@ const mockEdgesForAgglomerateMinCut = (
     },
   );
 
-// TODOM: wait for agglomerate skeleton being properly loaded and then continue to do a proofreading action with it.
 describe("Proofreading (With Agglomerate Skeleton interactions)", () => {
+  const initialLiveCollab = WkDevFlags.liveCollab;
   beforeEach<WebknossosTestContext>(async (context) => {
+    WkDevFlags.liveCollab = true;
     await setupWebknossosForTesting(context, "hybrid");
   });
 
   afterEach<WebknossosTestContext>(async (context) => {
+    WkDevFlags.liveCollab = initialLiveCollab;
     context.tearDownPullQueues();
     // Saving after each test and checking that the root saga didn't crash,
     expect(hasRootSagaCrashed()).toBe(false);
@@ -764,6 +767,5 @@ describe("Proofreading (With Agglomerate Skeleton interactions)", () => {
 
 // TODOM: Write test for backend manipulating same agglomerate skeleton - should this be synched? check with others
 
-// TODOM: write tests for partitionedMinCut -> new test file
 // TODOM: Write a test that pulling update fro the backend does not yield diff tracing changes that would be pushed to the backend in a live collab scenario!
 // TODOM: Write test without injected version and test via spy or so that no rebasing was tried and thus only the update actions were sent to the server!