[Octave-bug-tracker] [bug #61393] [octave forge] (sparsersb) Crash while "pkg test sparsersb"

[Markus Mützel]

Follow-up Comment #11, bug #61393 (project octave):

The patch didn't apply cleanly on the tarball. And you might have missed
replacing the literal file name with the new variable in the last of the
affected tests. So, I changed that part to:

 %!test
-%! [S, NROWS] = sparsersb("sparsersb_temporary_matrix_file.mtx", "D");
+%! sparsersb_temporary_matrix_file =
fullfile(tempdir(),"sparsersb_temporary_matrix_file.mtx");
+%! [S, NROWS] = sparsersb(sparsersb_temporary_matrix_file, "D");
 %! assert(NROWS==2);


With that in place, `pkg test sparsersb` only fails the following two tests
when installed in a read-only location:

>>>>> processing C:\Program Files\GNU
Octave\Octave-7.2.1\mingw64\lib\octave\packages\sparsersb-1.0.9\x86_64-w64-mingw32-api-v57\sparsersb.cc-tst
***** test
 rrm=sparsersb(sprand(1000,1000,0.001));
 sparsersb(rrm,"render", "sparsersb_temporary_render.eps" ,1024); # will use
defaults for rWidth
!!!!! test failed
librsb error: The user supplied corrupt or inconsistent data as argument.
***** test
 rrm=sparsersb(sprand(1000,1000,0.001));
 sparsersb(rrm,"render", "sparsersb_temporary_render.eps" ,1024,1024);
 # sparsersb(rrm,"renderb", "sparsersb_temporary_renderb.eps");
sparsersb(rrm,"renders", "sparsersb_temporary_renders.eps"); # FIXME
!!!!! test failed
librsb error: The user supplied corrupt or inconsistent data as argument.


It looks like these tests also involve temporary files. Maybe, those should
also move to the TEMP directory?

That's still an improvement from the 9 tests failing before. So, I pushed that
patch to MXE Octave here:
https://hg.octave.org/mxe-octave/rev/9b5b81c294f9



    _______________________________________________________

Reply to this item at:

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

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