Unify all testing Airs

Open Beginner friendly
#456 0 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
65/100
Issue type
Refactor
Clarity
Clearly specified
Activity status
Stale
Tech stack
rust
Domain
testing, tooling

Research direction

The issue lists four directories containing .air files. Start by examining each directory to understand the file structure and identify duplicates. The goal is to consolidate files from codegen/ace/src/tests/airs, parser/src/parser/tests/input, and possibly docs/examples into air-script/tests. Check for any existing test suites that depend on these file paths to ensure the consolidation doesn't break tests. A good first step is to run the existing tests for the parser and codegen components to verify they pass before moving files.

Written by the indexing model from the issue text.

Description

We currently have several places where we store example or testing .air files for different use cases.

Except perhaps for the examples in docs/examples, it seems like we could put all other testing source files in air-script/tests. While taking into account likely duplicates, it we could increase the number of test cases for the 3 components that rely on these (winterfell codegen, ace codegen, and parser).

Dominant language
Rust
Stars
97
Forks
39
No merged PRs in 30d

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from 0xMiden/air-script

All issues in 0xMiden/air-script

Similar issues

More Rust issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.