Skip to content

Move go RPC test to its own job serially #4353

Closed
bragaigor wants to merge 3 commits intomasterfrom
braga/rpc-test-separate
Closed

Move go RPC test to its own job serially #4353
bragaigor wants to merge 3 commits intomasterfrom
braga/rpc-test-separate

Conversation

@bragaigor
Copy link
Contributor

Move go RPC test to its own job serially

relates to NIT-4488

@codecov
Copy link

codecov bot commented Feb 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 33.16%. Comparing base (4bf979e) to head (5e4192f).
⚠️ Report is 152 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4353      +/-   ##
==========================================
+ Coverage   33.10%   33.16%   +0.06%     
==========================================
  Files         488      488              
  Lines       57843    57844       +1     
==========================================
+ Hits        19149    19185      +36     
+ Misses      35342    35298      -44     
- Partials     3352     3361       +9     

@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2026

❌ 6 Tests Failed:

Tests completed Failed Passed Skipped
4184 6 4178 0
View the top 3 failed tests by shortest run time
TestDataStreaming_PositiveScenario/Many_senders,_long_messages
Stack Traces | 0.150s run time
... [CONTENT TRUNCATED: Keeping last 20 lines]
        	/home/runner/work/nitro/nitro/util/testhelpers/testhelpers.go:29 +0x55
        github.com/offchainlabs/nitro/daprovider/data_streaming.testBasic.func1()
        	/home/runner/work/nitro/nitro/daprovider/data_streaming/protocol_test.go:230 +0x14f
        created by github.com/offchainlabs/nitro/daprovider/data_streaming.testBasic in goroutine 145
        	/home/runner/work/nitro/nitro/daprovider/data_streaming/protocol_test.go:223 +0x85
        
    protocol_test.go:230: �[31;1m [] too much time has elapsed since request was signed �[0;0m
INFO [02-10|15:31:17.067] rpc response                             method=datastreaming_start logId=10 err="too much time has elapsed since request was signed" result={} attempt=0 args="[\"0x698b4f44\", \"0x30\", \"0xd9\", \"0x27ee\", \"0xa\", \"0xf5bace7239a89f74e6e80271ea4f7a35579116f0d192bf72f81eeb96ac886d7824366285d8b5b9596abd09522a4eac7652373ef609105ae5682f309021ce937700\"]" errorData=null
    protocol_test.go:230: goroutine 311 [running]:
        runtime/debug.Stack()
        	/opt/hostedtoolcache/go/1.25.6/x64/src/runtime/debug/stack.go:26 +0x5e
        github.com/offchainlabs/nitro/util/testhelpers.RequireImpl({0x15694d0, 0xc0004fc540}, {0x154ff80, 0xc001817ef0}, {0x0, 0x0, 0x0})
        	/home/runner/work/nitro/nitro/util/testhelpers/testhelpers.go:29 +0x55
        github.com/offchainlabs/nitro/daprovider/data_streaming.testBasic.func1()
        	/home/runner/work/nitro/nitro/daprovider/data_streaming/protocol_test.go:230 +0x14f
        created by github.com/offchainlabs/nitro/daprovider/data_streaming.testBasic in goroutine 145
        	/home/runner/work/nitro/nitro/daprovider/data_streaming/protocol_test.go:223 +0x85
        
    protocol_test.go:230: �[31;1m [] too much time has elapsed since request was signed �[0;0m
--- FAIL: TestDataStreaming_PositiveScenario/Many_senders,_long_messages (0.15s)
TestDataStreaming_PositiveScenario
Stack Traces | 0.200s run time
=== RUN   TestDataStreaming_PositiveScenario
--- FAIL: TestDataStreaming_PositiveScenario (0.20s)
TestRedisProduceComplex/one_producer,_all_consumers_are_active
Stack Traces | 1.330s run time
... [CONTENT TRUNCATED: Keeping last 20 lines]
�[36mDEBUG�[0m[02-10|15:32:01.509] consumer: xack                           �[36mcid�[0m=0faf0d72-5cc3-443d-9189-8d1d14501b4f �[36mmessageId�[0m=1770737520367-5
�[36mDEBUG�[0m[02-10|15:32:01.509] consumer: xdel                           �[36mcid�[0m=0faf0d72-5cc3-443d-9189-8d1d14501b4f �[36mmessageId�[0m=1770737520367-5
�[36mDEBUG�[0m[02-10|15:32:01.559] checkResponses                           �[36mresponded�[0m=79 �[36merrored�[0m=0 �[36mchecked�[0m=95
�[36mDEBUG�[0m[02-10|15:32:01.565] redis producer: check responses starting
�[36mDEBUG�[0m[02-10|15:32:01.572] checkResponses                           �[36mresponded�[0m=16 �[36merrored�[0m=0 �[36mchecked�[0m=16
�[36mDEBUG�[0m[02-10|15:32:01.578] redis producer: check responses starting
�[36mDEBUG�[0m[02-10|15:32:01.578] checkResponses                           �[36mresponded�[0m=0  �[36merrored�[0m=0 �[36mchecked�[0m=0
�[31mERROR�[0m[02-10|15:32:01.580] Error from XpendingExt in getting PEL for auto claim �[31merr�[0m="context canceled" �[31mpendingLen�[0m=0
�[31mERROR�[0m[02-10|15:32:01.580] Error from XpendingExt in getting PEL for auto claim �[31merr�[0m="context canceled" �[31mpendingLen�[0m=0
�[36mDEBUG�[0m[02-10|15:32:01.583] redis producer: check responses starting
�[36mDEBUG�[0m[02-10|15:32:01.583] checkResponses                           �[36mresponded�[0m=0  �[36merrored�[0m=0 �[36mchecked�[0m=0
�[31mERROR�[0m[02-10|15:32:01.588] Error from XpendingExt in getting PEL for auto claim �[31merr�[0m="context canceled" �[31mpendingLen�[0m=0
�[36mDEBUG�[0m[02-10|15:32:01.588] redis producer: check responses starting
�[36mDEBUG�[0m[02-10|15:32:01.588] checkResponses                           �[36mresponded�[0m=0  �[36merrored�[0m=0 �[36mchecked�[0m=0
�[31mERROR�[0m[02-10|15:32:01.590] Error from XpendingExt in getting PEL for auto claim �[31merr�[0m="context canceled" �[31mpendingLen�[0m=0
�[31mERROR�[0m[02-10|15:32:01.590] Error from XpendingExt in getting PEL for auto claim �[31merr�[0m="context canceled" �[31mpendingLen�[0m=0
�[31mERROR�[0m[02-10|15:32:01.591] Error from XpendingExt in getting PEL for auto claim �[31merr�[0m="context canceled" �[31mpendingLen�[0m=0
    pubsub_test.go:408: mergeMaps() unexpected error: duplicate key: 1770737520335-0
�[36mDEBUG�[0m[02-10|15:32:01.659] Error destroying a stream group          �[36merror�[0m="dial tcp 127.0.0.1:32931: connect: connection refused"
--- FAIL: TestRedisProduceComplex/one_producer,_all_consumers_are_active (1.33s)

📣 Thoughts on this report? Let Codecov know! | Powered by Codecov

Signed-off-by: Igor Braga <5835477+bragaigor@users.noreply.github.com>
Signed-off-by: Igor Braga <5835477+bragaigor@users.noreply.github.com>
Signed-off-by: Igor Braga <5835477+bragaigor@users.noreply.github.com>
@bragaigor bragaigor force-pushed the braga/rpc-test-separate branch from 4e95818 to 5e4192f Compare February 10, 2026 15:21
@bragaigor
Copy link
Contributor Author

Further investigation will be conducted as part of NIT-4515 and such tests are being disabled temporarily in #4366

@bragaigor bragaigor closed this Feb 11, 2026
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.

1 participant