From 7305474e4a666d8b986d1d95e7a89fed64277935 Mon Sep 17 00:00:00 2001 From: Denis Anisimov Date: Sat, 23 Aug 2014 13:32:06 +0300 Subject: Make table selectable based on presence of ValueChangeListener (#13864). Change-Id: I272703f1e3178c91a2b1e3e4d0f7c79e4c86e552 --- .../server/component/table/TableSelectable.java | 75 ++++++++++++++++++++++ 1 file changed, 75 insertions(+) create mode 100644 server/tests/src/com/vaadin/tests/server/component/table/TableSelectable.java (limited to 'server/tests/src') diff --git a/server/tests/src/com/vaadin/tests/server/component/table/TableSelectable.java b/server/tests/src/com/vaadin/tests/server/component/table/TableSelectable.java new file mode 100644 index 0000000000..1af99a08eb --- /dev/null +++ b/server/tests/src/com/vaadin/tests/server/component/table/TableSelectable.java @@ -0,0 +1,75 @@ +/* + * Copyright 2000-2014 Vaadin Ltd. + * + * Licensed under the Apache License, Version 2.0 (the "License"); you may not + * use this file except in compliance with the License. You may obtain a copy of + * the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT + * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the + * License for the specific language governing permissions and limitations under + * the License. + */ +package com.vaadin.tests.server.component.table; + +import org.easymock.EasyMock; +import org.junit.Assert; +import org.junit.Test; + +import com.vaadin.data.Property.ValueChangeListener; +import com.vaadin.ui.Table; + +/** + * Tests for 'selectable' property of {@link Table} class. + * + * @author Vaadin Ltd + */ +public class TableSelectable { + + @Test + public void setSelectable_explicitSelectable_tableIsSelectable() { + Table table = new Table(); + table.setSelectable(true); + + Assert.assertTrue(table.isSelectable()); + } + + @Test + public void addValueChangeListener_explicitSelectable_tableIsSelectable() { + TestTable table = new TestTable(); + table.addValueChangeListener(EasyMock + .createMock(ValueChangeListener.class)); + + Assert.assertTrue(table.isSelectable()); + Assert.assertTrue(table.markAsDirtyCalled); + } + + @Test + public void tableIsNotSelectableByDefult() { + Table table = new Table(); + + Assert.assertFalse(table.isSelectable()); + } + + @Test + public void setSelectable_explicitNotSelectable_tableIsNotSelectable() { + Table table = new Table(); + table.setSelectable(false); + table.addValueChangeListener(EasyMock + .createMock(ValueChangeListener.class)); + + Assert.assertFalse(table.isSelectable()); + } + + private static final class TestTable extends Table { + @Override + public void markAsDirty() { + markAsDirtyCalled = true; + } + + private boolean markAsDirtyCalled; + } +} -- cgit v1.2.3 From 4ad7cefcc06e372df8b67a2456616d7410c768d8 Mon Sep 17 00:00:00 2001 From: wodencafe Date: Wed, 28 Jan 2015 09:42:18 -0600 Subject: Fixed UI.getPushConfiguration().getTransport() always returning null (#16499) Change-Id: I31a923faf5ae369ffc2160cdeb12584ca0babc9d --- server/src/com/vaadin/ui/PushConfiguration.java | 2 +- .../vaadin/ui/PushConfigurationTransportTest.java | 46 ++++++++++++++++++++++ shared/src/com/vaadin/shared/ui/ui/Transport.java | 8 ++++ 3 files changed, 55 insertions(+), 1 deletion(-) create mode 100644 server/tests/src/com/vaadin/ui/PushConfigurationTransportTest.java (limited to 'server/tests/src') diff --git a/server/src/com/vaadin/ui/PushConfiguration.java b/server/src/com/vaadin/ui/PushConfiguration.java index 84f59d0313..90ad28542c 100644 --- a/server/src/com/vaadin/ui/PushConfiguration.java +++ b/server/src/com/vaadin/ui/PushConfiguration.java @@ -208,7 +208,7 @@ class PushConfigurationImpl implements PushConfiguration { public Transport getTransport() { try { return Transport - .valueOf(getParameter(PushConfigurationState.TRANSPORT_PARAM)); + .getByIdentifier(getParameter(PushConfigurationState.TRANSPORT_PARAM)); } catch (IllegalArgumentException e) { return null; } diff --git a/server/tests/src/com/vaadin/ui/PushConfigurationTransportTest.java b/server/tests/src/com/vaadin/ui/PushConfigurationTransportTest.java new file mode 100644 index 0000000000..305b2e06cd --- /dev/null +++ b/server/tests/src/com/vaadin/ui/PushConfigurationTransportTest.java @@ -0,0 +1,46 @@ +/* + * Copyright 2000-2014 Vaadin Ltd. + * + * Licensed under the Apache License, Version 2.0 (the "License"); you may not + * use this file except in compliance with the License. You may obtain a copy of + * the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT + * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the + * License for the specific language governing permissions and limitations under + * the License. + */ +package com.vaadin.ui; + +import org.junit.Assert; +import org.junit.Test; + +import com.vaadin.server.VaadinRequest; +import com.vaadin.shared.ui.ui.Transport; + +/** + * @author Vaadin Ltd + */ +public class PushConfigurationTransportTest { + @Test + public void testTransportModes() throws Exception { + UI ui = new UI() { + + @Override + protected void init(VaadinRequest request) { + // TODO Auto-generated method stub + + } + + }; + for (Transport transport : Transport.values()) { + ui.getPushConfiguration().setTransport(transport); + Assert.assertEquals(ui.getPushConfiguration().getTransport(), + transport); + } + + } +} diff --git a/shared/src/com/vaadin/shared/ui/ui/Transport.java b/shared/src/com/vaadin/shared/ui/ui/Transport.java index 6eafba185e..39174caddf 100644 --- a/shared/src/com/vaadin/shared/ui/ui/Transport.java +++ b/shared/src/com/vaadin/shared/ui/ui/Transport.java @@ -46,4 +46,12 @@ public enum Transport { return identifier; } + public static Transport getByIdentifier(String identifier) { + for (Transport t : values()) { + if (t.getIdentifier().equals(identifier)) { + return t; + } + } + return null; + } } -- cgit v1.2.3