-
Notifications
You must be signed in to change notification settings - Fork 36
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
TestState fails with assertion error on state.phase #39
Comments
An equivalent problem occurs in function |
Btw, I don't know if runtests.mos is actually used, but it's missing the tests TestMedium.TestConstants, TestMedium.TestState, TestMedium.TestSat. |
A quick re-check with OM 1.18.1 and the new ExternalMedia 3.3.1 shows that this still occurs. |
Probably related to OpenModelica/OpenModelica#8591. |
I checked with a recent OM nightly (1.21.0-dev-318) and current master (db55653) with binaries from the CI job, and the assertion does not print anymore. The compilation warning still occurs:
The runtests.mos files seems heavily outdated/old, but that's probably easy to fix/not that relevant anymore. |
This should be fixed with ExternalMedia 3.3.2, if used with OMC 1.21.0 or later |
With ExternalMedia 3.3.0 and OM nightly (i.e. >1.17), running ExternalMedia.Test.TestMedium.TestState fails with
Note that the printed value actually complies with the required assertion range! Maybe the passed value is a string or somesuch type that fails the assertion but looks identical when printed?
I suspect that this compilation warning (also reported in #10) could be connected:
The text was updated successfully, but these errors were encountered: