microsoft / microsoft/react-native-windows

Separate IntegrationTests Native Project from IntegrationTests JS target

Open
#5,376 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Code Cleanup enhancement Quick Task Workstream: Test Coverage
Dominant language
C++
Stars
17.3k
Forks
1.2k
Avg merge
1d 13h
Merged PRs (30d)
33

Description

We have a project in vnext called IntegrationTests. This name is the same as RN's JS for IntegrationTests, which is one of our JS build targets. This means the same folder has source and targets, making it awkward to clean the JS or discover what bits are unique source.

The name of the JS should ideally match upstream RN, like other JS targets. We should consider moving the native project. This could potentially happen with upcoming changes to the desktop test runner.

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 inspecting the vnext IntegrationTests project and the IntegrationTests JS build target, then review the desktop test runner changes mentioned in the issue. Determine how the native project can be separated from the JS target. Done means they have distinct names or locations, and the JS source and target contents are easier to clean and distinguish.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp, react-native
Domain
build-system, testing
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.