allowing test to fail

waku-org · May 22, 2024 · cd50eb2 · cd50eb2
1 parent 1fe713c
commit cd50eb2
Showing 1 changed file with 8 additions and 16 deletions.
diff --git a/packages/tests/tests/filter/single_node/subscribe.node.spec.ts b/packages/tests/tests/filter/single_node/subscribe.node.spec.ts
@@ -262,23 +262,15 @@ describe("Waku Filter V2: Subscribe: Single Service Node", function () {
       });
     }
 
-    // Open issue here: https://github.com/waku-org/js-waku/issues/1790
-    // That's why we use the try catch block
-    try {
-      // Verify that each message was received on the corresponding topic.
-      expect(await messageCollector.waitForMessages(topicCount)).to.eq(true);
-      td.contentTopics.forEach((topic, index) => {
-        messageCollector.verifyReceivedMessage(index, {
-          expectedContentTopic: topic,
-          expectedMessageText: `Message for Topic ${index + 1}`,
-          expectedPubsubTopic: TestPubsubTopic
-        });
+    // Verify that each message was received on the corresponding topic.
+    expect(await messageCollector.waitForMessages(topicCount)).to.eq(true);
+    td.contentTopics.forEach((topic, index) => {
+      messageCollector.verifyReceivedMessage(index, {
+        expectedContentTopic: topic,
+        expectedMessageText: `Message for Topic ${index + 1}`,
+        expectedPubsubTopic: TestPubsubTopic
       });
-    } catch (error) {
-      console.warn(
-        "This test still fails because of https://github.com/waku-org/js-waku/issues/1790"
-      );
-    }
+    });
   });
 
   it("Error when try to subscribe to more than 101 topics (new limit)", async function () {