bevyengine / bevyengine/bevy

Add tests for BRP

Open
#19,371 0 comments 0 reactions 0 assignees View on GitHub
A-Dev-Tools C-Testing D-Modest S-Needs-Design X-Uncontroversial
Dominant language
Rust
Stars
48.2k
Forks
4.8k
Avg merge
3d 16h
Merged PRs (30d)
171

Description

I agree tests are much needed for BRP-related stuff. I am not sure what form they should take.

Should we make actual HTTP requests? That would probably be the best but I believe that would be much work and should go in its own pull request.

If you see a way to make lightweight unit tests, please assist me.

_Originally posted by @splo in https://github.com/bevyengine/bevy/issues/18871#issuecomment-2888760563_

Contributor guide

Open the contributing guide

Research direction

The issue does not name implementation files, existing tests, or a specific BRP entry point. Begin by locating the BRP code and its current test conventions, then decide whether lightweight unit tests or actual HTTP requests are appropriate. Done means BRP behavior is covered by tests, with the scope and test form agreed before implementation.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
api, testing-qa
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.