FuelLabs/sway

Need an E2E test that calls a contract from another contract

Open

#1140 opened on Apr 4, 2022

View on GitHub
 (0 comments) (0 reactions) (0 assignees)Rust (61,702 stars) (5,421 forks)batch import
P: highgood first issuetesting

Description

Specifically, a replacement for: https://github.com/FuelLabs/sway/tree/master/test/src/e2e_vm_tests/test_programs/should_pass/test_contracts/contract_abi_impl and its corresponding script: https://github.com/FuelLabs/sway/tree/master/test/src/e2e_vm_tests/test_programs/should_pass/require_contract_deployment/contract_call

The test is currently broken since the other contract it tries to call does not exist. I'm removing it shortly.

Contributor guide

Tech stack
rust
Domain
blockchaintesting
Issue type
test
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Sway contract developmentFuel E2E test setup
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
40
Research direction
Examine the referenced test directories in the issue: test/src/e2e vm tests/test programs/should pass/test contracts/contract abi impl and test/src/e2e vm tests/test programs/should pass/require contract deployment/contract call. Understand how the original test was structured and why it's broken (the dependency contract is missing). Study existing E2E tests in the repository to follow the pattern for writing a contract to contract call test. The new test should replace the old one and ensure the contract call works correctly.