diff --git a/openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ui/data/DataSetReportGenerator.java b/openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ui/data/DataSetReportGenerator.java
index 58163d1a78f781a25c9221e6694b9342cf8376de..7cb93027935c733728c53912666166ae11a78de1 100644
--- a/openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ui/data/DataSetReportGenerator.java
+++ b/openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ui/data/DataSetReportGenerator.java
@@ -90,7 +90,7 @@ public class DataSetReportGenerator
public String getId()
{
- final String reportKey = service.getKey(); // TODO escape?
+ final String reportKey = service.getKey();
return DataSetReporterGrid.createId(reportKey);
}
diff --git a/openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ui/experiment/ProjectSelectionTreeGridContainer.java b/openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ui/experiment/ProjectSelectionTreeGridContainer.java
index 0a2bddc0dd736ac4d17925715dc313e1154c310d..c3f3747ec0221eacd6ea1d10deb7e800f229135c 100644
--- a/openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ui/experiment/ProjectSelectionTreeGridContainer.java
+++ b/openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ui/experiment/ProjectSelectionTreeGridContainer.java
@@ -125,6 +125,8 @@ public final class ProjectSelectionTreeGridContainer extends LayoutContainer imp
{
final TreeGrid<ModelData> treeGrid =
new TreeGrid<ModelData>(new TreeStore<ModelData>(), columnModel);
+ // WORKAROUND see comment in AbstractBrowserGrid constructor
+ treeGrid.setLazyRowRender(0);
treeGrid.setId(ID);
treeGrid.setBorders(true);
treeGrid.setAutoExpandColumn(ModelDataPropertyNames.CODE);
@@ -142,10 +144,11 @@ public final class ProjectSelectionTreeGridContainer extends LayoutContainer imp
@Override
public void selectionChanged(SelectionChangedEvent<ModelData> se)
{
- if (selectedProjectLinkOrNull != null)
+ if (selectedProjectOrNull != null)
{
selectedProjectLinkOrNull.setVisible(false);
selectedProjectLinkOrNull = null;
+ selectedProjectOrNull = null;
}
ModelData selected = se.getSelectedItem();
@@ -352,7 +355,7 @@ public final class ProjectSelectionTreeGridContainer extends LayoutContainer imp
private void clearTree()
{
projectLinks.clear();
- tree.getStore().removeAll();
+ tree.getTreeStore().removeAll();
}
/**
@@ -383,6 +386,7 @@ public final class ProjectSelectionTreeGridContainer extends LayoutContainer imp
}
}
}
+
}
/** @return a sorted set of groups of given <var>projects</var> */
diff --git a/openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ui/grid/AbstractBrowserGrid.java b/openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ui/grid/AbstractBrowserGrid.java
index 7ac5d592a74d41e64aa990dfde822310f4a0e5aa..c63f24d5e7d32923621c9386b15e77a697d12da2 100644
--- a/openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ui/grid/AbstractBrowserGrid.java
+++ b/openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ui/grid/AbstractBrowserGrid.java
@@ -235,6 +235,12 @@ public abstract class AbstractBrowserGrid<T/* Entity */, M extends BaseEntityMod
this.customColumnsMetadataProvider = new CustomColumnsMetadataProvider();
this.grid = createGrid(pagingLoader, gridId);
+ // WORKAROUND
+ // Lazy loading of rows causes tests using experiment browser fail (selection of
+ // project in project tree grid doesn't work).
+ // Turning it off for all grids is the safest solution for our system tests framework
+ // and should improve GUI speed in development mode a bit.
+ grid.setLazyRowRender(0);
this.pagingToolbar =
new BrowserGridPagingToolBar(asActionInvoker(), viewContext, PAGE_SIZE, gridId);
pagingToolbar.bind(pagingLoader);
diff --git a/openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/util/GWTUtils.java b/openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/util/GWTUtils.java
index 92e0e7b147ce8f66ef0b3e19c4e02c846e904c76..6fb1f46da1d97e13ad1c10bf9dbe30e98343fc38 100644
--- a/openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/util/GWTUtils.java
+++ b/openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/util/GWTUtils.java
@@ -24,7 +24,6 @@ import com.extjs.gxt.ui.client.store.ListStore;
import com.extjs.gxt.ui.client.widget.form.ComboBox;
import com.extjs.gxt.ui.client.widget.form.Field;
import com.extjs.gxt.ui.client.widget.treegrid.TreeGrid;
-import com.extjs.gxt.ui.client.widget.treepanel.TreePanel;
import com.google.gwt.core.client.GWT;
import com.google.gwt.user.client.ui.ListBox;
@@ -128,22 +127,7 @@ public final class GWTUtils
public final static void setSelectedItem(final TreeGrid<ModelData> tree, final String property,
final String value)
{
- System.err.println("set selected " + property + "=" + value);
ModelData model = tryFindModel(tree.getTreeStore().getAllItems(), property, value);
- System.err.println(model);
- if (model != null)
- {
- tree.getSelectionModel().select(model, false);
- }
- }
-
- /**
- * Selects given <var>value</var> of given <var>tree</var>.
- */
- public final static void setSelectedItem(final TreePanel<ModelData> tree,
- final String property, final String value)
- {
- ModelData model = tryFindModel(tree.getStore().getAllItems(), property, value);
if (model != null)
{
tree.getSelectionModel().select(model, false);
diff --git a/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/EditingTestSuite.java b/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/EditingTestSuite.java
index 304b9f9e056f6d9eefc7d7cdfda4ec9bca44664c..2e986238b81a1de687671f2a5b9682ada71aa64e 100644
--- a/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/EditingTestSuite.java
+++ b/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/EditingTestSuite.java
@@ -16,11 +16,11 @@
package ch.systemsx.cisd.openbis.generic;
-import com.google.gwt.junit.tools.GWTTestSuite;
-
import junit.framework.Test;
import junit.framework.TestSuite;
+import com.google.gwt.junit.tools.GWTTestSuite;
+
import ch.systemsx.cisd.openbis.generic.client.web.client.application.AuthenticationTest;
import ch.systemsx.cisd.openbis.generic.client.web.client.application.AuthorizationGroupsTest;
import ch.systemsx.cisd.openbis.generic.client.web.client.application.AuthorizationManagementConsolTest;
@@ -33,8 +33,6 @@ import ch.systemsx.cisd.openbis.plugin.generic.client.web.client.application.exp
import ch.systemsx.cisd.openbis.plugin.generic.client.web.client.application.sample.GenericSampleRegistrationTest;
/**
- *
- *
* @author Chandrasekhar Ramakrishnan
*/
public class EditingTestSuite extends GWTTestSuite
diff --git a/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/ViewingTestSuite2.java b/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/ViewingTestSuite2.java
index 299cd6508ec43907b0dc1c1b38ee8d9d5e9cca05..b9a059f86b29c837c904fa2c5bfbbdcc435a29a0 100644
--- a/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/ViewingTestSuite2.java
+++ b/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/ViewingTestSuite2.java
@@ -35,8 +35,6 @@ import ch.systemsx.cisd.openbis.plugin.generic.client.web.client.application.exp
import ch.systemsx.cisd.openbis.plugin.generic.client.web.client.application.sample.GenericSampleViewerTest;
/**
- *
- *
* @author Chandrasekhar Ramakrishnan
*/
public class ViewingTestSuite2 extends GWTTestSuite
diff --git a/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ExperimentBrowserTest.java b/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ExperimentBrowserTest.java
index eea0c61813239c94ef63f87338a661b585bdac78..358c8fa72844a930037d76d83ccf42a7744363a1 100644
--- a/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ExperimentBrowserTest.java
+++ b/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ExperimentBrowserTest.java
@@ -16,9 +16,6 @@
package ch.systemsx.cisd.openbis.generic.client.web.client.application;
-import com.google.gwt.junit.DoNotRunWith;
-import com.google.gwt.junit.Platform;
-
import ch.systemsx.cisd.openbis.generic.client.web.client.application.menu.TopMenu.ActionMenuKind;
import ch.systemsx.cisd.openbis.generic.client.web.client.application.ui.experiment.CheckExperimentTable;
import ch.systemsx.cisd.openbis.generic.client.web.client.application.ui.experiment.ListExperiments;
@@ -31,7 +28,6 @@ import ch.systemsx.cisd.openbis.generic.shared.basic.dto.EntityType;
*
* @author Izabela Adamczyk
*/
-@DoNotRunWith(Platform.HtmlUnit)
public class ExperimentBrowserTest extends AbstractGWTTestCase
{
diff --git a/openbis/sourceTest/java/ch/systemsx/cisd/openbis/plugin/generic/client/web/client/application/experiment/GenericExperimentAttachmentDownloadTest.java b/openbis/sourceTest/java/ch/systemsx/cisd/openbis/plugin/generic/client/web/client/application/experiment/GenericExperimentAttachmentDownloadTest.java
index 4c014d64e4afa4dd61ba954f5573dbb99d1817db..05d3dc72717f6e0028b4bf7225ee224ee5077a23 100644
--- a/openbis/sourceTest/java/ch/systemsx/cisd/openbis/plugin/generic/client/web/client/application/experiment/GenericExperimentAttachmentDownloadTest.java
+++ b/openbis/sourceTest/java/ch/systemsx/cisd/openbis/plugin/generic/client/web/client/application/experiment/GenericExperimentAttachmentDownloadTest.java
@@ -40,6 +40,8 @@ import ch.systemsx.cisd.openbis.generic.shared.basic.TechId;
/**
* @author Tomasz Pylak
*/
+// FIXME
+@DoNotRunWith(Platform.HtmlUnit)
public class GenericExperimentAttachmentDownloadTest extends AbstractGWTTestCase
{
private static final String DEFAULT = "DEFAULT (CISD)";
@@ -56,7 +58,6 @@ public class GenericExperimentAttachmentDownloadTest extends AbstractGWTTestCase
remoteConsole.prepare(new ShowExperiment(experimentCode));
}
- @DoNotRunWith(Platform.HtmlUnit)
public final void testDownloadAttachment()
{
prepareShowExperiment(DEFAULT, SIRNA_HCS, EXP_REUSE);
diff --git a/openbis/sourceTest/java/ch/systemsx/cisd/openbis/plugin/generic/client/web/client/application/experiment/GenericExperimentEditorTest.java b/openbis/sourceTest/java/ch/systemsx/cisd/openbis/plugin/generic/client/web/client/application/experiment/GenericExperimentEditorTest.java
index 34223ce130fee3e4775831bf6f679b4da0cb4a42..6360b42cb08a681ed7ddedad9adc393cadb206b4 100644
--- a/openbis/sourceTest/java/ch/systemsx/cisd/openbis/plugin/generic/client/web/client/application/experiment/GenericExperimentEditorTest.java
+++ b/openbis/sourceTest/java/ch/systemsx/cisd/openbis/plugin/generic/client/web/client/application/experiment/GenericExperimentEditorTest.java
@@ -18,9 +18,6 @@ package ch.systemsx.cisd.openbis.plugin.generic.client.web.client.application.ex
import java.util.Date;
-import com.google.gwt.junit.DoNotRunWith;
-import com.google.gwt.junit.Platform;
-
import ch.systemsx.cisd.openbis.generic.client.web.client.application.menu.TopMenu.ActionMenuKind;
import ch.systemsx.cisd.openbis.generic.client.web.client.application.ui.experiment.CheckExperimentTable;
import ch.systemsx.cisd.openbis.generic.client.web.client.application.ui.experiment.ListExperiments;
@@ -35,7 +32,6 @@ import ch.systemsx.cisd.openbis.plugin.generic.client.web.client.application.Pro
*
* @author Izabela Adamczyk
*/
-@DoNotRunWith(Platform.HtmlUnit)
public class GenericExperimentEditorTest extends AbstractGWTTestCase
{
private static final String SIRNA_HCS = "SIRNA_HCS";
diff --git a/openbis/sourceTest/java/ch/systemsx/cisd/openbis/plugin/generic/client/web/client/application/experiment/GenericExperimentViewerTest.java b/openbis/sourceTest/java/ch/systemsx/cisd/openbis/plugin/generic/client/web/client/application/experiment/GenericExperimentViewerTest.java
index 0222209e102bb4a8db215ab7d6aee19e615b6118..79ad3c64f45643029e194f45cc76a441941f6b5e 100644
--- a/openbis/sourceTest/java/ch/systemsx/cisd/openbis/plugin/generic/client/web/client/application/experiment/GenericExperimentViewerTest.java
+++ b/openbis/sourceTest/java/ch/systemsx/cisd/openbis/plugin/generic/client/web/client/application/experiment/GenericExperimentViewerTest.java
@@ -16,9 +16,6 @@
package ch.systemsx.cisd.openbis.plugin.generic.client.web.client.application.experiment;
-import com.google.gwt.junit.DoNotRunWith;
-import com.google.gwt.junit.Platform;
-
import ch.systemsx.cisd.openbis.generic.client.web.client.ICommonClientServiceAsync;
import ch.systemsx.cisd.openbis.generic.client.web.client.application.AbstractAsyncCallback;
import ch.systemsx.cisd.openbis.generic.client.web.client.application.IViewContext;
@@ -126,7 +123,6 @@ public class GenericExperimentViewerTest extends AbstractGWTTestCase
launchTest();
}
- @DoNotRunWith(Platform.HtmlUnit)
public final void testShowExperimentDetails()
{
prepareShowExperiment(NEMO, SIRNA_HCS, EXP1);
@@ -147,7 +143,6 @@ public class GenericExperimentViewerTest extends AbstractGWTTestCase
launchTest();
}
- @DoNotRunWith(Platform.HtmlUnit)
public final void testShowInvalidExperimentDetails()
{
prepareShowExperiment(DEFAULT, SIRNA_HCS, EXP_X);
@@ -171,7 +166,6 @@ public class GenericExperimentViewerTest extends AbstractGWTTestCase
launchTest();
}
- @DoNotRunWith(Platform.HtmlUnit)
public final void testListOfAttachments()
{
prepareShowExperiment(DEFAULT, SIRNA_HCS, EXP_REUSE);
@@ -190,7 +184,6 @@ public class GenericExperimentViewerTest extends AbstractGWTTestCase
launchTest();
}
- @DoNotRunWith(Platform.HtmlUnit)
public final void testListOfSamples()
{
prepareShowExperiment(DEFAULT, SIRNA_HCS, EXP_REUSE);
@@ -218,7 +211,6 @@ public class GenericExperimentViewerTest extends AbstractGWTTestCase
launchTest();
}
- @DoNotRunWith(Platform.HtmlUnit)
public final void testListOfDataSets()
{
prepareShowExperiment(NEMO, SIRNA_HCS, EXP1);