Microsoft/react-native-windows

Add E2E Snapshot Tests for TextInput

Open

#11309 opened on Feb 27, 2023

View on GitHub
 (1 comment) (0 reactions) (0 assignees)C# (6,114 stars) (544 forks)batch import
Area: FabricArea: TestsArea: TextInputNew ArchitectureWorkstream: Test Coverageenhancementgood first issue

Description

Add E2E Jest Snapshot Tests for TextInput. Tests should exist as samples within RNTester.

STATUS TYPES

Status Description
Not Implemented Prop is not implemented for given architecture.
Not Completed No test or sample exists for the prop.
Sample Completed Sample exists for the prop, but sample is not run in a Jest Snapshot Test.
Completed Sample and Snapshot test exists for the prop.

PROPS STATUS FOR TextInput

Test Description Status (Paper) Status (Fabric) Notes
Default Case [x] Sample Completed – RNTester [x] Completed
AccessibilityLabel [ ] Not Completed [ ] Not Completed
AccessibilityActions [ ] Not Completed [ ] Not Completed
AccessibilityRole [ ] Not Completed [x] Completed
AccessibilityHint [ ] Not Completed [ ] Not Completed
AccessibilityState [ ] Not Completed [ ] Not Completed
AccessibilityPosInSet [ ] Not Completed [x] Completed
AccessibilitySetSize [ ] Not Completed [x] Completed
AccessibilityLiveRegion [ ] Not Completed [x] Completed
onAccessibilityAction [ ] Not Completed [ ] Not Completed
allowFontScaling [ ] Not Completed [ ] Not Completed
autoCapitalize [x] Sample Completed – RNTester [x] Completed
autoCorrect [x] Sample Completed – RNTester [x] Completed
testID [ ] Not Completed [x] Completed
autoFocus [x] Sample Completed – RNTester [ ] Not Completed
blurOnSubmit [x] Sample Completed – RNTester 10/22 Pass [ ] Not Completed
caretHidden [ ] Not Completed [x] Completed
contextMenuHidden [ ] Not Completed [ ] Not Completed
defaultValue [ ] Not Completed [x] Completed
Editable [ ] Not Completed [x] Completed
keyboardType [x] Sample Completed – RNTester 10/22 Pass [ ] Not Completed
maxFontSizeMultiplier [ ] Not Completed [ ] Not Completed
maxLength [x] Sample Completed – RNTester 10/22 Pass [x] Completed
Multiline [x] Sample Completed – RNTester 10/22 Pass [x] Completed
numberOfLines [ ] Not Completed [ ] Not Completed
onBlur [x] Sample Completed – RNTester 10/22 Pass [ ] Not Completed
onChange [x] Sample Completed – RNTester 10/22 Pass [ ] Not Completed
onChangeText [x] Sample Completed – RNTester 10/22 Pass [x] Completed
onContentSizeChange [x] Sample Completed – RNTester 10/22 Pass [x] Completed
onEndEditing [x] Sample Completed – RNTester 10/22 Pass [ ] Not Completed
onPressIn [x] Sample Completed – RNTester 10/22 Pass [x] Completed
onPressOut [x] Sample Completed – RNTester 10/22 Pass [x] Completed
onFocus [x] Sample Completed – RNTester 10/22 Pass [ ] Not Completed
onKeyPress [x] Sample Completed – RNTester 10/22 Pass [ ] Not Completed
onLayout [ ] Not Completed [ ] Not Completed
onScroll [ ] Not Completed [ ] Not Completed
onSelectionChange [x] Sample Completed – RNTester 10/22 Pass [x] Completed
onSubmitEditing [x] Sample Completed – RNTester 10/22 Pass [ ] Not Completed
Placeholder [x] Sample Completed – RNTester 10/22 Pass [x] Completed
placeholderTextColor [ ] Not Completed [x] Completed
returnKeyType [x] Sample Completed – RNTester 10/22 Pass [x] Completed
scrollEnabled [ ] Not Completed [ ] Not Completed
secureTextEntry [ ] Not Completed [x] Completed
selection [x] Sample Completed – RNTester 10/22 Pass [x] Completed
selectionColor [ ] Not Completed [x] Completed
selectTextOnFocus [ ] Not Completed [ ] Not Completed
showSoftInputOnFocus [ ] Not Completed [ ] Not Completed
textAlign [ ] Not Completed [x] Completed
Value [x] Sample Completed – RNTester 10/22 Pass [x] Completed
Focus() [ ] Not Completed [ ] Not Completed
Blur() [ ] Not Completed [ ] Not Completed
Clear() [x] Sample Completed – RNTester 10/22 Pass [ ] Not Completed
isFocused() [ ] Not Completed [ ] Not Completed
keyDownEvents() [x] Sample Completed – RNTester 10/22 Pass [ ] Not Completed
onKeyDown [x] Sample Completed – RNTester 10/22 Pass [x] Completed
onKeyDownCapture [x] Sample Completed – RNTester 10/22 Pass [x] Completed
onKeyUp [x] Sample Completed – RNTester 10/22 Pass [ ] Not Completed
Spellcheck [x] Sample Completed – RNTester 10/22 Pass [x] Completed
Accessible [ ] Not Completed [ ] Not Completed
Focusable [ ] Not Completed [ ] Not Completed
BackgroundColor [ ] Not Completed [x] Completed
Color [ ] Not Completed [x] Completed
FontFamily [ ] Not Completed [x] Completed
FontSize [ ] Not Completed [x] Completed
FontStyle [ ] Not Completed [x] Completed
Font Weight [ ] Not Completed [x] Completed
LetterSpacing [ ] Not Completed [x] Completed
LineHeight [ ] Not Completed [x] Completed
TextDecorationLine [ ] Not Completed [x] Completed
TextShadowColor [ ] Not Completed [x] Completed
TextShadowOffset [ ] Not Completed [x] Completed
TextShadowRadius [ ] Not Completed [x] Completed

Contributor guide

Tech stack
reactjesttypescriptjavascript
Domain
testingmobile
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.
3-5 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
JestReact Native TestingSnapshot Testing basicsRNTester structure
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
85
Research direction
The issue requires adding Jest snapshot tests for TextInput props in RNTester. Start by examining existing snapshot tests in the RNTester directory (e.g., RNTester/RNTesterTest/**). For each prop marked 'Not Completed', create a test case that renders TextInput with the prop and captures a snapshot. Refer to the existing sample code for props that already have samples. Ensure tests cover both Paper and Fabric architectures as indicated in the table. The maintainer has not responded to comments, so assume the scope is as described.