Skip to content

Commit 2da5bef

Browse files
TESTS: Tests on cancel() now assert that internal state is 'finished'
1 parent 16f66ba commit 2da5bef

File tree

1 file changed

+2
-8
lines changed

1 file changed

+2
-8
lines changed

inst/testme/test-cancel.R

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -29,21 +29,15 @@ for (strategy in strategies) {
2929

3030
message(" Cancel future, which also interrupts the future, if supported")
3131
f <- cancel(f)
32-
stopifnot({
33-
f[["state"]] %in% c("canceled", "interrupted") ||
34-
(f[["state"]] == "finished" && inherits(f, "SequentialFuture"))
35-
})
32+
stopifnot(f[["state"]] %in% "finished")
3633

3734
n <- nbrOfFreeWorkers()
3835
message(" Number of free workers (after cancel + interupt): ", n)
3936

4037
message(" Check if canceled + interrupted future is resolved")
4138
f <- resolve(f)
4239
stopifnot(resolved(f))
43-
stopifnot({
44-
f[["state"]] %in% c("canceled", "interrupted") ||
45-
(f[["state"]] == "finished" && inherits(f, "SequentialFuture"))
46-
})
40+
stopifnot(f[["state"]] == "finished")
4741

4842
n <- nbrOfFreeWorkers()
4943
message(" Number of free workers (after resolve): ", n)

0 commit comments

Comments
 (0)