Skip to content

Commit

Permalink
[KS-182] keystone: Refactor write_capability + add ChainWriter (#13259)
Browse files Browse the repository at this point in the history
* evm: Add a stub chainwriter impl

* evm: Fix config parameter

* evm: Change the chainwriter receiver name

* evm: Remove the chain writer interface to reference chainlink-common

* evm: Update common dep, and fix signature

* go.sum: Run gomodtidy

* .changeset: Add a changeset

* evm: Pseudo-implement the submit transaction method on chainwriter

* evm: Add txm dependency to chainwriter

* evm: Use the txm param properly

* Update code to use the new interface

* nix: use monthly foundry branch that's persistent

* capabilities: Add config validation to write_target

* capabilities: Pass context into InitializeWrite

* minor: Resolve some inapplicable TODOs

* capabilities: Refactor write target by extracting commmon bits

* Refactor WriteTarget to use ChainWriter

* capabilities: Move evm specific code inside the relayer

* Chainwriter tests (#13360)

* Started relayer evm tests

* Evm Relay tests

* Added generic tests and additional evm tests for WriteTarget

* added changeset

* Update real-tools-tap.md

* lint

* small fix

* Update chainlink-common to include the new interface

* write_target: Fix tests

* Move target capability init inside evm.NewRelayer()

* go mod tidy

* Address lints

* scripts: go mod tidy

* integration-tests: go mod tidy

* evm: Only initialize write target if config actually present

* more tidy, fix last lint

* evm: Move send strategy to be config driven

* evm: Add some todos back

* .changeset: Add a changeset

* write_target: Mock out Config before NewRelayer is called

* Regenerate mocks with right version

* fix evm tests

* goimports

* fix evm tests

---------

Co-authored-by: Nick Corin <nickcorin@gmail.com>
Co-authored-by: Silas Lenihan <32529249+silaslenihan@users.noreply.github.com>
  • Loading branch information
3 people committed May 31, 2024
1 parent 56320dd commit 76dbe19
Show file tree
Hide file tree
Showing 17 changed files with 949 additions and 224 deletions.
5 changes: 5 additions & 0 deletions .changeset/large-plants-count.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"chainlink": minor
---

#internal Added a configuration option to chain writer to set the tx send strategy.
5 changes: 5 additions & 0 deletions .changeset/real-tools-tap.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"chainlink": minor
---

#internal added tests for Chainwriter
5 changes: 5 additions & 0 deletions .changeset/tricky-flowers-exist.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"chainlink": minor
---

#added A ChainWriter implementation in the EVM relay.
189 changes: 189 additions & 0 deletions core/capabilities/targets/mocks/chain_reader.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

109 changes: 109 additions & 0 deletions core/capabilities/targets/mocks/chain_writer.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 76dbe19

Please sign in to comment.