Xcos: fix xcosPal related tests 11/18111/6
Clément DAVID [Thu, 21 Apr 2016 15:16:54 +0000 (17:16 +0200)]
Change-Id: I05b75311f538676c22c9ccd7c356ad935784bef0

scilab/modules/xcos/macros/xcosPal.sci
scilab/modules/xcos/src/java/org/scilab/modules/xcos/palette/PreLoadedElement.java
scilab/modules/xcos/tests/nonreg_tests/bug_11554.dia.ref
scilab/modules/xcos/tests/nonreg_tests/bug_11821.dia.ref
scilab/modules/xcos/tests/nonreg_tests/bug_11821.tst
scilab/modules/xcos/tests/unit_tests/SEP_45_xcos_palette.dia.ref
scilab/modules/xcos/tests/unit_tests/xcosPal.dia.ref

index 238ada4..75097a7 100644 (file)
@@ -61,7 +61,7 @@ function pal = xcosPal(name, scs_m)
         error(msprintf(gettext("%s: Wrong number of input arguments: %d to %d expected.\n"), "xcosPal", 0, 2));
     end
 
-    if exists("name", "l") == 0 then
+    if ~isdef("name", "l") then
         name = "New palette";
     elseif typeof(name) == "diagram" then
         scs_m = name;
@@ -95,7 +95,7 @@ function pal = xcosPal(name, scs_m)
     for block = scs_m.objs
         if typeof(block)=="Block" & block.gui == "PAL_f" then
             // customize palette name
-            name = block.model.rpar.props.title(1);
+            name = scs_m.props.title(1);
 
             // Add PAL_f children blocks
             children = block.model.rpar.objs;
index 796ef5c..513ab12 100644 (file)
@@ -132,7 +132,9 @@ public class PreLoadedElement extends AbstractElement<PreLoaded> {
                      */
                     final StringBuilder str = new StringBuilder();
                     str.append(ScilabInterpreterManagement.buildCall(ScilabDirectHandler.BLK + "=" + blockNames[i][j], "define"));
-                    str.append(ScilabInterpreterManagement.buildCall("xcosPalGenerateIcon", icons[i][j]));
+                    str.append("; ");
+                    str.append(ScilabInterpreterManagement.buildCall("xcosPalGenerateIcon", ScilabDirectHandler.BLK.toCharArray(), icons[i][j]));
+                    str.append("; ");
                     ScilabInterpreterManagement.putCommandInScilabQueue(str.toString());
                 }
                 iconPath.setPath(icons[i][j]);
index b3415e5..6b34664 100644 (file)
@@ -33,4 +33,3 @@ pal = xcosPal("My palette");
 pal = xcosPalAddBlock(pal, 'MY_BLOCK', pal_block_img, block_img);
 xcosPalAdd(pal);
 quit();
-exec('SCI/etc/scilab.quit',-1);quit;
index 27205cb..e0e86bc 100644 (file)
@@ -16,8 +16,5 @@
 pal = xcosPal();
 pal = xcosPalAddBlock(pal, "SUM_f");
 pal = xcosPalAddBlock(pal, "BIGSOM_f");
-status = xcosPalAdd(pal, "my Summation blocks")
- status  =
-  T  
+status = xcosPalAdd(pal, "my Summation blocks");
 assert_checktrue (status);
index fc9b946..038f17e 100644 (file)
 // http://bugzilla.scilab.org/show_bug.cgi?id=11821
 //
 // <-- Short Description -->
-// Running XcosPalAdd example from help led to deadlock  
+// Running XcosPalAdd example from help led to deadlock
 
 
 pal = xcosPal();
 pal = xcosPalAddBlock(pal, "SUM_f");
 pal = xcosPalAddBlock(pal, "BIGSOM_f");
 
-status = xcosPalAdd(pal, "my Summation blocks")
+status = xcosPalAdd(pal, "my Summation blocks");
 assert_checktrue (status);
 
index 17dfb0c..cb9f93e 100644 (file)
@@ -21,6 +21,14 @@ end
 // Import a Scicos palette
 exec(SCI + "/modules/scicos/palettes/Branching.cosf", -1);
 pal = xcosPal(scs_m);
+Xcos error: Wrong value for field graphics.pin: FIXME port has not been updated.
+Xcos error: Wrong value for field graphics.pin: FIXME port has not been updated.
+Xcos error: Wrong value for field graphics.pout: FIXME port has not been updated.
+Xcos error: Wrong value for field graphics.pein: FIXME port has not been updated.
+Xcos error: Wrong value for field graphics.pin: FIXME port has not been updated.
+Xcos error: Wrong value for field graphics.pein: FIXME port has not been updated.
+Xcos error: Wrong value for field graphics.pout: FIXME port has not been updated.
+Xcos error: Wrong value for field graphics.pein: FIXME port has not been updated.
 xcosPalExport(pal, exportedFile);
 xcosPalAdd(exportedFile);
 // unit test for a custom simple block
index 3e69deb..1eac06d 100644 (file)
@@ -7,7 +7,7 @@
 // <-- Short Description -->
 // White-box test for the xcosPal macro.
 fields=["name";"blockNames";"icons";"style"]
-fields  = 
+ fields  = 
 !name        !
 !            !
 !blockNames  !