Skip to content

Comments

(A001) Simplify workflow for unittests#411

Open
syntron wants to merge 2 commits intoOpenModelica:masterfrom
syntron:simplify_workflow_Test
Open

(A001) Simplify workflow for unittests#411
syntron wants to merge 2 commits intoOpenModelica:masterfrom
syntron:simplify_workflow_Test

Conversation

@syntron
Copy link
Contributor

@syntron syntron commented Feb 18, 2026

simplify main test workflow - do not run test_FMIRegression

  • these are test for OpenModelica / no specific check for OMPython
  • they are run via a cron job
  • it increases the runtime of the unittest job

furthermore add a test workflow which runs only the latest version (unittests in ./tests) - needed later if data for v400 is defined

see PR #404

* these are test for OpenModelica / no specific check for OMPython
* they are run via a cron job
* it increases the runtime of the unittest job
@adeas31
Copy link
Member

adeas31 commented Feb 19, 2026

simplify main test workflow - do not run test_FMIRegression

  • these are test for OpenModelica / no specific check for OMPython
  • they are run via a cron job
  • it increases the runtime of the unittest job

I agree these are more OpenModelica tests. I suggest we remove this test and its workflow completely. OMPython has no benefit having it.

furthermore add a test workflow which runs only the latest version (unittests in ./tests) -

Why is this workflow needed? Perhaps

needed later if data for v400 is defined

Better to add it when its required. Here its inclusion is not clear.

@syntron
Copy link
Contributor Author

syntron commented Feb 19, 2026

I agree these are more OpenModelica tests. I suggest we remove this test and its workflow completely. OMPython has no benefit having it.

OK; remove it completly (also the definiton using a cron job) or just deactivate the automated run such that it can be executed if needed?

Better to add it when its required. Here its inclusion is not clear.

OK; will remove it here. The idea was to be able to run both options v4xx and v400 separatly ...

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants