Skip to content

Commit

Permalink
Merge branch '3.2.x-new' of https://github.com/scalatest/scalatest in…
Browse files Browse the repository at this point in the history
…to 3.3-retries-fix
  • Loading branch information
cheeseng committed Sep 21, 2023
2 parents c27c76d + bbc03be commit e8b96dd
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions jvm/core/src/main/scala/org/scalatest/Retries.scala
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,6 @@
*/
package org.scalatest

import org.scalatest.concurrent.SleepHelper
import org.scalatest.concurrent.SleepHelper
import org.scalatest.concurrent.SleepHelper
import time.Span

Expand Down Expand Up @@ -345,11 +343,15 @@ trait Retries {
val firstOutcome = blk
firstOutcome match {
case Failed(ex) =>
if (delay != Span.Zero)
SleepHelper.sleep(delay.millisPart, delay.nanosPart)
blk match {
case Succeeded => Canceled(Resources.testFlickered, ex)
case other => firstOutcome
}
case Canceled(ex) =>
if (delay != Span.Zero)
SleepHelper.sleep(delay.millisPart, delay.nanosPart)
blk match {
case Succeeded => Succeeded
case failed: Failed => failed // Never hide a failure.
Expand Down Expand Up @@ -567,7 +569,7 @@ trait Retries {
firstOutcome match {
case Failed(ex) =>
if (delay != Span.Zero)
SleepHelper.sleep(delay.millisPart)
SleepHelper.sleep(delay.millisPart, delay.nanosPart)
blk match {
case Succeeded => Canceled(Resources.testFlickered, ex)
case other => firstOutcome
Expand Down Expand Up @@ -782,8 +784,7 @@ trait Retries {
firstOutcome match {
case Canceled(ex) =>
if (delay != Span.Zero)
SleepHelper.sleep(delay.millisPart)
SleepHelper.sleep(delay.millisPart)
SleepHelper.sleep(delay.millisPart, delay.nanosPart)
blk match {
case Succeeded => Succeeded
case failed: Failed => failed // Never hide a failure.
Expand Down

0 comments on commit e8b96dd

Please sign in to comment.