Skip to content

Commit

Permalink
Allow more time for credentials tests
Browse files Browse the repository at this point in the history
The switch from a single JenkinsRule instance at the class level to a
JenkinsRule per credentials test slows the individual tests but should
allow more total tests to be executed because the 180 second JenkinsRule
timeout won't be reached inside any single JenkinsRule test.

Still want to limit the total execution time of the CredentialsTest
class to not exceed 3 minutes.  Otherwise, total test execution time
becomes too long.
  • Loading branch information
MarkEWaite committed Nov 4, 2018
1 parent 760252f commit 76dade9
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -350,7 +350,7 @@ private void addCredential() throws IOException {
* @return true if another test should be allowed to start
*/
private boolean testPeriodNotExpired() {
return (System.currentTimeMillis() - firstTestStartTime) < ((180 - 120) * 1000L);
return (System.currentTimeMillis() - firstTestStartTime) < ((180 - 30) * 1000L);
}

@Test
Expand Down

0 comments on commit 76dade9

Please sign in to comment.