Remove more Fabric tests

facebook · Oct 8, 2019 · 9ce90ae · 9ce90ae
1 parent 55910ee
commit 9ce90ae
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 98 deletions.
diff --git a/packages/react-native-renderer/src/__tests__/ReactFabric-test.internal.js b/packages/react-native-renderer/src/__tests__/ReactFabric-test.internal.js
@@ -335,84 +335,6 @@ describe('ReactFabric', () => {
     });
   });
 
-  it('setNativeProps on native refs should no-op', () => {
-    const View = createReactNativeComponentClass('RCTView', () => ({
-      validAttributes: {foo: true},
-      uiViewClassName: 'RCTView',
-    }));
-
-    UIManager.updateView.mockReset();
-
-    let viewRef;
-    ReactFabric.render(
-      <View
-        foo="bar"
-        ref={ref => {
-          viewRef = ref;
-        }}
-      />,
-      11,
-    );
-
-    expect(UIManager.updateView).not.toBeCalled();
-    expect(() => {
-      ReactFabric.setNativeProps(viewRef, {foo: 'baz'});
-    }).toWarnDev([SET_NATIVE_PROPS_NOT_SUPPORTED_MESSAGE], {
-      withoutStack: true,
-    });
-    expect(UIManager.updateView).not.toBeCalled();
-  });
-
-  it('should warn and no-op if calling setNativeProps on non native refs', () => {
-    const View = createReactNativeComponentClass('RCTView', () => ({
-      validAttributes: {foo: true},
-      uiViewClassName: 'RCTView',
-    }));
-
-    class BasicClass extends React.Component {
-      render() {
-        return <React.Fragment />;
-      }
-    }
-
-    class Subclass extends ReactFabric.NativeComponent {
-      render() {
-        return <View />;
-      }
-    }
-
-    const CreateClass = createReactClass({
-      mixins: [NativeMethodsMixin],
-      render: () => {
-        return <View />;
-      },
-    });
-
-    [BasicClass, Subclass, CreateClass].forEach(Component => {
-      UIManager.updateView.mockReset();
-
-      let viewRef;
-      ReactFabric.render(
-        <Component
-          foo="bar"
-          ref={ref => {
-            viewRef = ref;
-          }}
-        />,
-        11,
-      );
-
-      expect(UIManager.updateView).not.toBeCalled();
-      expect(() => {
-        ReactFabric.setNativeProps(viewRef, {foo: 'baz'});
-      }).toWarnDev([SET_NATIVE_PROPS_NOT_SUPPORTED_MESSAGE], {
-        withoutStack: true,
-      });
-
-      expect(UIManager.updateView).not.toBeCalled();
-    });
-  });
-
   it('should call FabricUIManager.measure on ref.measure', () => {
     const View = createReactNativeComponentClass('RCTView', () => ({
       validAttributes: {foo: true},

diff --git a/packages/react-native-renderer/src/__tests__/ReactFabricAndNative-test.internal.js b/packages/react-native-renderer/src/__tests__/ReactFabricAndNative-test.internal.js
@@ -73,24 +73,4 @@ describe('ReactFabric', () => {
       [10, 20],
     );
   });
-
-  it('sets native props with setNativeProps on Fabric nodes with the RN renderer', () => {
-    UIManager.updateView.mockReset();
-    const View = createReactNativeComponentClass('RCTView', () => ({
-      validAttributes: {title: true},
-      uiViewClassName: 'RCTView',
-    }));
-
-    let ref = React.createRef();
-
-    ReactFabric.render(<View title="bar" ref={ref} />, 11);
-    expect(UIManager.updateView).not.toBeCalled();
-    ReactNative.setNativeProps(ref.current, {title: 'baz'});
-    expect(UIManager.updateView).toHaveBeenCalledTimes(1);
-    expect(UIManager.updateView).toHaveBeenCalledWith(
-      expect.any(Number),
-      'RCTView',
-      {title: 'baz'},
-    );
-  });
 });