From 13a8c4c7e6a4fd3e29521dc222f4ce893b1e1c1a Mon Sep 17 00:00:00 2001
From: Pascal Hartig <phartig@rdrei.net>
Date: Tue, 5 May 2020 06:56:50 -0700
Subject: [PATCH] Clean up datainspector test (#1101)

Summary:
Remove some unnecessary `await`s. Requires https://github.com/facebook/flipper/issues/1100.
Pull Request resolved: https://github.com/facebook/flipper/pull/1101

Test Plan: yarn test

Reviewed By: mweststrate

Differential Revision: D21401237

Pulled By: passy

fbshipit-source-id: 09d68964e8519612d643b83ea61d1cd231b7d169
---
 .../data-inspector/__tests__/DataInspector.node.tsx    | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/desktop/app/src/ui/components/data-inspector/__tests__/DataInspector.node.tsx b/desktop/app/src/ui/components/data-inspector/__tests__/DataInspector.node.tsx
index 8f3664e44..b05cb1f0f 100644
--- a/desktop/app/src/ui/components/data-inspector/__tests__/DataInspector.node.tsx
+++ b/desktop/app/src/ui/components/data-inspector/__tests__/DataInspector.node.tsx
@@ -62,7 +62,7 @@ const json = {
 test('changing collapsed property works', async () => {
   const res = render(<ManagedDataInspector data={json} collapsed expandRoot />);
   expect(await res.findByText(/is/)).toBeTruthy(); // from expandRoot
-  expect((await res.queryAllByText(/cool/)).length).toBe(0);
+  expect(res.queryAllByText(/cool/).length).toBe(0);
 
   res.rerender(
     <ManagedDataInspector data={json} collapsed={false} expandRoot />,
@@ -72,23 +72,23 @@ test('changing collapsed property works', async () => {
   res.rerender(
     <ManagedDataInspector data={json} collapsed={true} expandRoot />,
   );
-  expect((await res.queryAllByText(/cool/)).length).toBe(0);
+  expect(res.queryAllByText(/cool/).length).toBe(0);
 });
 
 test('can manually collapse properties', async () => {
   const res = render(<ManagedDataInspector data={json} collapsed expandRoot />);
 
   await res.findByText(/is/); // previewed as key, like: "data: {is, and}"
-  expect((await res.queryAllByText(/awesomely/)).length).toBe(0);
+  expect(res.queryAllByText(/awesomely/).length).toBe(0);
 
   // expand twice
   fireEvent.click(await res.findByText(/data/));
   await res.findByText(/awesomely/);
-  expect((await res.queryAllByText(/cool/)).length).toBe(0);
+  expect(res.queryAllByText(/cool/).length).toBe(0);
 
   fireEvent.click(await res.findByText(/is/));
   await res.findByText(/cool/);
-  expect((await res.queryAllByText(/json/)).length).toBe(0); // this node is not shown
+  expect(res.queryAllByText(/json/).length).toBe(0); // this node is not shown
 
   // collapsing everything again
   fireEvent.click(await res.findByText(/data/));