From ae4946f98369141caa52fd09d63400d4a28e1d1f Mon Sep 17 00:00:00 2001 From: Tim Yung Date: Fri, 7 May 2021 21:04:47 -0700 Subject: [PATCH] Animated: Remove `Animated.__PropsOnlyForTests` Summary: Deletes `Animated.__PropsOnlyForTests` because it is unnecessary for actually testing `AnimatedProps`. Changelog: [General][Removed] - Removed `Animated.__PropsOnlyForTests`. Reviewed By: kacieb Differential Revision: D28271629 fbshipit-source-id: 7d4c83d72f7298ed43e3659126cb45d271c5dac7 --- Libraries/Animated/AnimatedImplementation.js | 2 -- Libraries/Animated/AnimatedMock.js | 1 - Libraries/Animated/__tests__/Animated-test.js | 8 +++++--- 3 files changed, 5 insertions(+), 6 deletions(-) diff --git a/Libraries/Animated/AnimatedImplementation.js b/Libraries/Animated/AnimatedImplementation.js index 12b225abab0616..706b87ea01e9e1 100644 --- a/Libraries/Animated/AnimatedImplementation.js +++ b/Libraries/Animated/AnimatedImplementation.js @@ -706,6 +706,4 @@ module.exports = { * Expose Event class, so it can be used as a type for type checkers. */ Event: AnimatedEvent, - - __PropsOnlyForTests: AnimatedProps, }; diff --git a/Libraries/Animated/AnimatedMock.js b/Libraries/Animated/AnimatedMock.js index eecfbb884ec42b..50e69439a50453 100644 --- a/Libraries/Animated/AnimatedMock.js +++ b/Libraries/Animated/AnimatedMock.js @@ -152,5 +152,4 @@ module.exports = { forkEvent: AnimatedImplementation.forkEvent, unforkEvent: AnimatedImplementation.unforkEvent, Event: AnimatedEvent, - __PropsOnlyForTests: AnimatedProps, }; diff --git a/Libraries/Animated/__tests__/Animated-test.js b/Libraries/Animated/__tests__/Animated-test.js index f7a8bb05ace086..e24a4e5620fa7c 100644 --- a/Libraries/Animated/__tests__/Animated-test.js +++ b/Libraries/Animated/__tests__/Animated-test.js @@ -8,6 +8,7 @@ * @emails oncall+react_native */ +import AnimatedProps from '../nodes/AnimatedProps'; import TestRenderer from 'react-test-renderer'; import * as React from 'react'; @@ -21,6 +22,7 @@ jest.mock('../../BatchedBridge/NativeModules', () => ({ })); let Animated = require('../Animated'); + describe('Animated tests', () => { beforeEach(() => { jest.resetModules(); @@ -32,7 +34,7 @@ describe('Animated tests', () => { const callback = jest.fn(); - const node = new Animated.__PropsOnlyForTests( + const node = new AnimatedProps( { style: { backgroundColor: 'red', @@ -786,7 +788,7 @@ describe('Animated tests', () => { const callback = jest.fn(); - const node = new Animated.__PropsOnlyForTests( + const node = new AnimatedProps( { style: { opacity: vec.x.interpolate({ @@ -890,7 +892,7 @@ describe('Animated tests', () => { const value3 = new Animated.Value(0); const value4 = Animated.add(value3, Animated.multiply(value1, value2)); const callback = jest.fn(); - const view = new Animated.__PropsOnlyForTests( + const view = new AnimatedProps( { style: { transform: [