waku-org
/
js-waku
mirror of https://github.com/waku-org/js-waku.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

remove try catch no longer needed after 1790 got fixed (#2026)

This commit is contained in:
fbarbu15 2024-05-30 18:59:16 +03:00 committed by GitHub
parent 2311a595b2
commit c5302fd0c8
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 10 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
packages/tests/tests/filter/subscribe.node.spec.ts
View File

@ -309,25 +309,17 @@ const runTests = (strictCheckNodes: boolean): void => {
}); });
} }
// Open issue here: https://github.com/waku-org/js-waku/issues/1790 // Verify that each message was received on the corresponding topic.
// That's why we use the try catch block expect(
try { await serviceNodes.messageCollector.waitForMessages(topicCount)
// Verify that each message was received on the corresponding topic. ).to.eq(true);
expect( td.contentTopics.forEach((topic, index) => {
await serviceNodes.messageCollector.waitForMessages(topicCount) serviceNodes.messageCollector.verifyReceivedMessage(index, {
).to.eq(true); expectedContentTopic: topic,
td.contentTopics.forEach((topic, index) => { expectedMessageText: `Message for Topic ${index + 1}`,
serviceNodes.messageCollector.verifyReceivedMessage(index, { expectedPubsubTopic: TestPubsubTopic
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 () { it("Error when try to subscribe to more than 101 topics (new limit)", async function () {