webpack / webpack/webpack-bundle-analyzer

Help us add missing test coverage for analyzerMode: "server"

Open
#727 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

flag: Community help wanted flag: Needs Tests good first issue Status: Ready for PR
Dominant language
JavaScript
Stars
12.7k
Forks
514
Avg merge
6d 2h
Merged PRs (30d)
4

Description

I have noticed that analyzerMode === "server" is not really covered even in base branch so those changed lines also can't be verified by CI... would be lovely to see a change that would add missing coverage for that.

Originally posted by @valscion in https://github.com/webpack/webpack-bundle-analyzer/pull/725#pullrequestreview-4799763471

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the existing tests and implementation paths for analyzerMode: "server", then compare coverage on the base branch. Add coverage for the currently unverified server mode and run the project’s test and coverage checks so CI verifies the changed lines.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
testing-qa
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
52/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.