Update test in fileio. 52/12152/2
Adeline CARNIS [Fri, 26 Jul 2013 13:23:13 +0000 (15:23 +0200)]
Change-Id: I356bef19e76c0f17b4e254483924175fce845ff6

scilab/modules/fileio/tests/nonreg_tests/bug_8836.dia.ref
scilab/modules/fileio/tests/nonreg_tests/bug_8836.tst
scilab/modules/fileio/tests/nonreg_tests/bug_9821.dia.ref
scilab/modules/fileio/tests/nonreg_tests/bug_9821.tst
scilab/modules/fileio/tests/unit_tests/getrelativefilename.dia.ref
scilab/modules/fileio/tests/unit_tests/getrelativefilename.tst

index 8546c96..ba79641 100644 (file)
@@ -14,6 +14,6 @@
 //
 // <-- Short Description -->
 // fileparts crashed scilab when matrix of strings was passed as input argument.
-//
-if execstr("fileparts(ls(""SCI/modules/fileio/macros/*.sci""))", "errcatch") <> 999 then bugmes();quit;end
-if lasterror() <> msprintf(_("%s: Wrong size for input argument #%d: A string expected.\n"), "fileparts", 1) then bugmes();quit;end
+// Following the commit: https://codereview.scilab.org/#/c/11620/, fileparts 
+// manages the matrix of strings.
+assert_checktrue(execstr("fileparts(ls(""SCI/modules/fileio/macros/*.sci""))", "errcatch") == 0);
index decd4e6..1175c48 100644 (file)
@@ -14,7 +14,7 @@
 //
 // <-- Short Description -->
 // fileparts crashed scilab when matrix of strings was passed as input argument.
-//
+// Following the commit: https://codereview.scilab.org/#/c/11620/, fileparts 
+// manages the matrix of strings.
 
-if execstr("fileparts(ls(""SCI/modules/fileio/macros/*.sci""))", "errcatch") <> 999 then pause, end
-if lasterror() <> msprintf(_("%s: Wrong size for input argument #%d: A string expected.\n"), "fileparts", 1) then pause, end
\ No newline at end of file
+assert_checktrue(execstr("fileparts(ls(""SCI/modules/fileio/macros/*.sci""))", "errcatch") == 0);
index 7a904d3..ab6145e 100644 (file)
@@ -12,4 +12,6 @@
 //
 // <-- Short Description -->
 // getrelativename does not manage matrix of strings
-assert_checkequal(getrelativefilename([SCI+'/bin',SCI+'/bin'], [SCI+'/ACKNOWLEDGMENTS',SCI+'/ACKNOWLEDGMENTS']),["../ACKNOWLEDGMENTS", "../ACKNOWLEDGMENTS"]);
+computed = getrelativefilename([SCI+'/bin',SCI+'/bin'], [SCI+'/ACKNOWLEDGMENTS',SCI+'/ACKNOWLEDGMENTS']);
+expected = pathconvert(["../ACKNOWLEDGMENTS", "../ACKNOWLEDGMENTS"], %f);
+assert_checkequal(computed,expected);
index 1e7d384..e050e23 100644 (file)
@@ -13,4 +13,6 @@
 // <-- Short Description -->
 // getrelativename does not manage matrix of strings
 
-assert_checkequal(getrelativefilename([SCI+'/bin',SCI+'/bin'], [SCI+'/ACKNOWLEDGMENTS',SCI+'/ACKNOWLEDGMENTS']),["../ACKNOWLEDGMENTS", "../ACKNOWLEDGMENTS"]);
+computed = getrelativefilename([SCI+'/bin',SCI+'/bin'], [SCI+'/ACKNOWLEDGMENTS',SCI+'/ACKNOWLEDGMENTS']);
+expected = pathconvert(["../ACKNOWLEDGMENTS", "../ACKNOWLEDGMENTS"], %f);
+assert_checkequal(computed,expected);
index 2553355..debd08c 100644 (file)
@@ -40,7 +40,7 @@ assert_checkequal(getrelativefilename([__dir12,__dir12], [__file12,__file12]), [
 // Error messages
 errmsg1 = msprintf(_("%s: Wrong number of input argument(s): %d expected.\n"), "getrelativefilename", 2);
 assert_checkerror("getrelativefilename()", errmsg1);
-errmsg2 = msprintf(_("%s: Wrong type for input argument #%d: Matrix of strings expected.\n"), "getrelativefilename", 1);
+errmsg2 = msprintf(_("%s: Wrong type for input argument #%d: A matrix of strings expected.\n"), "getrelativefilename", 1);
 assert_checkerror("getrelativefilename([],[])", errmsg2);
 errmsg3 = msprintf(_("%s: Incompatible input arguments #%d and #%d: Same size expected.\n"), "getrelativefilename", 1, 2);
 assert_checkerror("getrelativefilename([__dir1,__dir1], __file1)", errmsg3);
index a1ddcf4..7232142 100644 (file)
@@ -54,7 +54,7 @@ assert_checkequal(getrelativefilename([__dir12,__dir12], [__file12,__file12]), [
 errmsg1 = msprintf(_("%s: Wrong number of input argument(s): %d expected.\n"), "getrelativefilename", 2);
 assert_checkerror("getrelativefilename()", errmsg1);
 
-errmsg2 = msprintf(_("%s: Wrong type for input argument #%d: Matrix of strings expected.\n"), "getrelativefilename", 1);
+errmsg2 = msprintf(_("%s: Wrong type for input argument #%d: A matrix of strings expected.\n"), "getrelativefilename", 1);
 assert_checkerror("getrelativefilename([],[])", errmsg2);
 
 errmsg3 = msprintf(_("%s: Incompatible input arguments #%d and #%d: Same size expected.\n"), "getrelativefilename", 1, 2);