Script executing execution api tests #1621
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
natanasow
previously approved these changes
Aug 11, 2023
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
konstantinabl
force-pushed
the
1475-design-explore-continuous-automated-rpc-api-schema-verification
branch
3 times, most recently
from
a47925c to
76c148c
Compare
konstantinabl
force-pushed
the
1475-design-explore-continuous-automated-rpc-api-schema-verification
branch
2 times, most recently
from
1b1345e to
ee7093e
Compare
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #1621 +/- ##
=======================================
Coverage 76.65% 76.65%
=======================================
Files 39 39
Lines 2913 2913
Branches 584 584
=======================================
Hits 2233 2233
Misses 499 499
Partials 181 181 ☔ View full report in Codecov by Sentry. |
konstantinabl
changed the title
konstantinabl
force-pushed
the
1475-design-explore-continuous-automated-rpc-api-schema-verification
branch
2 times, most recently
from
4fb240f to
03e7f40
Compare
konstantinabl
requested review from
AlfredoG87,
ebadiere and
Nana-EC
as code owners
Signed-off-by: Konstantina Blazhukova <[email protected]>
Signed-off-by: Konstantina Blazhukova <[email protected]>
Signed-off-by: Konstantina Blazhukova <[email protected]>
konstantinabl
force-pushed
the
1475-design-explore-continuous-automated-rpc-api-schema-verification
branch
from
7ee56de to
4b3d291
Compare
Signed-off-by: Konstantina Blazhukova <[email protected]>
Nana-EC
requested changes
Sep 7, 2023
Signed-off-by: Konstantina Blazhukova <[email protected]>
Signed-off-by: Konstantina Blazhukova <[email protected]>
Signed-off-by: Konstantina Blazhukova <[email protected]>
Signed-off-by: Konstantina Blazhukova <[email protected]>
Signed-off-by: Konstantina Blazhukova <[email protected]>
Nana-EC
previously approved these changes
Sep 11, 2023
konstantinabl
force-pushed
the
1475-design-explore-continuous-automated-rpc-api-schema-verification
branch
from
4c1bb3d to
023576c
Compare
Signed-off-by: Konstantina Blazhukova <[email protected]>
konstantinabl
force-pushed
the
1475-design-explore-continuous-automated-rpc-api-schema-verification
branch
from
023576c to
f35aa56
Compare
Signed-off-by: Konstantina Blazhukova <[email protected]>
Signed-off-by: Konstantina Blazhukova <[email protected]>
|
Kudos, SonarCloud Quality Gate passed!
|
Nana-EC
approved these changes
Sep 13, 2023
konstantinabl
deleted the
1475-design-explore-continuous-automated-rpc-api-schema-verification
branch
ebadiere
pushed a commit
that referenced
this pull request
Sep 18, 2023
* Draft of script executing execution api tests Signed-off-by: Konstantina Blazhukova <[email protected]> * Refactors script to make it more readable and modular Signed-off-by: Konstantina Blazhukova <[email protected]> * Fixes PR comments Signed-off-by: Konstantina Blazhukova <[email protected]> * Resolves conflict Signed-off-by: Konstantina Blazhukova <[email protected]> * Fixes for some transaction methods and sendRawTransaction Signed-off-by: Konstantina Blazhukova <[email protected]> * Fixes package-lock.json Signed-off-by: Konstantina Blazhukova <[email protected]> * Tries to fix package-lock.json conflicts Signed-off-by: Konstantina Blazhukova <[email protected]> * Package lock.json Signed-off-by: Konstantina Blazhukova <[email protected]> * Refactors script to handle different transaction for getTransactionByHash Signed-off-by: Konstantina Blazhukova <[email protected]> * Fixes script for all methods and optimizes it Signed-off-by: Konstantina Blazhukova <[email protected]> * Sets nonce via getTransactionCount Signed-off-by: Konstantina Blazhukova <[email protected]> * Refactors main to make it more readable Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds execution api repo as dependency Signed-off-by: Konstantina Blazhukova <[email protected]> * Refactors script to test file Signed-off-by: Konstantina Blazhukova <[email protected]> * Deletes script Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds command for executing new tests Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds conformity tests to github workflow Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds new dependencies to package-lock Signed-off-by: Konstantina Blazhukova <[email protected]> * Fixes typo in package.json Signed-off-by: Konstantina Blazhukova <[email protected]> * Changes github workflow name Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds comments and docs; Takes bytecode from json Signed-off-by: Konstantina Blazhukova <[email protected]> * Fixes script name for conformity tests Signed-off-by: Konstantina Blazhukova <[email protected]> * Removes redundant variable; moves variable to const Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds specifig tag to execution apis repo Signed-off-by: Konstantina Blazhukova <[email protected]> * Replaces file name string with constant variables Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds more informative variables for request.param checks Signed-off-by: Konstantina Blazhukova <[email protected]> * Combines sendrawTransaction checks Signed-off-by: Konstantina Blazhukova <[email protected]> --------- Signed-off-by: Konstantina Blazhukova <[email protected]> Signed-off-by: ebadiere <[email protected]>
ebadiere
pushed a commit
that referenced
this pull request
Sep 20, 2023
* Draft of script executing execution api tests Signed-off-by: Konstantina Blazhukova <[email protected]> * Refactors script to make it more readable and modular Signed-off-by: Konstantina Blazhukova <[email protected]> * Fixes PR comments Signed-off-by: Konstantina Blazhukova <[email protected]> * Resolves conflict Signed-off-by: Konstantina Blazhukova <[email protected]> * Fixes for some transaction methods and sendRawTransaction Signed-off-by: Konstantina Blazhukova <[email protected]> * Fixes package-lock.json Signed-off-by: Konstantina Blazhukova <[email protected]> * Tries to fix package-lock.json conflicts Signed-off-by: Konstantina Blazhukova <[email protected]> * Package lock.json Signed-off-by: Konstantina Blazhukova <[email protected]> * Refactors script to handle different transaction for getTransactionByHash Signed-off-by: Konstantina Blazhukova <[email protected]> * Fixes script for all methods and optimizes it Signed-off-by: Konstantina Blazhukova <[email protected]> * Sets nonce via getTransactionCount Signed-off-by: Konstantina Blazhukova <[email protected]> * Refactors main to make it more readable Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds execution api repo as dependency Signed-off-by: Konstantina Blazhukova <[email protected]> * Refactors script to test file Signed-off-by: Konstantina Blazhukova <[email protected]> * Deletes script Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds command for executing new tests Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds conformity tests to github workflow Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds new dependencies to package-lock Signed-off-by: Konstantina Blazhukova <[email protected]> * Fixes typo in package.json Signed-off-by: Konstantina Blazhukova <[email protected]> * Changes github workflow name Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds comments and docs; Takes bytecode from json Signed-off-by: Konstantina Blazhukova <[email protected]> * Fixes script name for conformity tests Signed-off-by: Konstantina Blazhukova <[email protected]> * Removes redundant variable; moves variable to const Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds specifig tag to execution apis repo Signed-off-by: Konstantina Blazhukova <[email protected]> * Replaces file name string with constant variables Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds more informative variables for request.param checks Signed-off-by: Konstantina Blazhukova <[email protected]> * Combines sendrawTransaction checks Signed-off-by: Konstantina Blazhukova <[email protected]> --------- Signed-off-by: Konstantina Blazhukova <[email protected]> Signed-off-by: ebadiere <[email protected]>
mshakeg
pushed a commit
to mshakeg/hedera-json-rpc-relay
that referenced
this pull request
Oct 18, 2023
* Draft of script executing execution api tests Signed-off-by: Konstantina Blazhukova <[email protected]> * Refactors script to make it more readable and modular Signed-off-by: Konstantina Blazhukova <[email protected]> * Fixes PR comments Signed-off-by: Konstantina Blazhukova <[email protected]> * Resolves conflict Signed-off-by: Konstantina Blazhukova <[email protected]> * Fixes for some transaction methods and sendRawTransaction Signed-off-by: Konstantina Blazhukova <[email protected]> * Fixes package-lock.json Signed-off-by: Konstantina Blazhukova <[email protected]> * Tries to fix package-lock.json conflicts Signed-off-by: Konstantina Blazhukova <[email protected]> * Package lock.json Signed-off-by: Konstantina Blazhukova <[email protected]> * Refactors script to handle different transaction for getTransactionByHash Signed-off-by: Konstantina Blazhukova <[email protected]> * Fixes script for all methods and optimizes it Signed-off-by: Konstantina Blazhukova <[email protected]> * Sets nonce via getTransactionCount Signed-off-by: Konstantina Blazhukova <[email protected]> * Refactors main to make it more readable Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds execution api repo as dependency Signed-off-by: Konstantina Blazhukova <[email protected]> * Refactors script to test file Signed-off-by: Konstantina Blazhukova <[email protected]> * Deletes script Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds command for executing new tests Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds conformity tests to github workflow Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds new dependencies to package-lock Signed-off-by: Konstantina Blazhukova <[email protected]> * Fixes typo in package.json Signed-off-by: Konstantina Blazhukova <[email protected]> * Changes github workflow name Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds comments and docs; Takes bytecode from json Signed-off-by: Konstantina Blazhukova <[email protected]> * Fixes script name for conformity tests Signed-off-by: Konstantina Blazhukova <[email protected]> * Removes redundant variable; moves variable to const Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds specifig tag to execution apis repo Signed-off-by: Konstantina Blazhukova <[email protected]> * Replaces file name string with constant variables Signed-off-by: Konstantina Blazhukova <[email protected]> * Adds more informative variables for request.param checks Signed-off-by: Konstantina Blazhukova <[email protected]> * Combines sendrawTransaction checks Signed-off-by: Konstantina Blazhukova <[email protected]> --------- Signed-off-by: Konstantina Blazhukova <[email protected]> Signed-off-by: Mo Shaikjee <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description:
The goal of this PR is to introduce a script which runs whenever a PR is created and tests if we conform to the JSON RPC Specification
Related issue(s):
Fixes #
#1475
Notes for reviewer:
Checklist