diff options
-rw-r--r-- | org.eclipse.egit.ui.test/src/org/eclipse/egit/ui/test/history/HistoryViewTest.java | 21 |
1 files changed, 17 insertions, 4 deletions
diff --git a/org.eclipse.egit.ui.test/src/org/eclipse/egit/ui/test/history/HistoryViewTest.java b/org.eclipse.egit.ui.test/src/org/eclipse/egit/ui/test/history/HistoryViewTest.java index 3be86945c0..7ef62d324f 100644 --- a/org.eclipse.egit.ui.test/src/org/eclipse/egit/ui/test/history/HistoryViewTest.java +++ b/org.eclipse.egit.ui.test/src/org/eclipse/egit/ui/test/history/HistoryViewTest.java @@ -30,6 +30,7 @@ import org.eclipse.jgit.lib.Constants; import org.eclipse.jgit.lib.Repository; import org.eclipse.jgit.revwalk.RevCommit; import org.eclipse.swt.widgets.Display; +import org.eclipse.swt.widgets.TableItem; import org.eclipse.swtbot.eclipse.finder.widgets.SWTBotPerspective; import org.eclipse.swtbot.eclipse.finder.widgets.SWTBotView; import org.eclipse.swtbot.swt.finder.junit.SWTBotJunit4ClassRunner; @@ -284,8 +285,9 @@ public class HistoryViewTest extends LocalRepositoryTestCase { Display.getDefault().syncExec(new Runnable() { public void run() { - commit[0] = (RevCommit) table.widget.getSelection()[0] - .getData(); + TableItem tableItem = table.widget.getSelection()[0]; + ensureTableItemLoaded(tableItem); + commit[0] = (RevCommit) tableItem.getData(); } }); @@ -314,8 +316,9 @@ public class HistoryViewTest extends LocalRepositoryTestCase { Display.getDefault().syncExec(new Runnable() { public void run() { - commit[0] = (RevCommit) table.widget.getSelection()[0] - .getData(); + TableItem tableItem = table.widget.getSelection()[0]; + ensureTableItemLoaded(tableItem); + commit[0] = (RevCommit) tableItem.getData(); } }); @@ -325,4 +328,14 @@ public class HistoryViewTest extends LocalRepositoryTestCase { waitInUI(); assertEquals(commit[0].getId().name(), repo.getBranch()); } + + /** + * Workaround to ensure that the TableItem of a SWT table with style + * SWT_VIRTUAL is loaded. + * + * @param item + */ + private static void ensureTableItemLoaded(TableItem item) { + item.setText(item.getText()); // TODO: is there a better solution? + } } |