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

[Feature] Add tests for Errors #438

Open
3 tasks
Ph0enixKM opened this issue Sep 2, 2024 · 1 comment
Open
3 tasks

[Feature] Add tests for Errors #438

Ph0enixKM opened this issue Sep 2, 2024 · 1 comment
Labels
compiler enhancement New feature or request help wanted Extra attention is needed tests

Comments

@Ph0enixKM
Copy link
Member

Ph0enixKM commented Sep 2, 2024

Is your feature request related to a problem? Please describe.
Error tests that check whether Amber shows the correct errors.

  • Write an erroring system
  • Add internationalization along the way (support for other languages)
  • Write tests for the errors

Describe the solution you'd like
A directory tests/errors that contains error tests where we have a script and corresponding error code. in a comment (default 1).

Ex.

// Error
// E705

"word" - 42

Describe alternatives you've considered
Some character based erroring like: SUB05

Additional context
N/A

@Ph0enixKM Ph0enixKM added enhancement New feature or request help wanted Extra attention is needed tests compiler labels Sep 2, 2024
@Ph0enixKM Ph0enixKM added this to the Amber 0.5.0-alpha milestone Oct 4, 2024
@Mte90
Copy link
Member

Mte90 commented Oct 15, 2024

How we want to name the errors?
I think that E when Amber crash and W when there is an alert but still compile, and for the numbers doens't matter.
For the SUB05 I think that this ticket need some examples.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compiler enhancement New feature or request help wanted Extra attention is needed tests
Projects
None yet
Development

No branches or pull requests

2 participants