beautifier/js-beautify

Task: Write Tests for Web Common Functions

Open

#1,547 opened on Sep 10, 2018

View on GitHub
 (2 comments) (0 reactions) (0 assignees)JavaScript (8,342 stars) (1,460 forks)batch import
help wantedimplementation: websitetype: task

Description

This is in relation to #1546 The issue was fixed but the lack of testing allowed it in the first place. Having some solid tests for web-common functions would prevent this type of issue going forward.

Contributor guide

Tech stack
javascript
Domain
testing
Issue type
test
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-2 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Familiarity with js beautify codebaseBasic testing knowledge
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
40
Research direction
Review the linked issue #1546 to understand what was fixed and why tests are needed. Identify the 'web common functions' in the repository (likely under the web common directory). Examine existing test files to understand the testing framework and conventions. Write tests covering the edges of the previously fixed bug and ensure all functions are tested.