[Octave-patch-tracker] [patch #7934] Adding tests for operator overloading in the legacy class system (bug #38128)

[Jordi Gutiérrez Hermoso]

Update of patch #7934 (project octave):

                  Status:                    None => Need Info              

    _______________________________________________________

Follow-up Comment #1:

Can you please explain these lines?


+%%        Note(JBect, 2013/01/27) : in order to ease to process of testing
+%%        Matlab compatibility, the syntax assert(observed, expected) should
+%%        be avoided. I use assert(isequal(observed, expected) instead.


We don't generally care if Octave code runs in Matlab. Tests are just Octave
code. Using ugly syntax just to please Matlab is very much not what we
generally do. Are you just doing this because Matlab doesn't have a
two-argument version of assert?

Furthermore,  I'm not sure about your decision to put these tests on stable.
It may be ok. After all, they are just tests. On the other hand, we try to be
extremely conservative about patching on stable, and new tests could
potentially break packagers' build scripts if they expect a clean-ish exit
from "make check".

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?7934>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/