Merge lp:~mmcm/akiban-server/direct-update-it-bindings into lp:~akiban-technologies/akiban-server/trunk

Proposed by Mike McMahon
Status: Merged
Approved by: Nathan Williams
Approved revision: 2648
Merged at revision: 2649
Proposed branch: lp:~mmcm/akiban-server/direct-update-it-bindings
Merge into: lp:~akiban-technologies/akiban-server/trunk
Diff against target: 23 lines (+6/-0)
1 file modified
src/test/java/com/akiban/direct/DirectUpdateIT.java (+6/-0)
To merge this branch: bzr merge lp:~mmcm/akiban-server/direct-update-it-bindings
Reviewer Review Type Date Requested Status
Nathan Williams Approve
Review via email: mp+161202@code.launchpad.net

Description of the change

Newly added IT didn't always run properly because didn't guarantee services restart.

The order of test execution is somewhat unpredictable, so it passed in Jenkins.

To post a comment you must log in.
Revision history for this message
Nathan Williams (nwilliams) wrote :

As described.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/test/java/com/akiban/direct/DirectUpdateIT.java'
2--- src/test/java/com/akiban/direct/DirectUpdateIT.java 2013-04-24 15:50:16 +0000
3+++ src/test/java/com/akiban/direct/DirectUpdateIT.java 2013-04-26 17:43:33 +0000
4@@ -26,6 +26,7 @@
5 import java.sql.Date;
6 import java.sql.SQLException;
7 import java.sql.Timestamp;
8+import java.util.Map;
9
10 import org.junit.After;
11 import org.junit.Before;
12@@ -57,6 +58,11 @@
13 return super.serviceBindingsProvider().bindAndRequire(EmbeddedJDBCService.class, EmbeddedJDBCServiceImpl.class);
14 }
15
16+ @Override
17+ protected Map<String, String> startupConfigProperties() {
18+ return uniqueStartupConfigProperties(DirectUpdateIT.class);
19+ }
20+
21 @Before
22 public void setUp() throws Exception {
23 startTestServices();

Subscribers

People subscribed via source and target branches