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

Dvm test case #428

Merged
merged 9 commits into from
Jan 7, 2021
Merged

Dvm test case #428

merged 9 commits into from
Jan 7, 2021

Conversation

hujw77
Copy link
Contributor

@hujw77 hujw77 commented Jan 4, 2021

Add Test Case

  • test contract log and anonymous log
  • test solidity opcodes
  • test net rpc (partial skip)
  • test filter rpc (skip)
  • test debug rpc (rpc not implement)

@boundless-forest
Copy link
Member

The RPC test seems to run forever ....

@hujw77
Copy link
Contributor Author

hujw77 commented Jan 4, 2021

The RPC test seems to run forever ....

It caused by ws connection.
I close it after test.

Copy link
Member

@boundless-forest boundless-forest left a comment

Choose a reason for hiding this comment

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

LGTM!

@aurexav
Copy link
Member

aurexav commented Jan 5, 2021

Is this ready for merge?

@boundless-forest
Copy link
Member

Is this ready for merge?

LGTM. can be merged.

@boundless-forest
Copy link
Member

@AurevoirXavier @hackfisher Please check this pr again. The changes looks good to me and i do not have merge authority for this repo.

@hackfisher hackfisher merged commit 3c7eaa9 into master Jan 7, 2021
@hackfisher hackfisher deleted the dvm-test branch January 7, 2021 03:47
@hackfisher hackfisher mentioned this pull request Jan 7, 2021
@hujw77 hujw77 mentioned this pull request Mar 16, 2021
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.

4 participants