CesiumGS / CesiumGS/cesium

Enable jasmine random test ordering and fix broken tests

Open
#6,385 0 comments 0 reactions 0 assignees View on GitHub
category - packaging & tooling type - cleanup
Dominant language
JavaScript
Stars
15.7k
Forks
3.9k
Avg merge
4d 6h
Merged PRs (30d)
34

Description

#6383 upgraded Jasmine which includes random testing ordering as the default. This is desired because it creates more robust, standalone unit tests and make sure unit tests don't conflict with each other. Unfortunately, Cesium's current unit tests have lots of this going on. Someone needs to re-enable the flag in `karma.conf.js` and fix any test failures exposed by repeatedly running `npm run test`. (this doesn't affect our ancient copy of jasmine that runs in the browser)

Contributor guide

Open the contributing guide

Research direction

Start in karma.conf.js by locating the Jasmine random-ordering flag, then run npm run test repeatedly to identify failures caused by test order. Done means the flag is re-enabled and the exposed Cesium unit-test failures are fixed without changing the ancient browser Jasmine copy.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
testing-qa
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.