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.

Sign up for GitHub

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

Jump to bottom

Updating contract tests after IR. Also, some cleanup. #1141

Merged
merged 4 commits into from
Apr 5, 2022
Merged

Updating contract tests after IR. Also, some cleanup. #1141

merged 4 commits into from
Apr 5, 2022

Conversation

mohammadfawaz
Copy link
Contributor

  • Updating contract IDs where needed
  • Enabling storage_accesses tests.
  • Disabling one broken test (contract_abi_impl).
  • Disabling broken parts of one test (context_testing_contract).
  • Some cleanup.

@mohammadfawaz mohammadfawaz self-assigned this Apr 4, 2022
@mohammadfawaz mohammadfawaz added the testing General testing label Apr 4, 2022
@adlerjohn
Copy link
Contributor

Updating contract IDs where needed

We really need script parameters 😂

otrho
otrho previously approved these changes Apr 4, 2022
View reviewed changes
@mohammadfawaz mohammadfawaz merged commit 56b3eb9 into master Apr 5, 2022
@mohammadfawaz mohammadfawaz deleted the mohammadfawaz/updating_contract_tests_after_ir branch April 5, 2022 01:53
@mohammadfawaz mohammadfawaz mentioned this pull request Apr 5, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@otrho otrho otrho approved these changes

@sezna sezna Awaiting requested review from sezna

@emilyaherbert emilyaherbert Awaiting requested review from emilyaherbert

Assignees

@mohammadfawaz mohammadfawaz

Labels
testing General testing
Projects
Fuel Network
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mohammadfawaz @adlerjohn @otrho