Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(execution-engine): VM execution step negative tests [fixes VM-251] #504

Closed
wants to merge 1 commit into from

Conversation

raftedproc
Copy link
Contributor

No description provided.

@linear
Copy link

linear bot commented Mar 10, 2023

@raftedproc raftedproc self-assigned this Mar 10, 2023
@raftedproc raftedproc added A-execution-engine area: execution engine of AquaVM A-tests area: tests labels Mar 10, 2023
@raftedproc raftedproc marked this pull request as ready for review March 10, 2023 10:04
@coveralls
Copy link

coveralls commented Mar 10, 2023

Pull Request Test Coverage Report for Build 4383402664

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.3%) to 71.711%

Totals Coverage Status
Change from base Build 4383270508: 0.3%
Covered Lines: 13694
Relevant Lines: 19096

💛 - Coveralls

}

#[test]
fn nonstring_in_triplet_for_call() {
Copy link
Member

Choose a reason for hiding this comment

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

Let's add 6 more test cases where all of these put into other 2 positions in a call triplet.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

}

#[test]
fn ap_scalar_variable_not_found_error() {
Copy link
Member

Choose a reason for hiding this comment

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

Suggest naming tests in this file following this pattern
<error_name>_<specific>, so this one would be variable_not_found_ap_scalar, and without error.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@raftedproc raftedproc force-pushed the chore/VM-251-catchable-negative-tests branch from 1b708e8 to fda6559 Compare March 13, 2023 12:34
@raftedproc raftedproc closed this Mar 13, 2023
@raftedproc raftedproc deleted the chore/VM-251-catchable-negative-tests branch March 13, 2023 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-execution-engine area: execution engine of AquaVM A-tests area: tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants