-
-
Notifications
You must be signed in to change notification settings - Fork 420
Closed
Labels
completedThe issue has been fully resolved and the change will be in the next Skript update.The issue has been fully resolved and the change will be in the next Skript update.enhancementFeature request, an issue about something that could be improved, or a PR improving something.Feature request, an issue about something that could be improved, or a PR improving something.
Description
Skript/Server Version
dev/featureBug Description
When using the new SecCatchErrors which utilizes RuntimeErrorCatcher, turns out the results can be hindered from RuntimeErrorManager due to the "frame limit".
So using this new ability in tests for the test environments, will sooner or later fail because some errors are being rejected in RuntimeErrorManager because the limit for that frame has been reached, thus causing no errors to be caught in RuntimeErrorCatcher and then not updating ExprCaughtError.
Expected Behavior
RuntimeErrorManager to bypass the frame limit when the error being printed is going to be caught by a RuntimeErrorCatcher.
Steps to Reproduce
Make atleast 8 tests using SecCatchErrors with the intent to have an error thrown and caught. Then assert the ExprCaughtErrors
Errors or Screenshots
No response
Other
No response
Agreement
- I have read the guidelines above and affirm I am following them with this report.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
completedThe issue has been fully resolved and the change will be in the next Skript update.The issue has been fully resolved and the change will be in the next Skript update.enhancementFeature request, an issue about something that could be improved, or a PR improving something.Feature request, an issue about something that could be improved, or a PR improving something.