Opened 6 years ago

Last modified 6 years ago

#1063 new enhancement

Need unit tests for combined models P(Q)S(Q) etc

Reported by: awashington Owned by: awashington
Priority: critical Milestone: SasView Next Release +1
Component: sasmodels Keywords: tests
Cc: Work Package: Beta Approximation Project

Description

The combined model code currently doesn't appear to have any unit tests. We should add some tests in to prevent future regressions.

Change History (4)

comment:1 Changed 6 years ago by butler

  • Work Package changed from SasView Bug Fixing to Beta Approximation Project

assuming this is part of the combine models that needs to be worked on for the Beta so moving to that work package. Can be moved back (currently SasView Bug Fixing) if it is not appropriate here

comment:2 Changed 6 years ago by richardh

Perhaps we could "save analysis" and save I(Q) for a P(Q)S(Q). Then for the test, load analysis, compute and compare a load of the previously saved I(Q).

Any other ideas?

Paul K says this ought to be done in sasmodels, this would require new code to run integration tests in the source tree, so they would run on sasmodels being installed. See also #822

Last edited 6 years ago by richardh (previous) (diff)

comment:3 Changed 6 years ago by richardh

  • Milestone changed from SasView 4.3.0 to SasView Next Release +1
  • Priority changed from minor to critical
  • Summary changed from Need unit tests for combined models to Need unit tests for combined models P(Q)S(Q) etc
  • Type changed from defect to enhancement

comment:4 Changed 6 years ago by richardh

Though this ticket is really about testing P(Q)S(Q) from sasview, as an example of what can go wrong there were initially issues during the Sept 2018 code camp with the calling of multiplicity models from the new qt5 gui.
(a) the number of shells was not being passed properly, (b) there were issues with the initially loaded default parameters continuing to be being sent to sasmodels from the gui instead of the parameters input by the user. Both of these issues were of course resolved, however note that in both cases the models themselves loaded and passed their "unit tests" correctly! Thus I think that we really should have some functionality testing of the whole sasview gui when dealing with more complex models or P(Q)S(Q).

Note: See TracTickets for help on using tickets.