2.20.0 beta fixes for llo

[samsondav]

No description provided.

[github-actions]

I see you updated files related to core. Please run pnpm changeset in the root directory to add a changeset as well as in the text include at least one of the following tags:

• #added For any new functionality added.
• #breaking_change For any functionality that requires manual action for the node to boot.
• #bugfix For bug fixes.
• #changed For any change to the existing functionality.
• #db_update For any feature that introduces updates to database schema.
• #deprecation_notice For any upcoming deprecation functionality.
• #internal For changesets that need to be excluded from the final changelog.
• #nops For any feature that is NOP facing and needs to be in the official Release Notes for the release.
• #removed For any functionality/config that is removed.
• #updated For any functionality that is updated.
• #wip For any change that is not ready yet and external communication about it should be held off till it is feature complete.

🎖️ No JIRA issue number found in: PR title, commit message, or branch name. Please include the issue ID in one of these.

[cl-sonarqube-production]

Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

[github-actions]

AER Report: CI Core

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

1. Receiver name inconsistency in mercuryServer methods: [Golang Lint]

Source of Error:

##[error]core/services/llo/grpc/client_test.go:125:1: receiver-naming: receiver name s should be consistent with previous receiver name srv for mercuryServer (revive)
func (s *mercuryServer) Transmit(ctx context.Context, req *rpc.TransmitRequest) (*rpc.TransmitResponse, error) {
	s.packetsCh <- packet{ctx, req}

	return &rpc.TransmitResponse{
		Code: 1,
		Error: "",
	}, nil
}
##[error]core/services/llo/grpc/client_test.go:134:1: receiver-naming: receiver name s should be consistent with previous receiver name srv for mercuryServer (revive)
func (s *mercuryServer) LatestReport(ctx context.Context, lrr *rpc.LatestReportRequest) (*rpc.LatestReportResponse, error) {
	panic("should not be called")
}

**Why**: The receiver name `s` is inconsistent with the previously used receiver name `srv` for the `mercuryServer` type, causing a linting error.

Suggested fix: Rename the receiver s to srv in the methods to maintain consistency.

2. Unnecessary conversion: [Golang Lint]

Source of Error:

##[error]core/services/llo/grpc/client_test.go:43:66: unnecessary conversion (unconvert)
	serverURL := srv.start(t, []ed25519.PublicKey{ed25519.PublicKey(clientPrivKey.Public().(ed25519.PublicKey))})
		 ^

**Why**: The conversion `ed25519.PublicKey(clientPrivKey.Public().(ed25519.PublicKey))` is unnecessary and redundant.

Suggested fix: Remove the unnecessary conversion to simplify the code.

3. Struct contains a context.Context field: [Golang Lint]

Source of Error:

##[error]core/services/llo/grpc/client_test.go:121:2: found a struct that contains a context.Context field (containedctx)
	ctx context.Context
	^

**Why**: Including a `context.Context` field in a struct is discouraged as it can lead to context misuse and memory leaks.

Suggested fix: Refactor the code to pass context.Context as a parameter to functions instead of embedding it in a struct.

4. Variable shadowing: [Golang Lint]

Source of Error:

##[error]core/services/llo/grpc/client_test.go:65:8: shadow: declaration of "packet" shadows declaration at line 120 (govet)
	case packet := <-ch:
		 ^

**Why**: The variable `packet` declared in the `case` statement shadows another variable with the same name declared at line 120, which can lead to unexpected behavior.

Suggested fix: Rename one of the packet variables to avoid shadowing and ensure clarity in the code.