diff --git a/tests/src/com/vaadin/tests/components/abstractfield/AbstractFieldTest.java b/tests/src/com/vaadin/tests/components/abstractfield/AbstractFieldTest.java index 26df5114b5..d7bcdfe1db 100644 --- a/tests/src/com/vaadin/tests/components/abstractfield/AbstractFieldTest.java +++ b/tests/src/com/vaadin/tests/components/abstractfield/AbstractFieldTest.java @@ -1,6 +1,8 @@ package com.vaadin.tests.components.abstractfield; import java.util.ArrayList; +import java.util.Collection; +import java.util.Collections; import java.util.LinkedHashMap; import java.util.List; @@ -136,8 +138,22 @@ public abstract class AbstractFieldTest extends } }; + @SuppressWarnings({ "rawtypes", "unchecked" }) public void valueChange(com.vaadin.data.Property.ValueChangeEvent event) { Object o = event.getProperty().getValue(); + if (o instanceof Collection) { + // Sort collections to avoid problems with values printed in + // different order + try { + ArrayList c = new ArrayList( + (Collection) o); + Collections.sort(c); + o = c; + } catch (Exception e) { + // continue with unsorted if sorting fails for some reason + log("Exception while sorting value: " + e.getMessage()); + } + } // Distinguish between null and 'null' String value = "null"; diff --git a/tests/src/com/vaadin/tests/components/table/TableMultiSelectMouse.html b/tests/src/com/vaadin/tests/components/table/TableMultiSelectMouse.html index a082e002fa..8d7c0d98b0 100644 --- a/tests/src/com/vaadin/tests/components/table/TableMultiSelectMouse.html +++ b/tests/src/com/vaadin/tests/components/table/TableMultiSelectMouse.html @@ -94,7 +94,7 @@ assertText vaadin=runcomvaadintestscomponentstableTables::PID_SLog/ChildComponentContainer[0]/VLabel[0] - 1. ValueChangeEvent, new value: '[Item 3, Item 4, Item 9, Item 5, Item 6, Item 7, Item 8]' + 1. ValueChangeEvent, new value: '[Item 3, Item 4, Item 5, Item 6, Item 7, Item 8, Item 9]' screenCapture @@ -144,7 +144,7 @@ assertText vaadin=runcomvaadintestscomponentstableTables::PID_SLog/ChildComponentContainer[0]/VLabel[0] - 5. ValueChangeEvent, new value: '[Item 9, Item 5, Item 6, Item 10, Item 7, Item 8]' + 5. ValueChangeEvent, new value: '[Item 10, Item 5, Item 6, Item 7, Item 8, Item 9]' screenCapture @@ -189,7 +189,7 @@ assertText vaadin=runcomvaadintestscomponentstableTables::PID_SLog/ChildComponentContainer[0]/VLabel[0] - 9. ValueChangeEvent, new value: '[Item 4, Item 9, Item 5, Item 6, Item 10, Item 7, Item 8]' + 9. ValueChangeEvent, new value: '[Item 10, Item 4, Item 5, Item 6, Item 7, Item 8, Item 9]' @@ -259,7 +259,7 @@ assertText vaadin=runcomvaadintestscomponentstableTables::PID_SLog/ChildComponentContainer[0]/VLabel[0] - 1. ValueChangeEvent, new value: '[Item 3, Item 4, Item 9, Item 5, Item 6, Item 7, Item 8]' + 1. ValueChangeEvent, new value: '[Item 3, Item 4, Item 5, Item 6, Item 7, Item 8, Item 9]' screenCapture @@ -309,7 +309,7 @@ assertText vaadin=runcomvaadintestscomponentstableTables::PID_SLog/ChildComponentContainer[0]/VLabel[0] - 5. ValueChangeEvent, new value: '[Item 9, Item 5, Item 6, Item 10, Item 7, Item 8]' + 5. ValueChangeEvent, new value: '[Item 10, Item 5, Item 6, Item 7, Item 8, Item 9]' screenCapture @@ -354,7 +354,7 @@ assertText vaadin=runcomvaadintestscomponentstableTables::PID_SLog/ChildComponentContainer[0]/VLabel[0] - 9. ValueChangeEvent, new value: '[Item 4, Item 9, Item 5, Item 6, Item 10, Item 7, Item 8]' + 9. ValueChangeEvent, new value: '[Item 10, Item 4, Item 5, Item 6, Item 7, Item 8, Item 9]'