Skip to content

Conversation

@AliceInHunterland
Copy link
Contributor

@AliceInHunterland AliceInHunterland commented Mar 13, 2024

@AliceInHunterland AliceInHunterland force-pushed the CommitteeChanged branch 2 times, most recently from 59d47f7 to 7a2e79e Compare March 15, 2024 11:47
@AliceInHunterland AliceInHunterland force-pushed the CommitteeChanged branch 2 times, most recently from e325924 to 104e913 Compare March 15, 2024 15:59
@AliceInHunterland AliceInHunterland force-pushed the CommitteeChanged branch 2 times, most recently from 41239c8 to 20fa789 Compare March 18, 2024 14:45
Copy link
Member

@AnnaShaleva AnnaShaleva left a comment

Choose a reason for hiding this comment

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

To fix failing findstorage tests just update the testchain's block 20 stateroot, it was changed after one of the previous PRs:

diff --git a/pkg/services/rpcsrv/server_test.go b/pkg/services/rpcsrv/server_test.go
index c0b0fe15..f66844da 100644
--- a/pkg/services/rpcsrv/server_test.go
+++ b/pkg/services/rpcsrv/server_test.go
@@ -87,7 +87,7 @@ const (
        faultedTxHashLE                   = "82279bfe9bada282ca0f8cb8e0bb124b921af36f00c69a518320322c6f4fef60"
        faultedTxBlock             uint32 = 23
        invokescriptContractAVM           = "VwIADBQBDAMOBQYMDQIODw0DDgcJAAAAAErZMCQE2zBwaEH4J+yMqiYEEUAMFA0PAwIJAAIBAwcDBAUCAQAOBgwJStkwJATbMHFpQfgn7IyqJgQSQBNA"
-       block20StateRootLE                = "858c873539d6d24a70f2be13f9dafc61aef2b63c2aa16bb440676de6e44e3cf1"
+       block20StateRootLE                = "397c69adbc0201d59623fa913bfff4a2da25c792c484d1d278c061709f2c21cf"
 )
 
 var (

@codecov
Copy link

codecov bot commented Mar 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.83%. Comparing base (a81dc5c) to head (7c25755).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3351      +/-   ##
==========================================
+ Coverage   84.79%   84.83%   +0.03%     
==========================================
  Files         331      331              
  Lines       44939    44961      +22     
==========================================
+ Hits        38107    38143      +36     
+ Misses       5319     5309      -10     
+ Partials     1513     1509       -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@AnnaShaleva AnnaShaleva left a comment

Choose a reason for hiding this comment

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

Otherwise LGTM.

@AnnaShaleva AnnaShaleva merged commit 308475f into master Mar 21, 2024
@AnnaShaleva AnnaShaleva deleted the CommitteeChanged branch March 21, 2024 16:37
AnnaShaleva added a commit that referenced this pull request May 13, 2025
AnnaShaleva added a commit that referenced this pull request May 13, 2025
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.

Port CommitteeChanged events

3 participants