4832 / Skipped test (#2210)

* initial test to see if format number test runs  in CI

* remove only modifier in test

* redo line chart test

* fix import

* fix notional test

* display pop up in web

* remove snapshot

* remove PNL snapshot test

* test set filter

* initial date filter test

* handle date filter

* Quick test across columns

src/client/src/App/Analytics/PnL/__tests__/PNLBar.test.tsx

@@ -27,17 +27,4 @@ describe("PNLBar", () => {
 
     expect(screen.getByTestId("priceLabel").textContent).toBe(`-26,043.69`)
   })
-
-  it("renders the bar correctly", () => {
-    const { container } = render(
-      <TestThemeProvider>
-        <PNLBar
-          symbol={"EURAUD"}
-          basePnl={-26043.691207338878}
-          maxVal={1239939.1231111237}
-        />
-      </TestThemeProvider>,
-    )
-    expect(container.firstChild).toMatchSnapshot()
-  })
 })

src/client/src/App/Analytics/Positions/__tests__/Positions.test.tsx

@@ -119,33 +119,4 @@ describe("Positions", () => {
 
     expect(screen.getByTestId("tooltip").textContent).toBe("AUD -1,557,031")
   })
-
-  // TODO (5350): fix test failure - Windows snapshot incompatible with Mac
-  it.skip("should display the correct bubble chart", async () => {
-    const positionMock$ = new BehaviorSubject<
-      Record<string, CurrencyPairPosition>
-    >(positionMock)
-    analyticsMock.__setPositionMock(positionMock$)
-
-    const subscription = positions$.subscribe()
-    const { container } = render(
-      <TestThemeProvider>
-        <Subscribe source$={positions$} fallback="No data">
-          <Positions />
-        </Subscribe>
-      </TestThemeProvider>,
-    )
-
-    expect(container.firstChild).toMatchSnapshot()
-
-    act(() => {
-      positionMock$.next(positionMock2)
-    })
-
-    await new Promise((res) => setTimeout(res, 2200))
-
-    expect(container.firstChild).toMatchSnapshot()
-
-    subscription.unsubscribe()
-  })
 })