Skip to content
This repository was archived by the owner on Mar 14, 2025. It is now read-only.

Conversation

@stackman27
Copy link
Collaborator

Motivation

Solution

DylanTinianov and others added 2 commits January 8, 2025 10:15
* Update rpc client

* Update RPCClient

* Update RPCClient

* use Eth Subscription for heads

* Make RpcClient public

* Update node

* Update Multinode

* Fix build + generate

* Udate multinode

* update multinode

* fix tests

* Fix mocks

* Update node_lifecycle_test.go

* Fix all client tests

* Fix tests

* Update client_test.go

* go mod tidy

* fix tests

* Fix tests

* Clean up

* Fix features test mocking

* Fix logging

* Remove logging

* Fix tests

* Fix context

* lint

* Update node_lifecycle_test.go

* Remove unused generics

* Add state locking

* Set block difficulty

* Update node_lifecycle.go

* Fix tests

* Make NodeStates public

* Update multi_node_test.go

* Update Unsubscribe

* Remove HEAD generic from Node interface

* Remove unneeded generics

* Remove unneeded generics from Multinode

* Remove Multinode as interface

* Add PoolChainInfoProvider

* Setup SendOnly nodes

* Test empty context

* Add err to log

* Add rpc newHeads method

* Fix context

* Changeset

* Remove unused mocks

* Create Transaction Sender

* Implement TransactionSender

* Fix transaction sender types

* Implement transaction sender tests

* Ensure MultiNode is running

* Address comments

* Remove ChainClientRPC interface

* Remove unneeded test

* Generate mocks

* Fix tests

* fix tests

* Use UnsubscribeAllExcept

* Fix rpc client tests

* Address comments

* Remove unused code

* Generate private mock

* lint

* Fix locks and unsubscribing

* Update node.go

* fixed flaky headtracker tests

* Update node_lifecycle_test.go

* Update node_lifecycle_test.go

* Update transaction sender

* Update chain_client_test.go

* Remove unused variables

* lint

* enable secondary url test

* Use http if not nil

* Update transaction_sender.go

* Add close method

* Update rpc_client.go

* changeset

* Add Close to transaction sender

* Return struct not interface

* Remove flaky test

* Fix context

* Test if mn closes

* Fix context cancel

* Fix DoAll

* lint generate

* Revert

* Fix mocks

* Update node_lifecycle.go

* lint

* Delete rpc_client.go

* Address comments

* Fix tests

* Update common/client/send_only_node.go

Co-authored-by: Dmytro Haidashenko <[email protected]>

* Address comments

* Address comments

* Rename SubscribeToHeads

* Return code

* Update transaction_sender.go

* Fix merge conflicts

* Handle batch requests

* Delete mock_rpc_client_test.go

* Update common/client/types.go

Co-authored-by: Dmytro Haidashenko <[email protected]>

* Move RPC methods

* Use map for subs

* Don't cancel context

* Update transaction_sender.go

* Fix lint

* Delete late-mails-battle.md

* lint

* lint

* Check Transaction Sender state

* Return Retryable

* Update core/chains/evm/client/chain_client_test.go

Co-authored-by: Jordan Krage <[email protected]>

* Update sub_forwarder.go

* Update config.go

* lint

* Don't export nodeState

* Update changeset

* Use subsSliceMu

* lint

* Update rpc_client.go

* nil ws url for sendonly nodes

* Use head polling

---------

Co-authored-by: Dmytro Haidashenko <[email protected]>
Co-authored-by: Dmytro Haidashenko <[email protected]>
Co-authored-by: Jordan Krage <[email protected]>
* add sei chain and error mapping

* fix changeset and config_test

* remove sei chain type

* add pricemax

* custom calculation of log's index for Sei

* fix lint issues & tests

---------

Co-authored-by: flodesi <[email protected]>
@stackman27 stackman27 requested a review from a team as a code owner January 8, 2025 18:25
@stackman27 stackman27 requested a review from a team as a code owner January 8, 2025 18:29
@cl-sonarqube-production
Copy link

Quality Gate failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 75%)

See analysis details on SonarQube

@stackman27 stackman27 closed this Jan 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants