Browse Source

Fix DataCommunicator parameter not used in Grid constructor (#10075)

Fixes #9944
tags/8.2.0.alpha2
JenoDK 6 years ago
parent
commit
9776ea2e85

+ 1
- 1
server/src/main/java/com/vaadin/ui/Grid.java View File

* @since 8.0.7 * @since 8.0.7
*/ */
protected Grid(Class<T> beanType, DataCommunicator<T> dataCommunicator) { protected Grid(Class<T> beanType, DataCommunicator<T> dataCommunicator) {
this(BeanPropertySet.get(beanType));
this(BeanPropertySet.get(beanType), dataCommunicator);
this.beanType = beanType; this.beanType = beanType;
} }



+ 10
- 0
server/src/test/java/com/vaadin/tests/server/component/grid/GridTest.java View File

import java.util.stream.Collectors; import java.util.stream.Collectors;
import java.util.stream.Stream; import java.util.stream.Stream;


import com.vaadin.data.provider.DataCommunicator;
import org.easymock.Capture; import org.easymock.Capture;
import org.junit.Assert; import org.junit.Assert;
import org.junit.Before; import org.junit.Before;
.setId("randomColumnId"); .setId("randomColumnId");
} }


@Test
public void testCreateGridWithDataCommunicator() {
DataCommunicator specificDataCommunicator = new DataCommunicator<>();

TestGrid<String> grid = new TestGrid(String.class, specificDataCommunicator);

assertEquals(specificDataCommunicator, grid.getDataCommunicator());
}

@Test @Test
public void testGridHeightModeChange() { public void testGridHeightModeChange() {
assertEquals("Initial height mode was not CSS", HeightMode.CSS, assertEquals("Initial height mode was not CSS", HeightMode.CSS,

+ 17
- 0
server/src/test/java/com/vaadin/tests/server/component/grid/TestGrid.java View File

package com.vaadin.tests.server.component.grid;

import com.vaadin.data.provider.DataCommunicator;
import com.vaadin.ui.Grid;

/**
* {@link Grid} class for testing purposes
* @param <T>
* the grid bean type
*/
public class TestGrid<T> extends Grid<T> {

public TestGrid(Class<T> beanType, DataCommunicator<T> dataCommunicator) {
super(beanType, dataCommunicator);
}

}

Loading…
Cancel
Save