Repository navigation
Use a profile with tab information in the tab selector URL tests - #6370
Merged
Merged
Conversation
The tests were using tab IDs that didn't exist in the profile, which caused 'Failed to find the thread indexes for given tab' warnings in the test output.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6370 +/- ##
==========================================
- Coverage 84.12% 84.11% -0.01%
==========================================
Files 356 356
Lines 38445 38445
Branches 10787 10888 +101
==========================================
- Hits 32340 32338 -2
- Misses 5676 5678 +2
Partials 429 429 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
fatadel
approved these changes
Oct 5, 2026
Merged
via the queue into
firefox-devtools:main
with commit Oct 6, 2026
e46a095
23 of 24 checks passed
fatadel
added a commit
that referenced
this pull request
Oct 13, 2026
Changes: [Florian Quèze] profiler-cli: accept several marker handles and ranges in marker info (#6277) [Nazım Can Altınova] Add "profiler-cli profile save <path>" (#6340) [Nazım Can Altınova] Fix the upload progress bar not showing in light mode (#6348) [Nazım Can Altınova] Fix the variable name passed to the malformed response error strings (#6351) [Markus Stange] Add missing await when calling Chrome importer (#6346) [Markus Stange] Use "falls through" for all fallthrough comments (#6344) [Markus Stange] Move source map related profile logic files into profile-logic/source-maps/ (#6349) [Markus Stange] Make the source view actually read-only. (#6313) [Markus Stange] Run CI on merge queue branches (#6352) [Markus Stange] Serve production build by default during local development (#6353) [Nazım Can Altınova] Fix "... was not wrapped in act" warnings in tests (#6362) [Nazım Can Altınova] Add a scope tree test for shorthand destructuring defaults (#6367) [fatadel] Accept --session before subcommands in profiler-cli (#6358) [Nazım Can Altınova] Replace the deprecated loose option with Babel assumptions (#6366) [fatadel] Clarify how to save profiles for later access (#6355) [Markus Stange] Speed up activity graph drawing (#5888) [Nazım Can Altınova] Make `yarn test-cli` and `yarn test-all` faster (#6369) [Nazım Can Altınova] Add global `--permalink` and `--short-permalink` flags to profiler-cli (#6338) [Nazım Can Altınova] Use a profile with tab information in the tab selector URL tests (#6370) [fatadel] Move FileIO table labels into the Gecko profile upgrader (#6375) [fatadel] Clear preview selections when zooming out in the CLI (#6380) [Nazım Can Altınova] Add the ability to apply source maps to the profiles using `profiler-edit` (#6382) [fatadel] 🔃 Sync: l10n -> main (October 13, 2026) (#6389) And special thanks to our localizers: de: Ralf Duehnfahr el: Jim Spentzos en-GB: Ian Neal es-CL: ravmn fr: Théo Chevalier fy-NL: Fjoerfoks ia: Melo46 it: Francesco Lodolo [:flod] nl: Fjoerfoks ro: IulianArian ro: robbp ru: Valery Ledovskoy sv-SE: Andreas Pettersson tr: Selim Şumlu tr: Sena nur zh-TW: Pin-guang Chen
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
2 participants
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.
Main | Deploy preview
Fixes #6049.
The tests were using tab IDs that didn't exist in the profile, which caused 'Failed to find the thread indexes for given tab' warnings in the test output.