Remove uses of $FlowExpectedError as a type across fbsource

Reviewed By: marcoww6

Differential Revision: D80579770

fbshipit-source-id: 9911e0f25e0dcef259f6d4980994be4a68c46e47

Author: SamChou19815

.flowconfig

@@ -17,7 +17,6 @@ munge_underscores=true
 
 experimental.error_code_migration=new
 suppress_type=$FlowFixMe
-suppress_type=$FlowExpectedError
 
 format.bracket_spacing=false
 

packages/react-relay/__tests__/RelayResolverNullableModelClientEdge-test.js

@@ -503,7 +503,7 @@ test('Errors thrown when reading the model a client edge points to are caught as
       handled: false,
     },
   ]);
-  const data: $FlowExpectedError = snapshot.data;
+  const data: $FlowFixMe = snapshot.data;
   expect(data.edge_to_model_that_throws).toBe(null);
 });
 
@@ -537,7 +537,7 @@ test('Errors thrown when reading plural client edge are caught as resolver error
       handled: false,
     },
   ]);
-  const data: $FlowExpectedError = snapshot.data;
+  const data: $FlowFixMe = snapshot.data;
   expect(data.edge_to_plural_models_that_throw).toStrictEqual([null, null]);
 });
 
@@ -564,7 +564,7 @@ test('Errors thrown when reading plural client edge are caught as resolver error
       handled: false,
     },
   ]);
-  const data: $FlowExpectedError = snapshot.data;
+  const data: $FlowFixMe = snapshot.data;
   expect(data.edge_to_plural_models_some_throw).toStrictEqual([
     null,
     {id: 'a valid id!'},

packages/react-relay/relay-hooks/__tests__/useIsParentQueryActive-test.js

@@ -46,11 +46,7 @@ beforeEach(() => {
   const source = new RecordSource();
   const store = new Store(source);
   fetch = jest.fn(
-    (
-      _query: $FlowExpectedError,
-      _variables: $FlowExpectedError,
-      _cacheConfig: $FlowExpectedError,
-    ) =>
+    (_query: $FlowFixMe, _variables: $FlowFixMe, _cacheConfig: $FlowFixMe) =>
       Observable.create(
         (
           sink: Sink<

packages/relay-runtime/store/OperationExecutor.js

@@ -570,9 +570,7 @@ class Executor<TMutation: MutationParameters> {
       const payloadFollowups = [];
       for (let i = 0; i < normalizedResponses.length; i++) {
         const response = normalizedResponses[i];
-        const source = new RelayRecordSource(
-          response.data as $FlowExpectedError,
-        );
+        const source = new RelayRecordSource(response.data as $FlowFixMe);
         const isFinal = response.extensions?.is_final === true;
         const payload: RelayResponsePayload = {
           errors: [],

packages/relay-runtime/store/__tests__/RelayModernEnvironment-ExecuteWithProvidedVariable-test.js

@@ -34,11 +34,7 @@ function getEnvironment(
 ): RelayModernEnvironment {
   let subject;
   const fetch = jest.fn(
-    (
-      _query: $FlowExpectedError,
-      _variables: $FlowExpectedError,
-      _cacheConfig: $FlowExpectedError,
-    ) =>
+    (_query: $FlowFixMe, _variables: $FlowFixMe, _cacheConfig: $FlowFixMe) =>
       RelayObservable.create(
         (
           sink: Sink<{

packages/relay-runtime/store/__tests__/RelayModernEnvironment-ExecuteWithUndeclaredUnusedArgument-test.js

@@ -78,11 +78,7 @@ describe('query with undeclared, unused fragment argument', () => {
     `;
     operation = createOperationDescriptor(query, {id: '4'});
     fetch = jest.fn(
-      (
-        _query: $FlowExpectedError,
-        _variables: $FlowExpectedError,
-        _cacheConfig: $FlowExpectedError,
-      ) =>
+      (_query: $FlowFixMe, _variables: $FlowFixMe, _cacheConfig: $FlowFixMe) =>
         RelayObservable.create(
           (
             sink: Sink<{

packages/relay-runtime/store/__tests__/resolvers/LiveResolvers-test.js

@@ -86,7 +86,7 @@ test('unsubscribe happens when record is updated due to missing data', () => {
 
   const snapshot = environment.lookup(operation.fragment);
   // lookup() doesn't have the nice types of reading a fragment through the actual APIs
-  let data: $FlowExpectedError = snapshot.data;
+  let data: $FlowFixMe = snapshot.data;
   const environmentUpdateHandler = jest.fn(() => {
     data = environment.lookup(operation.fragment).data;
   });
@@ -266,7 +266,7 @@ test('Errors thrown during _initial_ read() are caught as resolver errors', () =
       handled: false,
     },
   ]);
-  const data: $FlowExpectedError = snapshot.data;
+  const data: $FlowFixMe = snapshot.data;
   expect(data.counter_throws_when_odd).toBe(null);
 });
 
@@ -300,7 +300,7 @@ test('Errors thrown during read() _after update_ are caught as resolver errors',
 
   // Confirm there are no initial errors
   expect(snapshot.fieldErrors).toEqual(null);
-  const data: $FlowExpectedError = snapshot.data;
+  const data: $FlowFixMe = snapshot.data;
   expect(data.counter_throws_when_odd).toBe(0);
 
   // This should trigger a read that throws
@@ -320,7 +320,7 @@ test('Errors thrown during read() _after update_ are caught as resolver errors',
       handled: false,
     },
   ]);
-  const nextData: $FlowExpectedError = nextSnapshot.data;
+  const nextData: $FlowFixMe = nextSnapshot.data;
   expect(nextData.counter_throws_when_odd).toBe(null);
 
   handler.mockReset();
@@ -332,7 +332,7 @@ test('Errors thrown during read() _after update_ are caught as resolver errors',
 
   // Confirm there are no initial errors
   expect(finalSnapshot.fieldErrors).toEqual(null);
-  const finalData: $FlowExpectedError = finalSnapshot.data;
+  const finalData: $FlowFixMe = finalSnapshot.data;
   expect(finalData.counter_throws_when_odd).toBe(2);
 });