[Octave-bug-tracker] [bug #63229] Missing self test for option OutputSel in ode23, ode23s, ode45

[Nicholas Jankowski]

Follow-up Comment #4, bug #63229 (project octave):

yes, I was looking at it recently. I understand the idea that you're adding a
common function to get used in multiple tests.  I'm not certain that having
the function itself call the assert statements is the right way to go.  I've
not seen it done that way before. I was going to try to force some test errors
to see how it behaved when calling e.g., 'test ode23'. Generally, I would
think it best for the function to return a value that the calling test line
will check with an assert statement.  

But maybe someone else has an opinion on whether either form is preferred or
problematic?


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?63229>

_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/