FormidableLabs/radium

Rewrite all tests as blackbox

Open

#386 opened on Oct 18, 2015

View on GitHub
 (0 comments) (0 reactions) (0 assignees)JavaScript (7,414 stars) (352 forks)batch import
chorehelp wanted

Description

There are two types of tests in Radium.

  1. The older, whitebox, tests that require internal modules and test them individually. These tests are fragile and hard to maintain, relying on module injection. Even worse, they don't represent real usage.
  2. The newer, blackbox, tests, which test Radium exactly as external callers would (using the decorator or wrapper function). The new tests are easier to write, since they are using the public API, and represent actual use cases for Radium.

We should rewrite the entire test suite, wherever possible , in the new style. This probably also means splitting up the tests from the single file they are in now, radium-test.js.

Contributor guide

Tech stack
javascriptreact
Domain
frontendtesting
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.
3-5 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.
mostly clear
Prerequisites
knowledge of Reactfamiliarity with testing frameworks
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
45
Research direction
The issue proposes rewriting whitebox tests in `radium test.js` to blackbox tests that use the public API. The main file to modify is `radium test.js`. No prior pull requests or comments exist, so the contributor should start by examining the existing tests and understanding the difference between whitebox and blackbox testing as described. The goal is to split the single test file into multiple files and ensure each test uses the decorator or wrapper function externally.