Readable message for watch command when list is already watched.

#630

src/commands/WatchUnwatchCommand.ts

@@ -19,7 +19,7 @@ import {
   describeCommand,
   tuple,
 } from "@the-draupnir-project/interface-manager";
-import { Result } from "@gnuxie/typescript-result";
+import { Result, ResultError } from "@gnuxie/typescript-result";
 import { Draupnir } from "../Draupnir";
 import {
   DraupnirContextToCommandContextTranslator,
@@ -49,6 +49,14 @@ export const DraupnirWatchPolicyRoomCommand = describeCommand({
     if (isError(policyRoom)) {
       return policyRoom;
     }
+    if (
+      issuerManager.allWatchedLists.some(
+        (profile) =>
+          profile.room.toRoomIDOrAlias() === policyRoom.ok.toRoomIDOrAlias()
+      )
+    ) {
+      return ResultError.Result("We are already watching this list.");
+    }
     return await issuerManager.watchList(
       PropagationType.Direct,
       policyRoom.ok,

test/unit/commands/WatchUnwatchCommandTest.ts

@@ -13,6 +13,7 @@ import {
   PolicyListConfig,
   PropagationType,
   RoomResolver,
+  isError,
   isOk,
 } from "matrix-protection-suite";
 import { createMock } from "ts-auto-mock";
@@ -50,6 +51,20 @@ describe("Test the WatchUnwatchCommmands", function () {
     );
     expect(isOk(result)).toBe(true);
   });
+  it("Draupnir watch command should return an error if the room is already being watched", async function () {
+    const issuerManagerWithWatchedList = createMock<PolicyListConfig>({
+      allWatchedLists: [
+        { room: policyRoom, propagation: PropagationType.Direct, options: {} },
+      ],
+    });
+    const result = await CommandExecutorHelper.execute(
+      DraupnirWatchPolicyRoomCommand,
+      { issuerManager: issuerManagerWithWatchedList, roomResolver },
+      {},
+      policyRoom
+    );
+    expect(isError(result)).toBe(true);
+  });
   it("DraupnirUnwatchCommand", async function () {
     const result = await CommandExecutorHelper.execute(
       DraupnirUnwatchPolicyRoomCommand,