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

CCIP-4058 replace f with fObserve and fSign in RMHome and RMNRemote #15605

Open
wants to merge 26 commits into
base: develop
Choose a base branch
from

Conversation

jhweintraub
Copy link
Collaborator

@jhweintraub jhweintraub commented Dec 10, 2024

Replace RMN Configs f with fObserve in RMNHome and fSign in RMNRemote to be more descriptive and modify all tests as well as update the relevant ones to use the new test naming guide.

@jhweintraub jhweintraub requested a review from a team as a code owner December 10, 2024 16:47
Copy link
Contributor

github-actions bot commented Dec 10, 2024

Static analysis results are available

Hey @asoliman92, you can view Slither reports in the job summary here or download them as artifact here.
Please check them before merging and make sure you have addressed all issues.

Copy link
Contributor

github-actions bot commented Dec 10, 2024

AER Report: CI Core

aer_workflow , commit , Clean Go Tidy & Generate , Detect Changes , Scheduled Run Frequency , GolangCI Lint (core/scripts) , GolangCI Lint (.) , GolangCI Lint (integration-tests/load) , GolangCI Lint (integration-tests) , GolangCI Lint (deployment) , test-scripts , Core Tests (go_core_tests) , Core Tests (go_core_tests_integration) , Core Tests (go_core_ccip_deployment_tests) , Core Tests (go_core_fuzz) , Core Tests (go_core_race_tests) , lint , SonarQube Scan

1. usdcReader.MessageHashes undefined:[Golang Lint (.)]

Source of Error:
core/capabilities/ccip/ccip_integration_tests/usdcreader/usdcreader_test.go:198:31: usdcReader.MessageHashes undefined (type "github.com/smartcontractkit/chainlink-ccip/pkg/reader".USDCMessageReader has no field or method MessageHashes)
core/capabilities/ccip/ccip_integration_tests/usdcreader/usdcreader_test.go:273:31: usdcReader.MessageHashes undefined (type "github.com/smartcontractkit/chainlink-ccip/pkg/reader".USDCMessageReader has no field or method MessageHashes)

Why: The error occurs because the USDCMessageReader type from the github.com/smartcontractkit/chainlink-ccip/pkg/reader package does not have a field or method named MessageHashes. This indicates a mismatch between the expected and actual structure of the USDCMessageReader type.

Suggested fix: Verify the definition of USDCMessageReader in the github.com/smartcontractkit/chainlink-ccip/pkg/reader package and ensure it includes the MessageHashes field or method. If it does not, update the test code to use the correct field or method that exists in USDCMessageReader.

2. usdcReader.MessageHashes undefined:[Run tests]

Source of Error:
core/capabilities/ccip/ccip_integration_tests/usdcreader/usdcreader_test.go:198:31: usdcReader.MessageHashes undefined (type "github.com/smartcontractkit/chainlink-ccip/pkg/reader".USDCMessageReader has no field or method MessageHashes)
core/capabilities/ccip/ccip_integration_tests/usdcreader/usdcreader_test.go:273:31: usdcReader.MessageHashes undefined (type "github.com/smartcontractkit/chainlink-ccip/pkg/reader".USDCMessageReader has no field or method MessageHashes)

Why: The error occurs because the USDCMessageReader type from the github.com/smartcontractkit/chainlink-ccip/pkg/reader package does not have a field or method named MessageHashes. This indicates a mismatch between the expected and actual structure of the USDCMessageReader type.

Suggested fix: Verify the definition of USDCMessageReader in the github.com/smartcontractkit/chainlink-ccip/pkg/reader package and ensure it includes the MessageHashes field or method. If it does not, update the test code to use the correct field or method that exists in USDCMessageReader.

AER Report: Operator UI CI ran successfully ✅

aer_workflow , commit

@jhweintraub jhweintraub requested a review from a team as a code owner December 11, 2024 15:36
@jhweintraub jhweintraub requested a review from Atrax1 December 11, 2024 15:36
@jhweintraub jhweintraub changed the title CCIP-4058 replace f with fObserve in RMHome and RMNRemote CCIP-4058 replace f with fObserve and fSign in RMHome and RMNRemote Dec 13, 2024
@jhweintraub jhweintraub requested a review from a team as a code owner December 17, 2024 14:58
@jhweintraub jhweintraub requested a review from a team as a code owner January 8, 2025 15:35
@jhweintraub jhweintraub requested a review from a team as a code owner January 8, 2025 15:44
@jhweintraub jhweintraub force-pushed the fix/renamefObserve_rmn branch from e436a3c to d99e573 Compare January 8, 2025 15:45
@cl-sonarqube-production
Copy link

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.

3 participants