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

Refactor all tests #171

Merged
merged 3 commits into from
Feb 7, 2024
Merged

Refactor all tests #171

merged 3 commits into from
Feb 7, 2024

Conversation

vfusco
Copy link
Contributor

@vfusco vfusco commented Jan 10, 2024

Move all tests to tests directory.

@vfusco vfusco self-assigned this Jan 10, 2024
@vfusco vfusco force-pushed the feature/move-tests branch 14 times, most recently from 551b495 to c886da7 Compare January 24, 2024 14:04
@vfusco vfusco added the enhancement New feature or request label Jan 24, 2024
@vfusco vfusco added this to the v0.16.0 milestone Jan 24, 2024
@vfusco vfusco force-pushed the feature/move-tests branch 6 times, most recently from 1a2b2a9 to 12f26d7 Compare January 30, 2024 05:36
@vfusco vfusco marked this pull request as ready for review January 30, 2024 12:27
Makefile Show resolved Hide resolved
Makefile Show resolved Hide resolved
Makefile Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
tools/template/cartesi-machine.template Show resolved Hide resolved
tests/machine/CONTRIBUTING.md Outdated Show resolved Hide resolved
tests/Makefile Show resolved Hide resolved
tests/Makefile Outdated Show resolved Hide resolved
tests/scripts/test-jsonrpc-server.sh Show resolved Hide resolved
tests/Makefile Outdated Show resolved Hide resolved
@vfusco vfusco force-pushed the feature/move-tests branch 6 times, most recently from 30e29fa to cfb066d Compare February 6, 2024 14:09
@edubart edubart linked an issue Feb 6, 2024 that may be closed by this pull request
@vfusco vfusco force-pushed the feature/move-tests branch 2 times, most recently from 70bb8e4 to 10992e8 Compare February 6, 2024 17:10
mpolitzer
mpolitzer previously approved these changes Feb 6, 2024
Copy link
Contributor

@mpolitzer mpolitzer left a comment

Choose a reason for hiding this comment

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

LGTM

edubart
edubart previously approved these changes Feb 6, 2024
@vfusco vfusco merged commit 40ea8e5 into main Feb 7, 2024
16 checks passed
@vfusco vfusco deleted the feature/move-tests branch February 7, 2024 00:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Move RISC-V machine tests into emulator repository
3 participants