Amber icon indicating copy to clipboard operation
Amber copied to clipboard

[Feature] Add tests for Errors

Open Ph0enixKM opened this issue 1 year ago • 5 comments

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

  • [x] Write an error testing system
  • [ ] Write tests for the errors

Ph0enixKM avatar Sep 02 '24 09:09 Ph0enixKM

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.

Mte90 avatar Oct 15 '24 13:10 Mte90

We have now the feature and want the error output description. We have to implement them (https://github.com/amber-lang/amber/issues/605).

My suggestion is to open all the .rs files and look for exception and implement tests for the cases found.

Mte90 avatar Dec 23 '24 14:12 Mte90

I'll update the PR description. We will just match the error message content.

Ph0enixKM avatar Dec 23 '24 15:12 Ph0enixKM

We just need to write the tests for the errors

Ph0enixKM avatar Dec 23 '24 15:12 Ph0enixKM

https://github.com/search?q=repo%3Aamber-lang%2Famber+Message%3A%3Anew&type=code from here it is possible to find all the reference in the code where a warning message is showed so it is more simple to add the missing tests.

I don't know if there are also other errors in heraclitus that we don't track in this repository.

Mte90 avatar Dec 24 '24 10:12 Mte90