Skip to content

Gh 788 multinode test - #577

Open
czarte wants to merge 280 commits into
masterfrom
GH-788_multinode_test
Open

Gh 788 multinode test#577
czarte wants to merge 280 commits into
masterfrom
GH-788_multinode_test

Conversation

@czarte

@czarteczarte commented Feb 4, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

czarte added 30 commits May 13, 2024 19:05
@czarte
fixing tests for gossip_acceptor
3c05c5b
@czarte
fixing tests for gossip acceptor
6ef0d20
@czarte
fixing tests for gossip_acceptor
72ef4d0
@czarte
fixed tests for gossip_acceptor
1a1e3f3
@czarte
take out performance tests for node_location from ci/all, fixed tests…
40ff41a
… for gossip_acceptor works with country_code
@czarte
add mock for location in make_node_record, fixing tests
79dc64f
@czarte
removing obsolete injections
afbc510
@czarte
fixed last test for gossip_acceptor
eb73341
@czarte
add country_code and metadata for country_code into the multinode int…
1a9eb4f
…egration test
@czarte
formatting
0c8b33d
@czarte
add start blockchain to test
c7b72e2
@czarte
fixing debtors_are_credited_once_but_not_twice
811f569
@czarte
fix the panic
f242cb0
@czarte
fixing test debtors_are_credited_once_but_not_twice failing on local …
a2b4fdd
…machine and in actions
@czarte
test for routing_engine failing on wrong country code
e403d0a
@czarte
fixed test debtors_are_credited_once_but_not_twice - added response f…
b4bb58c
…or start block request
@czarte
formatting and removing condition for debtors_are_credited_once_but_n…
5b7a27c
…ot_twice to run in Actions
@czarte
fixed Error messages in test for BlockchainBridge
2c93011
@czarte
remove unused import
d90d558
@czarte
exclude test debtors_are_credited_once_but_not_twice from Actions aga…
0243e9c
…in, test_node_1 does not contain local descriptor in log after startup
@czarte
target_country_opt variable to RouteQueryMessage
2023ce4
@czarte
move get_node_location out of arguments for construction of NodeRecor…
f2426a9
…dMetadata::new(), test for two debut in the same time with empty db
@czarte
fix gossip_acceptor try_accept_debut to be able reply correctly in ca…
69005b1
…se of no full naighborship in DB
@czarte
add more testing for two_debut_in_same_time_with_empty_db_handled_by_…
b0c050e
…try_accept_debut, start of binary search in country_finder
@czarte
introduce in memory Vec<CountryBlock> to speed up ip_country lookup, …
3232480
…one test on testing data for ipV6 still failing
@czarte
introducing in memory country_finder blocks for speeding up the ip_co…
05a5fab
…untry lookup
@czarte
fix node_location for CountryCodeFinder::find_country, adjust perform…
8e8d275
…ace tests to match countries
@czarte
formatting
53d583c
@czarte
remove commented code
48136dd
@czarte
fixing the tests handling gossips & formatting
1d6e0b6
bertllll
bertllll reviewed Feb 27, 2025
View reviewed changes
Comment thread.idea/modules.xml
bertllll
bertllll reviewed Feb 27, 2025
View reviewed changes
Comment threadip_country/ci/license.sh
Comment threadip_country/src/ip_country.rs
Comment threadmasq/src/commands/exit_location_command.rs
Comment threadmasq/src/commands/exit_location_command.rs
Comment threadmasq/src/commands/exit_location_command.rs
.help(SHOW_COUNTRIES_HELP)
.takes_value(false)
.required(false),
)

@bertllllbertllllFeb 27, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tady pak bys mohl pridat dalsi nastaveni pomoci 'ArgGroups', ktery ti dovoli upravovat vztah mezi jednotlivymi argumenty. Napr. kdyz je jeden pritomny, jiny nesmi byt pritomny. Muzes se inspirovat, u nejakeho jineho prikazu to take je.

@bertllllbertllllApr 6, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

K tomu testovani, jak ses me ptal:

Tohle lze testovat pres chybovy hlasky. V tom testu muzes dodat dva konfliktni argumenty a assertovat na chybovou hlasku (Clap si toho vsimne a zmini ten konflikt v te hlasce).
Ale bude se opet jednat pouze o unit test. Dost podobny nejakym tem, ktere uz jsi napsal.

