Skip to content

Commit

Permalink
Fix inline styles in IntegrationTests (facebook#22165)
Browse files Browse the repository at this point in the history
Summary:
Fix `react-native/no-inline-styles` eslint warning for IntegrationTests module.
Pull Request resolved: facebook#22165

Differential Revision: D12946915

Pulled By: TheSavior

fbshipit-source-id: 438bb74cc34dd5893f725e4865568715ce949c3a
  • Loading branch information
ignacioola authored and facebook-github-bot committed Nov 6, 2018
1 parent a1316c3 commit 592cb71
Show file tree
Hide file tree
Showing 4 changed files with 27 additions and 6 deletions.
11 changes: 9 additions & 2 deletions IntegrationTests/AsyncStorageTest.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@

const React = require('react');
const ReactNative = require('react-native');
const {AsyncStorage, Text, View} = ReactNative;
const {AsyncStorage, Text, View, StyleSheet} = ReactNative;
const {TestModule} = ReactNative.NativeModules;

const deepDiffer = require('deepDiffer');
Expand Down Expand Up @@ -191,7 +191,7 @@ class AsyncStorageTest extends React.Component<{}, $FlowFixMeState> {

render() {
return (
<View style={{backgroundColor: 'white', padding: 40}}>
<View style={styles.container}>
<Text>
{/* $FlowFixMe(>=0.54.0 site=react_native_fb,react_native_oss) This
* comment suppresses an error found when Flow v0.54 was deployed.
Expand All @@ -205,6 +205,13 @@ class AsyncStorageTest extends React.Component<{}, $FlowFixMeState> {
}
}

const styles = StyleSheet.create({
container: {
backgroundColor: 'white',
padding: 40,
},
});

AsyncStorageTest.displayName = 'AsyncStorageTest';

module.exports = AsyncStorageTest;
5 changes: 4 additions & 1 deletion IntegrationTests/ImageCachePolicyTest.js
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ class ImageCachePolicyTest extends React.Component<Props, $FlowFixMeState> {

render() {
return (
<View style={{flex: 1}}>
<View style={styles.container}>
<Text>Hello</Text>
<Image
source={{
Expand Down Expand Up @@ -110,6 +110,9 @@ class ImageCachePolicyTest extends React.Component<Props, $FlowFixMeState> {
}

const styles = StyleSheet.create({
container: {
flex: 1,
},
base: {
width: 100,
height: 100,
Expand Down
11 changes: 9 additions & 2 deletions IntegrationTests/IntegrationTestHarnessTest.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
const requestAnimationFrame = require('fbjs/lib/requestAnimationFrame');
const React = require('react');
const ReactNative = require('react-native');
const {Text, View} = ReactNative;
const {Text, View, StyleSheet} = ReactNative;
const {TestModule} = ReactNative.NativeModules;

type Props = $ReadOnly<{|
Expand Down Expand Up @@ -54,7 +54,7 @@ class IntegrationTestHarnessTest extends React.Component<Props, State> {

render() {
return (
<View style={{backgroundColor: 'white', padding: 40}}>
<View style={styles.container}>
<Text>
{/* $FlowFixMe(>=0.54.0 site=react_native_fb,react_native_oss) This
* comment suppresses an error found when Flow v0.54 was deployed.
Expand All @@ -67,6 +67,13 @@ class IntegrationTestHarnessTest extends React.Component<Props, State> {
}
}

const styles = StyleSheet.create({
container: {
backgroundColor: 'white',
padding: 40,
},
});

IntegrationTestHarnessTest.displayName = 'IntegrationTestHarnessTest';

module.exports = IntegrationTestHarnessTest;
6 changes: 5 additions & 1 deletion IntegrationTests/SimpleSnapshotTest.js
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ class SimpleSnapshotTest extends React.Component<{}> {

render() {
return (
<View style={{backgroundColor: 'white', padding: 100}}>
<View style={styles.container}>
<View style={styles.box1} />
<View style={styles.box2} />
</View>
Expand All @@ -43,6 +43,10 @@ class SimpleSnapshotTest extends React.Component<{}> {
}

const styles = StyleSheet.create({
container: {
backgroundColor: 'white',
padding: 100,
},
box1: {
width: 80,
height: 50,
Expand Down

0 comments on commit 592cb71

Please sign in to comment.