Opened 7 years ago
Last modified 21 months ago
#4427 new defect
Tests for globalscripts — at Version 6
Reported by: | fatherbushido | Owned by: | |
---|---|---|---|
Priority: | Should Have | Milestone: | Backlog |
Component: | Simulation | Keywords: | |
Cc: | Itms | Patch: |
Description (last modified by )
Currently the tests for the Math globalscripts are in the component/tests folder. It would be nice to move them in globalscripts/tests or something like that and so we could add tests of other globalscripts in that folder. (I want to use write some tests for global functions of #4263 but I find it ugly of putting them in simulation/tests)
Change History (6)
comment:1 by , 7 years ago
comment:4 by , 7 years ago
Milestone: | Alpha 22 → Work In Progress |
---|
Moving to the Work In Progress milestone, since there is a patch asking for feedback, but since it is not strictly bound to a specific release.
Note:
See TracTickets
for help on using tickets.
I agree, thanks for bringing the issue. I have a half-baked patch (needed for #2951) that proposes the following (which is more general that just globalscripts tests):
test_scripts.h
somewhere else in the source hierarchy (probably underscriptinterface/
)test_simulationComponents
would call the tests undersimulation/components/tests
,test_globalScripts
would call the tests underglobalscripts/tests
, etc.)Additionally, it would be nice to be able to pass a
-mod
argument totest
so this multi-purposetest_scripts.h
would load specific mods. That way the unit tests are fully moddable and we can even get rid of the warning that the public mod is absent when testing the release bundle without the data bundle. (useful for packagers)