Skip to content
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

E2E_PART2 #83

Closed
wants to merge 19 commits into from
Closed

E2E_PART2 #83

wants to merge 19 commits into from

Conversation

AYAHASSAN287
Copy link
Collaborator

@AYAHASSAN287 AYAHASSAN287 commented Oct 30, 2024

PR Details

NOTE: this branch changes already added in E2E_CI_FIX

This PR has 3 tests for E2E

1-test_filter_20_senders_1_receiver

  1. 20 nodes with relay = true
  2. node 21 is filter node
  3. node 22 makes filter subscription and marks node 31 as filter node
  4. the 20 nodes publish 20 messages
  5. node22 makes filter inquiry to get published messages
  6. check is done to confirm receiving of 20 messages

NOTE: for NODE_2 = GO-WAKU 20 nodes also tested and works fine


2-test_filter_3_senders_multiple_msg_1_receiver
1- 4 nodes started with relay = true
2- node 4 is filter node
3- node5 makes filter request and marks node4 as filter node
4- 3 nodes publish messages_num*3 messages (each publish messages_num)
5- node 5 makes filter inquiry to get published messages
6. check is done to confirm receiving of messages_num messages


3-test_filter_many_subscribed_nodes
1- 2 nodes started with relay = true
2- node 2 is filter node
3- "max_subscribed_nodes" are created and making node2 as their filter node
4- node 1 publish message
5- the "max_subscribed_nodes" inquiry the published messages
6- check is done to ensure each node receives 1 message

NOTE: for NODE_2 = GO-WAKU "max_subscribed_nodes" nodes also tested and works fine


Additional changes

1- Adding flag --max-connections=1000 at file waku_node.py to increase number of connected peers
2- Adding var stress_enabled at env_vars.py when this var = true some limits in number of nodes will increase to allow
extensive testing

Issues reported:

  • Can't test more than 20 nodes due to unexpected errors of connection

@AYAHASSAN287 AYAHASSAN287 marked this pull request as draft October 30, 2024 15:03
@AYAHASSAN287 AYAHASSAN287 changed the title add draft test case E2E_PART2 Oct 31, 2024
@AYAHASSAN287 AYAHASSAN287 marked this pull request as ready for review November 6, 2024 10:02
@AYAHASSAN287 AYAHASSAN287 marked this pull request as draft November 7, 2024 09:45
@fbarbu15 fbarbu15 closed this Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants