Skip to content
This repository has been archived by the owner on Apr 4, 2024. It is now read-only.

[ENG-626] Rpc types integration tests #1207

Merged
merged 10 commits into from
Aug 8, 2022
Merged

Conversation

ramacarlucho
Copy link
Contributor

Closes: #XXX

Description

Check rpc return types against geth, and expected results


For contributor use:

  • Targeted PR against correct branch (see CONTRIBUTING.md)
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Code follows the module structure standards.
  • Wrote unit and integration tests
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added relevant godoc comments.
  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • Re-reviewed Files changed in the Github PR explorer

For admin use:

  • Added appropriate labels to PR (ex. WIP, R4R, docs, etc)
  • Reviewers assigned
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

@github-actions github-actions bot added the Type: Tests issues and PR related to tests label Jul 27, 2022
@facs95 facs95 marked this pull request as ready for review August 1, 2022 13:21
tests/integration_tests/test_types.py Outdated Show resolved Hide resolved
@yihuang
Copy link
Contributor

yihuang commented Aug 4, 2022

You need to merge with the recent main branch to get the integration test running, currently, it's not running in github actions.

@ramacarlucho ramacarlucho requested a review from danburck as a code owner August 4, 2022 12:15
@codecov
Copy link

codecov bot commented Aug 4, 2022

Codecov Report

Merging #1207 (41484b1) into main (ccbaf1f) will not change coverage.
The diff coverage is 20.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1207   +/-   ##
=======================================
  Coverage   57.20%   57.20%           
=======================================
  Files          94       94           
  Lines        8424     8424           
=======================================
  Hits         4819     4819           
  Misses       3358     3358           
  Partials      247      247           
Impacted Files Coverage Δ
rpc/backend/evm_backend.go 24.19% <20.00%> (ø)

Copy link
Contributor

@fedekunze fedekunze left a comment

Choose a reason for hiding this comment

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

minor comments

rpc/namespaces/ethereum/eth/api.go Outdated Show resolved Hide resolved
rpc/namespaces/ethereum/eth/api.go Show resolved Hide resolved
@facs95 facs95 merged commit d1543ce into main Aug 8, 2022
@facs95 facs95 deleted the rama/rpc-types-integration-tests branch August 8, 2022 13:01
facs95 pushed a commit that referenced this pull request Aug 12, 2022
* types tests

* cleanup

* cleanup

* fix differences in the rpc calls with geth

* cleanup

* fix linter. getProof future queries now behave as geth

* fix spelling

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
hoanguyenkh pushed a commit to AstraProtocol/ethermint that referenced this pull request Aug 17, 2022
* types tests

* cleanup

* cleanup

* fix differences in the rpc calls with geth

* cleanup

* fix linter. getProof future queries now behave as geth

* fix spelling

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
devon-chain pushed a commit to PundiAI/ethermint that referenced this pull request Nov 17, 2022
* types tests

* cleanup

* cleanup

* fix differences in the rpc calls with geth

* cleanup

* fix linter. getProof future queries now behave as geth

* fix spelling

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
(cherry picked from commit e8fd0e3)
@danburck danburck mentioned this pull request Nov 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Type: Tests issues and PR related to tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants