Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The decorator.sequencer testcase fails intermittently #1456

Closed
yurivict opened this issue Jun 26, 2023 · 3 comments · Fixed by #1457
Closed

The decorator.sequencer testcase fails intermittently #1456

yurivict opened this issue Jun 26, 2023 · 3 comments · Fixed by #1457

Comments

@yurivict
Copy link

It succeeds most of the time, but sometimes it fails like this:

 30/207 Test  #30: decorator.sequencer ........................Subprocess aborted***Exception:   0.01 sec
+----------------------------------------------------------------------+
                          decorator.sequencer
+----------------------------------------------------------------------+

- identity
** /usr/ports/devel/actor-framework/work/actor-framework-0.19.2/libcaf_core/test/decorator/sequencer.cpp:74   passed
** /usr/ports/devel/actor-framework/work/actor-framework-0.19.2/libcaf_core/test/decorator/sequencer.cpp:76   passed
** /usr/ports/devel/actor-framework/work/actor-framework-0.19.2/libcaf_core/test/decorator/sequencer.cpp:77   passed
** /usr/ports/devel/actor-framework/work/actor-framework-0.19.2/libcaf_core/test/decorator/sequencer.cpp:78   passed
** /usr/ports/devel/actor-framework/work/actor-framework-0.19.2/libcaf_core/test/decorator/sequencer.cpp:79   passed
** /usr/ports/devel/actor-framework/work/actor-framework-0.19.2/libcaf_core/test/decorator/sequencer.cpp:80   passed
** /usr/ports/devel/actor-framework/work/actor-framework-0.19.2/libcaf_core/test/decorator/sequencer.cpp:81   passed
  -> 7 checks took 3 ms

- lifetime_1a
** /usr/ports/devel/actor-framework/work/actor-framework-0.19.2/libcaf_core/test/decorator/sequencer.cpp:54   passed
** /usr/ports/devel/actor-framework/work/actor-framework-0.19.2/libcaf_core/test/decorator/sequencer.cpp:91   passed
  -> 2 checks took 667 us

- lifetime_1b
** /usr/ports/devel/actor-framework/work/actor-framework-0.19.2/libcaf_core/test/decorator/sequencer.cpp:54   passed
** /usr/ports/devel/actor-framework/work/actor-framework-0.19.2/libcaf_core/test/decorator/sequencer.cpp:101  passed
  -> 2 checks took 586 us

- lifetime_2a

Version: 0.19.2
FreeBSD 13.2

@Neverlord
Copy link
Member

Thanks for reporting! This suite tests an already deprecated feature that we'll remove in the next release, so it's not really worth the time to hunt down whether it's an issue with the test itself or an actual issue with the decorator. I'll just remove the test suite ahead of time.

@yurivict
Copy link
Author

@Neverlord

How can this code be tested then?

@Neverlord
Copy link
Member

What do you mean? The sequencer code won't change since we will drop it soon, so we don't need regression tests in place.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants