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

Unhandled promise rejection: undefined thrown in CI #11958

Open
ggetz opened this issue Apr 25, 2024 · 4 comments
Open

Unhandled promise rejection: undefined thrown in CI #11958

ggetz opened this issue Apr 25, 2024 · 4 comments

Comments

@ggetz
Copy link
Contributor

ggetz commented Apr 25, 2024

I'm seeing some intermittent CI failures in the release test task (but not coverage) which appears to be due to a race condition in the Cesium3DTileset specs:

  Scene/Cesium3DTileset
    ✗ verify statistics
	Unhandled promise rejection: undefined thrown
@ggetz ggetz mentioned this issue Aug 22, 2024
6 tasks
@jjspace
Copy link
Contributor

jjspace commented Sep 3, 2024

These seem to be much more frequent than "intermittent" lately. I feel like I'm re-running a job almost daily because of this one "verify statics" test failure. any chance we can try and track this one down soon or exclude it completely until it can be fixed so we can rely on the little red X from CI actually being an error more?
https://github.com/CesiumGS/cesium/actions/runs/10684175472/job/29613965055

@ggetz
Copy link
Contributor Author

ggetz commented Sep 3, 2024

Let's exclude it for the time being, but leave this issue open to track the problem.

jjspace added a commit that referenced this issue Sep 3, 2024
@jjspace
Copy link
Contributor

jjspace commented Sep 17, 2024

Been seeing more of the other "statistics" tests failing apart from the one I already excluded

https://github.com/CesiumGS/cesium/actions/runs/10903427776/job/30257483763

  Scene/Cesium3DTileset
    ✗ verify batched features statistics
	Unhandled promise rejection: undefined thrown
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants