Fix: Add displayName to ActivityIndicator (#22417)

Summary:
Similar to #21950, this adds displayName to `ActivityIndicator`, so it displays the correct component name in snapshots instead of `Component`.

Fixes #21937 (a little bit more than it was already fixed).
Pull Request resolved: https://github.com/facebook/react-native/pull/22417

Differential Revision: D13205376

Pulled By: TheSavior

fbshipit-source-id: 21e71d4980d159a1f33d32d83346b48257ec21d4
This commit is contained in:
Michael Blanchard
2018-12-12 16:54:47 +00:00
committed by Lorenzo Sciandra
parent 2151c30474
commit 1d8744f0c6
3 changed files with 36 additions and 0 deletions
@@ -112,6 +112,7 @@ const ActivityIndicator = (
// $FlowFixMe - TODO T29156721 `React.forwardRef` is not defined in Flow, yet.
const ActivityIndicatorWithRef = React.forwardRef(ActivityIndicator);
ActivityIndicatorWithRef.displayName = 'ActivityIndicator';
ActivityIndicatorWithRef.defaultProps = {
animating: true,
@@ -0,0 +1,25 @@
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
* This source code is licensed under the MIT license found in the
* LICENSE file in the root directory of this source tree.
*
* @format
* @emails oncall+react_native
*/
'use strict';
const React = require('React');
const ReactTestRenderer = require('react-test-renderer');
const ActivityIndicator = require('ActivityIndicator');
describe('ActivityIndicator', () => {
it('renders correctly', () => {
const instance = ReactTestRenderer.create(
<ActivityIndicator size="large" color="#0000ff" />,
);
expect(instance.toJSON()).toMatchSnapshot();
});
});
@@ -0,0 +1,10 @@
// Jest Snapshot v1, https://goo.gl/fbAQLP
exports[`ActivityIndicator renders correctly 1`] = `
<ActivityIndicator
animating={true}
color="#0000ff"
hidesWhenStopped={true}
size="large"
/>
`;