firefox-devtools/profiler

Test coverage for GC marker processing

Open

#731 opened on Jan 24, 2018

View on GitHubContributor guide
 (0 comments) (0 reactions) (0 assignees)TypeScript (486 forks)auto 404
code maintenancehelp wantedtests

Repository metrics

Stars
 (1,515 stars)
PR merge metrics
 (PR metrics pending)

Description

While looking for top file offenders for code coverage, this one stands out. We have no coverage for GC marker processing.

https://codecov.io/gh/devtools-html/perf.html/src/dde68013d994ec113c7513f85f2b0eadee9af2cf/src/profile-logic/process-profile.js#L494

┆Issue is synchronized with this Jira Task

Contributor guide

Research direction
Examine the uncovered code at src/profile logic/process profile.js line 494 to understand GC marker processing. Write unit tests to cover the relevant functions and ensure proper handling of GC markers.
Tech stack
javascripttypescript
Domain
performancebackend
Issue type
Test
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Mostly clear
Prerequisites
GitJavaScript
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
75