status-im / status-go

The Status module that consumes go-ethereum
https://status.im
Mozilla Public License 2.0
719 stars 243 forks source link

Test/ut router #5419

Open IvanBelyakoff opened 6 days ago

IvanBelyakoff commented 6 days ago

Router V2 refactoring to smaller pieces to allow unit testing

These are sample tests to demo. Proper better return values and params should be used.

The code is complex, so within a limited timeframe I had I've made an attempt to:

More attention should be to paid to:

Separately should be tested:

status-im-auto commented 6 days ago

Jenkins Builds

:grey_question: Commit :hash: Finished (UTC) Duration Platform Result
:heavy_multiplication_x: 448fe3f1 #1 2024-06-25 01:41:31 ~1 min tests :page_facing_up:log
:heavy_multiplication_x: 448fe3f1 #2 2024-06-26 10:08:12 ~3 min tests :page_facing_up:log
:heavy_check_mark: 448fe3f1aa0858c29f557111cc8df61645f0ce7c #1 2024-06-25 01:44:14 ~4 min ios :package:zip
:heavy_check_mark: 448fe3f1aa0858c29f557111cc8df61645f0ce7c #1 2024-06-25 01:44:18 ~4 min linux :package:zip
:heavy_check_mark: 448fe3f1aa0858c29f557111cc8df61645f0ce7c #1 2024-06-25 01:45:43 ~5 min android :package:aar
:heavy_check_mark: 448fe3f1aa0858c29f557111cc8df61645f0ce7c #2 2024-06-26 10:07:25 ~2 min ios :package:zip
:heavy_check_mark: 448fe3f1aa0858c29f557111cc8df61645f0ce7c #2 2024-06-26 10:07:40 ~3 min android :package:aar
:heavy_check_mark: 448fe3f1aa0858c29f557111cc8df61645f0ce7c #2 2024-06-26 10:08:32 ~3 min linux :package:zip