-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(static sharding): add tests for multiple pubsub topics #1624
Conversation
size-limit report 📦
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
@@ -14,7 +14,7 @@ export interface Args { | |||
peerExchange?: boolean; | |||
discv5Discovery?: boolean; | |||
storeMessageDbUrl?: string; | |||
topic?: string; | |||
topic?: Array<string>; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
topic?: Array<string>; | |
topics?: Array<string>; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't start the nwaku container if I change from topic
to topics
I see Unrecognized option 'topics' in the container log
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes because topic
is the flag on nwaku side
i was suggesting an API within js-waku to pass topics
which would be translated to --topic:
flags
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
but imo it's fine to leave it as it is since this is an internal code and keeps things simple
we can address in the future if required
Problem
There are no tests with mulitple pubSub topics
Solution
Added new tests for filter, lightPush, filter and store protocol related to multiple pubSub topic feature
Changed existing utils to work with multiple pubSub topics and updated existing tests accordingly