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

Tests for failure #328

Closed
Tracked by #415
SuperFola opened this issue Oct 10, 2021 · 2 comments
Closed
Tracked by #415

Tests for failure #328

SuperFola opened this issue Oct 10, 2021 · 2 comments
Assignees
@SuperFola
Labels
enhancement New feature or request 🤔 help wanted Extra attention is needed ❗️ priority/medium 🧪 tests Related to the project tests
Projects
[public] Road to v4

Comments

@SuperFola
Copy link
Member

Is your feature request related to a problem? Please describe.
We are currently testing correct behaviour, hit we should also have tests for our errors messages: do they pop up when expected? Are errors correctly reported?

Describe the solution you'd like
We will need to refactor the tests to have unit tests in ArkScript ; integration tests in C++ ; "fail tests" with a C++ launcher, ArkScript code and expected output stored in separate files. We need the C++ launcher because errors in ArkScript are unrecoverable by design.
@SuperFola SuperFola added enhancement New feature or request hacktoberfest 🤔 help wanted Extra attention is needed ❗️ priority/medium 🧪 tests Related to the project tests labels Oct 10, 2021
@SuperFola SuperFola mentioned this issue Oct 14, 2021
Merged
@SuperFola SuperFola added this to Triage needed in [public] Road to v4 Mar 9, 2022
@SuperFola SuperFola moved this from Triage needed to v.3.4.0 in [public] Road to v4 Mar 16, 2022
@SuperFola
Copy link
Member Author

#386 will be a requirement for this.

@SuperFola SuperFola mentioned this issue Sep 27, 2022
Open
SuperFola added a commit that referenced this issue Nov 1, 2022
@SuperFola
tests: added some failure tests (#328)
65bc316
SuperFola added a commit that referenced this issue Nov 6, 2022
@SuperFola
tests: added some failure tests (#328)
5e97f6c 
Former-commit-id: 3b0b7a4d7887442016a48bda03bcff920b72a967
@SuperFola SuperFola mentioned this issue Nov 16, 2022
Open
28 tasks
@SuperFola SuperFola self-assigned this Feb 26, 2023
@SuperFola
Copy link
Member Author

Done in #457

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SuperFola SuperFola

Labels
enhancement New feature or request 🤔 help wanted Extra attention is needed ❗️ priority/medium 🧪 tests Related to the project tests
Projects
[public] Road to v4
Status: v3.6.0
[public] Road to v4
v.3.4.0
Milestone
No milestone
Development

No branches or pull requests
1 participant
@SuperFola