Bug #14446 fixed - Corrected the number of argument in the error message generated... 46/17946/6
rishubhjain [Mon, 21 Mar 2016 14:44:38 +0000 (07:44 -0700)]
Change-Id: Ibc897193ca466c5686d93246eecb614b38c6bcff

scilab/CHANGES
scilab/modules/hdf5/sci_gateway/cpp/sci_hdf5_save.cpp
scilab/modules/hdf5/tests/nonreg_tests/bug_14446.dia.ref [new file with mode: 0644]
scilab/modules/hdf5/tests/nonreg_tests/bug_14446.tst [new file with mode: 0644]

index 18b3fb1..7435cfd 100644 (file)
@@ -373,6 +373,8 @@ In 6.0.0:
 
 * Bug #14434 fixed - PlotSparse did not work anymore.
 
+* Bug #14446 fixed - Corrected the number of argument in the error message generated by save() on passing wrong type of argument.
+
 * Bug #14468 fixed - Scinotes was unable to export to HTML.
 
 In 6.0.0 beta-1:
index 39f67ad..78e002d 100644 (file)
@@ -138,7 +138,7 @@ types::Function::ReturnValue sci_hdf5_save(types::typed_list &in, int _iRetCount
         {
             if (in[i]->getId() != types::InternalType::IdScalarString)
             {
-                Scierror(999, _("%s: Wrong type for input argument #%d: A String expected.\n"), fname.data(), 1);
+                Scierror(999, _("%s: Wrong type for input argument #%d: A String expected.\n"), fname.data(), i+1);
                 return types::Function::Error;
             }
 
diff --git a/scilab/modules/hdf5/tests/nonreg_tests/bug_14446.dia.ref b/scilab/modules/hdf5/tests/nonreg_tests/bug_14446.dia.ref
new file mode 100644 (file)
index 0000000..0966a44
--- /dev/null
@@ -0,0 +1,34 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2016 - Rishubh Jain
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+//
+// <-- Non-regression test for bug 14446 -->
+// <-- ENGLISH IMPOSED -->
+// <-- CLI SHELL MODE -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/14446
+//
+// <-- Short Description -->
+//
+//Wrong number of argument in error message when wrong type of argument is passed to save().
+function f()
+ save("TMPDIR/a.sod",1);
+endfunction
+function f1()
+ num1=1;
+ num2=1;
+ save("TMPDIR/b.sod","num1",num2);
+endfunction
+function f2()
+ num1=1;
+ num2=1;
+ num3=1;
+ save("TMPDIR/b.sod","num1","num2",num3);
+endfunction
+assert_checkerror("f()","save: Wrong type for input argument #2: A String expected.");
+assert_checkerror("f1()","save: Wrong type for input argument #3: A String expected.");
+assert_checkerror("f2()","save: Wrong type for input argument #4: A String expected.");
\ No newline at end of file
diff --git a/scilab/modules/hdf5/tests/nonreg_tests/bug_14446.tst b/scilab/modules/hdf5/tests/nonreg_tests/bug_14446.tst
new file mode 100644 (file)
index 0000000..b6a5e03
--- /dev/null
@@ -0,0 +1,38 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2016 - Rishubh Jain
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+//
+// <-- Non-regression test for bug 14446 -->
+// <-- ENGLISH IMPOSED -->
+// <-- CLI SHELL MODE -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/14446
+//
+// <-- Short Description -->
+//
+//Wrong number of argument in error message when wrong type of argument is passed to save().
+
+function f()
+ save("TMPDIR/a.sod",1);
+endfunction
+
+function f1()
+ num1=1;
+ num2=1;
+ save("TMPDIR/b.sod","num1",num2);
+endfunction
+
+function f2()
+ num1=1;
+ num2=1;
+ num3=1;
+ save("TMPDIR/b.sod","num1","num2",num3);
+endfunction
+
+assert_checkerror("f()","save: Wrong type for input argument #2: A String expected.");
+assert_checkerror("f1()","save: Wrong type for input argument #3: A String expected.");
+assert_checkerror("f2()","save: Wrong type for input argument #4: A String expected.");
\ No newline at end of file