feat: add scene-explorer-tests locally and improve coverage workflow#951
Draft
feat: add scene-explorer-tests locally and improve coverage workflow#951
Conversation
Contributor
📊 Benchmark Report
Click to expand full benchmark reportDecentraland Godot Explorer - Benchmark ReportGenerated: 2025-12-29_17-56-54 Total Tests: 5 📊 Comparison: vs main branch baseline
Table of ContentsSummary OverviewMemory Metrics
Object Counts
Rendering Metrics
Resource Analysis
Detailed Test ResultsTest 1: 1_Terms_and_ConditionsBenchmark Report: 1_Terms_and_ConditionsTimestamp: 2025-12-29_17-54-25 Memory Metrics
Object Counts
Rendering Metrics
Test 2: 2_LobbyBenchmark Report: 2_LobbyTimestamp: 2025-12-29_17-54-31 Memory Metrics
Object Counts
Rendering Metrics
Test 3: 3_MenuBenchmark Report: 3_MenuTimestamp: 2025-12-29_17-54-38 Memory Metrics
Object Counts
Rendering Metrics
Test 4: 4_Explorer_Goerli_PlazaBenchmark Report: 4_Explorer_Goerli_PlazaTimestamp: 2025-12-29_17-55-10 Memory Metrics
Object Counts
Rendering Metrics
Resource Analysis
Test 5: 4_Explorer_Genesis_PlazaBenchmark Report: 4_Explorer_Genesis_PlazaTimestamp: 2025-12-29_17-56-39 Memory Metrics
Object Counts
Rendering Metrics
Resource Analysis
📋 Logs & Artifacts
🔄 Updated: 2025-12-29 17:56:58 UTC |
kuruk-mm
force-pushed
the
feat/scene-explorer-tests-local
branch
from
43c64e7 to
659f8c1
Compare
- Add scene-explorer-tests source code to tests/scene-explorer-tests/ - Create index.json to control which scenes are tested (easy enable/disable) - Move coverage logic to separate src/coverage.rs module - Integrate test server management into xtask coverage command - Simplify CI workflow by removing npm package download - Disable livekit during coverage runs (use_deno only)
kuruk-mm
force-pushed
the
feat/scene-explorer-tests-local
branch
from
659f8c1 to
2c8550a
Compare
Contributor
📸 Snapshot Test DifferencesFound 70 snapshot(s) with differences. Download the coverage-snapshots artifact to review the full images.
How to review
To update snapshotsIf the changes are expected, copy the generated snapshots to replace the originals. |
Contributor
📦 Build Report🤖 Android
Build Status: ✅ Success 🍏 iOS
🔗 Workflow Run: View logs 🔄 Updated: 2025-12-29 18:05:18 UTC |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
tests/scene-explorer-tests/instead of downloading from npmindex.jsonto control which scenes are tested (easy enable/disable)src/coverage.rsmodulecargo run -- coveragecommanduse_denoonly)Changes
src/coverage.rs- Coverage-related logic extracted from main.rstests/scene-explorer-tests/- Scene test source code from decentraland/scene-explorer-tests.github/workflows/coverage.yml- Simplified workflow, removed npm downloadsrc/main.rs- Cleaned up, delegates to coverage moduleHow it works
When running
cargo run -- coverage:npm install,npm run build,npm run export-static-local)http://localhost:7666/scene-explorer-testsTest plan
cargo run -- coveragelocally