2019-02-06 14:49:11 +00:00
|
|
|
# Chronos Test Suite
|
|
|
|
# (c) Copyright 2018-Present
|
2018-05-22 10:16:56 +00:00
|
|
|
# Status Research & Development GmbH
|
|
|
|
#
|
|
|
|
# Licensed under either of
|
|
|
|
# Apache License, version 2.0, (LICENSE-APACHEv2)
|
|
|
|
# MIT license (LICENSE-MIT)
|
exception tracking (#166)
* exception tracking
This PR adds minimal exception tracking to chronos, moving the goalpost
one step further.
In particular, it becomes invalid to raise exceptions from `callSoon`
callbacks: this is critical for writing correct error handling because
there's no reasonable way that a user of chronos can possibly _reason_
about exceptions coming out of there: the event loop will be in an
indeterminite state when the loop is executing an _random_ callback.
As expected, there are several issues in the error handling of chronos:
in particular, it will end up in an inconsistent internal state whenever
the selector loop operations fail, because the internal state update
functions are not written in an exception-safe way. This PR turns this
into a Defect, which probably is not the optimal way of handling things
- expect more work to be done here.
Some API have no way of reporting back errors to callers - for example,
when something fails in the accept loop, there's not much it can do, and
no way to report it back to the user of the API - this has been fixed
with the new accept flow - the old one should be deprecated.
Finally, there is information loss in the API: in composite operations
like `poll` and `waitFor` there's no way to differentiate internal
errors from user-level errors originating from callbacks.
* store `CatchableError` in future
* annotate proc's with correct raises information
* `selectors2` to avoid non-CatchableError IOSelectorsException
* `$` should never raise
* remove unnecessary gcsafe annotations
* fix exceptions leaking out of timer waits
* fix some imports
* functions must signal raising the union of all exceptions across all
platforms to enable cross-platform code
* switch to unittest2
* add `selectors2` which supercedes the std library version and fixes
several exception handling issues in there
* fixes
* docs, platform-independent eh specifiers for some functions
* add feature flag for strict exception mode
also bump version to 3.0.0 - _most_ existing code should be compatible
with this version of exception handling but some things might need
fixing - callbacks, existing raises specifications etc.
* fix AsyncCheck for non-void T
2021-03-24 09:08:33 +00:00
|
|
|
import unittest2
|
2019-02-06 14:49:11 +00:00
|
|
|
import ../chronos
|
2018-05-22 10:16:56 +00:00
|
|
|
|
2023-03-31 05:35:04 +00:00
|
|
|
{.used.}
|
2019-10-24 13:01:57 +00:00
|
|
|
|
2019-03-30 22:31:10 +00:00
|
|
|
suite "callSoon() tests suite":
|
|
|
|
const CallSoonTests = 10
|
|
|
|
var soonTest1 = 0'u
|
|
|
|
var timeoutsTest1 = 0
|
|
|
|
var timeoutsTest2 = 0
|
|
|
|
var soonTest2 = 0
|
2018-05-22 10:16:56 +00:00
|
|
|
|
2019-03-30 22:31:10 +00:00
|
|
|
proc callback1(udata: pointer) {.gcsafe.} =
|
|
|
|
soonTest1 = soonTest1 xor cast[uint](udata)
|
2018-05-22 10:16:56 +00:00
|
|
|
|
2019-03-30 22:31:10 +00:00
|
|
|
proc test1(): uint =
|
|
|
|
callSoon(callback1, cast[pointer](0x12345678'u))
|
|
|
|
callSoon(callback1, cast[pointer](0x23456789'u))
|
|
|
|
callSoon(callback1, cast[pointer](0x3456789A'u))
|
|
|
|
callSoon(callback1, cast[pointer](0x456789AB'u))
|
|
|
|
callSoon(callback1, cast[pointer](0x56789ABC'u))
|
|
|
|
callSoon(callback1, cast[pointer](0x6789ABCD'u))
|
|
|
|
callSoon(callback1, cast[pointer](0x789ABCDE'u))
|
|
|
|
callSoon(callback1, cast[pointer](0x89ABCDEF'u))
|
|
|
|
callSoon(callback1, cast[pointer](0x9ABCDEF1'u))
|
|
|
|
callSoon(callback1, cast[pointer](0xABCDEF12'u))
|
|
|
|
callSoon(callback1, cast[pointer](0xBCDEF123'u))
|
|
|
|
callSoon(callback1, cast[pointer](0xCDEF1234'u))
|
|
|
|
callSoon(callback1, cast[pointer](0xDEF12345'u))
|
|
|
|
callSoon(callback1, cast[pointer](0xEF123456'u))
|
|
|
|
callSoon(callback1, cast[pointer](0xF1234567'u))
|
|
|
|
callSoon(callback1, cast[pointer](0x12345678'u))
|
|
|
|
## All callbacks must be processed exactly with 1 poll() call.
|
|
|
|
poll()
|
|
|
|
result = soonTest1
|
2018-05-22 10:16:56 +00:00
|
|
|
|
2019-03-30 22:31:10 +00:00
|
|
|
proc testProc() {.async.} =
|
|
|
|
for i in 1..CallSoonTests:
|
|
|
|
await sleepAsync(100.milliseconds)
|
|
|
|
timeoutsTest1 += 1
|
2018-05-22 10:16:56 +00:00
|
|
|
|
2023-06-05 20:21:50 +00:00
|
|
|
var callbackproc: proc(udata: pointer) {.gcsafe, raises: [].}
|
|
|
|
callbackproc = proc (udata: pointer) {.gcsafe, raises: [].} =
|
2019-03-30 22:31:10 +00:00
|
|
|
timeoutsTest2 += 1
|
exception tracking (#166)
* exception tracking
This PR adds minimal exception tracking to chronos, moving the goalpost
one step further.
In particular, it becomes invalid to raise exceptions from `callSoon`
callbacks: this is critical for writing correct error handling because
there's no reasonable way that a user of chronos can possibly _reason_
about exceptions coming out of there: the event loop will be in an
indeterminite state when the loop is executing an _random_ callback.
As expected, there are several issues in the error handling of chronos:
in particular, it will end up in an inconsistent internal state whenever
the selector loop operations fail, because the internal state update
functions are not written in an exception-safe way. This PR turns this
into a Defect, which probably is not the optimal way of handling things
- expect more work to be done here.
Some API have no way of reporting back errors to callers - for example,
when something fails in the accept loop, there's not much it can do, and
no way to report it back to the user of the API - this has been fixed
with the new accept flow - the old one should be deprecated.
Finally, there is information loss in the API: in composite operations
like `poll` and `waitFor` there's no way to differentiate internal
errors from user-level errors originating from callbacks.
* store `CatchableError` in future
* annotate proc's with correct raises information
* `selectors2` to avoid non-CatchableError IOSelectorsException
* `$` should never raise
* remove unnecessary gcsafe annotations
* fix exceptions leaking out of timer waits
* fix some imports
* functions must signal raising the union of all exceptions across all
platforms to enable cross-platform code
* switch to unittest2
* add `selectors2` which supercedes the std library version and fixes
several exception handling issues in there
* fixes
* docs, platform-independent eh specifiers for some functions
* add feature flag for strict exception mode
also bump version to 3.0.0 - _most_ existing code should be compatible
with this version of exception handling but some things might need
fixing - callbacks, existing raises specifications etc.
* fix AsyncCheck for non-void T
2021-03-24 09:08:33 +00:00
|
|
|
{.gcsafe.}:
|
2021-12-10 09:55:25 +00:00
|
|
|
callSoon(callbackproc)
|
2018-05-22 10:16:56 +00:00
|
|
|
|
2019-03-30 22:31:10 +00:00
|
|
|
proc test2(timers, callbacks: var int) =
|
2021-12-10 09:55:25 +00:00
|
|
|
callSoon(callbackproc)
|
2019-03-30 22:31:10 +00:00
|
|
|
waitFor(testProc())
|
|
|
|
timers = timeoutsTest1
|
|
|
|
callbacks = timeoutsTest2
|
2018-05-22 10:16:56 +00:00
|
|
|
|
2019-03-30 22:31:10 +00:00
|
|
|
proc testCallback(udata: pointer) =
|
|
|
|
soonTest2 = 987654321
|
2018-05-25 20:00:32 +00:00
|
|
|
|
2019-03-30 22:31:10 +00:00
|
|
|
proc test3(): bool =
|
|
|
|
callSoon(testCallback)
|
|
|
|
poll()
|
|
|
|
result = soonTest2 == 987654321
|
2018-05-25 20:00:32 +00:00
|
|
|
|
2019-03-30 22:31:10 +00:00
|
|
|
test "User-defined callback argument test":
|
|
|
|
var values = [0x12345678'u, 0x23456789'u, 0x3456789A'u, 0x456789AB'u,
|
|
|
|
0x56789ABC'u, 0x6789ABCD'u, 0x789ABCDE'u, 0x89ABCDEF'u,
|
|
|
|
0x9ABCDEF1'u, 0xABCDEF12'u, 0xBCDEF123'u, 0xCDEF1234'u,
|
|
|
|
0xDEF12345'u, 0xEF123456'u, 0xF1234567'u, 0x12345678'u]
|
|
|
|
var expect = 0'u
|
|
|
|
for item in values:
|
|
|
|
expect = expect xor item
|
|
|
|
check test1() == expect
|
|
|
|
test "`Asynchronous dead end` #7193 test":
|
|
|
|
var timers, callbacks: int
|
|
|
|
test2(timers, callbacks)
|
|
|
|
check:
|
|
|
|
timers == CallSoonTests
|
|
|
|
callbacks > CallSoonTests * 2
|
|
|
|
test "`callSoon() is not working prior getGlobalDispatcher()` #7192 test":
|
|
|
|
check test3() == true
|