Comment threadmasq/src/commands/exit_location_command.rs
Comment threadmasq/src/commands/exit_location_command.rs
Comment threadmasq/src/commands/exit_location_command.rs
Comment threadmasq/src/commands/exit_location_command.rs
Comment threadmasq/src/commands/exit_location_command.rs
bertllll
bertllll reviewed Feb 27, 2025
View reviewed changes
Comment threadmasq_lib/src/constants.rs
Comment threadmasq_lib/src/messages.rs
Comment threadmasq_lib/src/messages.rs
Comment threadmasq_lib/src/messages.rs
Comment threadmasq_lib/src/messages.rs
Comment threadnode/src/bootstrapper.rs
Comment threadnode/src/daemon/setup_reporter.rs
Comment threadnode/src/node_configurator/unprivileged_parse_args_configuration.rs
Comment threadnode/tests/initialization_test.rs
Comment threadnode/tests/ui_gateway_test.rs
bertllll
bertllll reviewed Mar 3, 2025
View reviewed changes
Comment threadnode/src/neighborhood/gossip_acceptor.rs
Comment threadnode/src/neighborhood/gossip_acceptor.rs
Comment threadnode/src/neighborhood/gossip_acceptor.rs
Comment threadnode/src/neighborhood/gossip_acceptor.rs
Comment threadnode/src/neighborhood/gossip_acceptor.rs
Comment threadnode/src/neighborhood/gossip_acceptor.rs
Comment threadnode/src/neighborhood/gossip_acceptor.rs
Comment threadnode/src/neighborhood/gossip_acceptor.rs
Comment threadnode/src/neighborhood/gossip_acceptor.rs
Comment threadnode/src/neighborhood/gossip_acceptor.rs
bertllll
bertllll reviewed Mar 3, 2025
View reviewed changes
Comment threadnode/src/test_utils/neighborhood_test_utils.rs
Comment threadnode/src/test_utils/neighborhood_test_utils.rs
Comment threadnode/src/neighborhood/node_location.rs
bertllll
bertllll reviewed Mar 4, 2025
View reviewed changes
Comment threadnode/src/neighborhood/neighborhood_database.rs
Comment threadnode/src/neighborhood/neighborhood_database.rs
Comment threadnode/src/neighborhood/neighborhood_database.rs
Comment threadnode/src/neighborhood/neighborhood_database.rs
Comment threadnode/src/neighborhood/neighborhood_database.rs
Comment threadnode/src/neighborhood/neighborhood_database.rs
Comment threadnode/src/neighborhood/neighborhood_database.rs
Comment threadnode/src/neighborhood/node_record.rs
bertllll
bertllll reviewed Mar 4, 2025
View reviewed changes
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadmasq_lib/src/messages.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadmasq_lib/src/messages.rs
bertllll
bertllll reviewed Mar 5, 2025
View reviewed changes
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
bertllll
bertllll reviewed Mar 5, 2025
View reviewed changes
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
bertllll
bertllll reviewed Mar 5, 2025
View reviewed changes
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
bertllll
bertllll reviewed Mar 6, 2025
View reviewed changes
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadmasq_lib/src/messages.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
bertllll
bertllll reviewed Mar 6, 2025
View reviewed changes
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
bertllll
bertllll reviewed Mar 6, 2025
View reviewed changes
Comment threadnode/src/neighborhood/mod.rs
bertllll
bertllll reviewed Mar 7, 2025
View reviewed changes
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
bertllll
bertllll reviewed Mar 7, 2025
View reviewed changes
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
bertllll
bertllll reviewed Mar 7, 2025
View reviewed changes
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
bertllll
bertllll reviewed Mar 8, 2025
View reviewed changes
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
bertllll
bertllll reviewed Mar 9, 2025
View reviewed changes
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
bertllll
bertllll reviewed Mar 9, 2025
View reviewed changes
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
bertllll
bertllll reviewed Mar 12, 2025
View reviewed changes
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
bertllll
bertllll reviewed Mar 14, 2025
View reviewed changes
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
bertllll
bertllll reviewed Mar 14, 2025
View reviewed changes
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Comment threadnode/src/neighborhood/mod.rs
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bertllllbertllllbertllll left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@czarte@bertllll