CesiumGS / CesiumGS/cesium

Replace "can support 2xx HTTP status (other than 200)" test

Open
#3,757 2 comments 0 reactions 0 assignees View on GitHub
good first issue test failure
Dominant language
JavaScript
Stars
15.7k
Forks
3.9k
Avg merge
4d 6h
Merged PRs (30d)
34

Description

The test `can support 2xx HTTP status (other than 200)` occasionally fails because it's relying on an external server that seems to have spotty availability. This causes CI to someone fail for no good reason. We should update this test to spy/mock the XHR request so that it doesn't need to access the network at all.

Contributor guide

Open the contributing guide

Research direction

Locate the test named `can support 2xx HTTP status (other than 200)` and inspect how its XHR request reaches the external server. Replace the network dependency with an XHR spy or mock, then run the relevant test to confirm it passes without network access and still checks the 2xx response behavior.

Written by the indexing model from the issue text.

Assessment

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.