From 6745f2efd708f06c470086b37bcb8b2f5c77e117 Mon Sep 17 00:00:00 2001 From: Janne Savolainen Date: Tue, 12 Apr 2022 15:02:34 +0300 Subject: [PATCH] Fix snapshots/code-style after merging stuff to master (#5235) * Update snapshots after merging stuff to master Signed-off-by: Janne Savolainen * Fix code style Signed-off-by: Janne Savolainen --- ...gation-using-application-menu.test.ts.snap | 6 +- .../navigation-using-application-menu.test.ts | 3 +- .../order-of-sidebar-items.test.tsx.snap | 24 +++-- ...-and-tab-navigation-for-core.test.tsx.snap | 84 ++++++++++------ ...ab-navigation-for-extensions.test.tsx.snap | 96 ++++++++++++------- .../visibility-of-sidebar-items.test.tsx.snap | 24 +++-- .../cluster/order-of-sidebar-items.test.tsx | 6 +- ...debar-and-tab-navigation-for-core.test.tsx | 12 ++- ...and-tab-navigation-for-extensions.test.tsx | 3 +- .../visibility-of-sidebar-items.test.tsx | 3 +- ...-characters-in-page-registrations.test.tsx | 6 +- ...gation-using-application-menu.test.ts.snap | 6 +- .../navigation-using-application-menu.test.ts | 3 +- .../navigate-to-extension-page.test.tsx | 6 +- .../navigating-between-routes.test.tsx | 6 +- .../closing-preferences.test.tsx.snap | 28 ++++-- ...on-to-application-preferences.test.ts.snap | 12 ++- ...igation-to-editor-preferences.test.ts.snap | 12 ++- ...tension-specific-preferences.test.tsx.snap | 16 +++- ...ion-to-kubernetes-preferences.test.ts.snap | 20 ++-- ...vigation-to-proxy-preferences.test.ts.snap | 12 ++- ...ion-to-telemetry-preferences.test.tsx.snap | 24 +++-- ...ation-to-terminal-preferences.test.ts.snap | 12 ++- ...gation-using-application-menu.test.ts.snap | 6 +- .../preferences/closing-preferences.test.tsx | 6 +- ...igation-to-application-preferences.test.ts | 3 +- .../navigation-to-editor-preferences.test.ts | 3 +- ...to-extension-specific-preferences.test.tsx | 3 +- ...vigation-to-kubernetes-preferences.test.ts | 3 +- .../navigation-to-proxy-preferences.test.ts | 3 +- ...vigation-to-telemetry-preferences.test.tsx | 3 +- ...navigation-to-terminal-preferences.test.ts | 3 +- .../navigation-using-application-menu.test.ts | 6 +- .../navigation-using-application-menu.test.ts | 3 +- 34 files changed, 311 insertions(+), 155 deletions(-) diff --git a/src/behaviours/add-cluster/__snapshots__/navigation-using-application-menu.test.ts.snap b/src/behaviours/add-cluster/__snapshots__/navigation-using-application-menu.test.ts.snap index 221440818f..4c1203829a 100644 --- a/src/behaviours/add-cluster/__snapshots__/navigation-using-application-menu.test.ts.snap +++ b/src/behaviours/add-cluster/__snapshots__/navigation-using-application-menu.test.ts.snap @@ -5,7 +5,7 @@ exports[`add-cluster - navigation using application menu renders 1`] = `
` exports[`add-cluster - navigation using application menu when navigating to add cluster using application menu renders 1`] = `
diff --git a/src/behaviours/add-cluster/navigation-using-application-menu.test.ts b/src/behaviours/add-cluster/navigation-using-application-menu.test.ts index 7ff34c4d00..cc7749a7e9 100644 --- a/src/behaviours/add-cluster/navigation-using-application-menu.test.ts +++ b/src/behaviours/add-cluster/navigation-using-application-menu.test.ts @@ -4,7 +4,8 @@ */ import type { RenderResult } from "@testing-library/react"; -import { ApplicationBuilder, getApplicationBuilder } from "../../renderer/components/test-utils/get-application-builder"; +import type { ApplicationBuilder } from "../../renderer/components/test-utils/get-application-builder"; +import { getApplicationBuilder } from "../../renderer/components/test-utils/get-application-builder"; import isAutoUpdateEnabledInjectable from "../../main/is-auto-update-enabled.injectable"; // TODO: Make components free of side effects by making them deterministic diff --git a/src/behaviours/cluster/__snapshots__/order-of-sidebar-items.test.tsx.snap b/src/behaviours/cluster/__snapshots__/order-of-sidebar-items.test.tsx.snap index 4a5b4f1360..092337ec82 100644 --- a/src/behaviours/cluster/__snapshots__/order-of-sidebar-items.test.tsx.snap +++ b/src/behaviours/cluster/__snapshots__/order-of-sidebar-items.test.tsx.snap @@ -6,17 +6,21 @@ exports[`cluster - order of sidebar items when rendered renders 1`] = ` class="flex flex-col" data-testid="cluster-sidebar" > -
+
??
-
+
-
+
??
-
+
-
+
??
-
+
-
+
??
-
+
-
+
??
-
+
-
+
??
-
+
-
+
??
-
+
-
+
??
-
+
-
+
??
-
+
-
+
??
-
+
-
+
??
-
+
-
+
??
-
+
-
+
??
-
+
-
+
??
-
+
-
+
??
-
+
-
+
??
-
+
-
+
??
-
+
-
+
??
-
+
-
+
??
-
+
{ let applicationBuilder: ApplicationBuilder; diff --git a/src/behaviours/extensions/__snapshots__/navigation-using-application-menu.test.ts.snap b/src/behaviours/extensions/__snapshots__/navigation-using-application-menu.test.ts.snap index a60732c101..4878bbb0b7 100644 --- a/src/behaviours/extensions/__snapshots__/navigation-using-application-menu.test.ts.snap +++ b/src/behaviours/extensions/__snapshots__/navigation-using-application-menu.test.ts.snap @@ -20,7 +20,7 @@ exports[`extensions - navigation using application menu when navigating to exten Extensions

Add new features via Lens Extensions. @@ -96,10 +96,11 @@ exports[`extensions - navigation using application menu when navigating to exten

diff --git a/src/behaviours/extensions/navigation-using-application-menu.test.ts b/src/behaviours/extensions/navigation-using-application-menu.test.ts index 31fe918b20..ef714a75d3 100644 --- a/src/behaviours/extensions/navigation-using-application-menu.test.ts +++ b/src/behaviours/extensions/navigation-using-application-menu.test.ts @@ -4,7 +4,8 @@ */ import type { RenderResult } from "@testing-library/react"; -import { ApplicationBuilder, getApplicationBuilder } from "../../renderer/components/test-utils/get-application-builder"; +import type { ApplicationBuilder } from "../../renderer/components/test-utils/get-application-builder"; +import { getApplicationBuilder } from "../../renderer/components/test-utils/get-application-builder"; import isAutoUpdateEnabledInjectable from "../../main/is-auto-update-enabled.injectable"; import extensionsStoreInjectable from "../../extensions/extensions-store/extensions-store.injectable"; import type { ExtensionsStore } from "../../extensions/extensions-store/extensions-store"; diff --git a/src/behaviours/navigate-to-extension-page.test.tsx b/src/behaviours/navigate-to-extension-page.test.tsx index eb034defdf..915d3bab2b 100644 --- a/src/behaviours/navigate-to-extension-page.test.tsx +++ b/src/behaviours/navigate-to-extension-page.test.tsx @@ -2,9 +2,11 @@ * Copyright (c) OpenLens Authors. All rights reserved. * Licensed under MIT License. See LICENSE in root directory for more information. */ -import { getRendererExtensionFake, TestExtension } from "../renderer/components/test-utils/get-renderer-extension-fake"; +import type { TestExtension } from "../renderer/components/test-utils/get-renderer-extension-fake"; +import { getRendererExtensionFake } from "../renderer/components/test-utils/get-renderer-extension-fake"; import React from "react"; -import { fireEvent, RenderResult } from "@testing-library/react"; +import type { RenderResult } from "@testing-library/react"; +import { fireEvent } from "@testing-library/react"; import isEmpty from "lodash/isEmpty"; import queryParametersInjectable from "../renderer/routes/query-parameters.injectable"; import currentPathInjectable from "../renderer/routes/current-path.injectable"; diff --git a/src/behaviours/navigating-between-routes.test.tsx b/src/behaviours/navigating-between-routes.test.tsx index 152830e463..4cb55cd290 100644 --- a/src/behaviours/navigating-between-routes.test.tsx +++ b/src/behaviours/navigating-between-routes.test.tsx @@ -2,13 +2,15 @@ * Copyright (c) OpenLens Authors. All rights reserved. * Licensed under MIT License. See LICENSE in root directory for more information. */ -import { DiContainer, getInjectable } from "@ogre-tools/injectable"; +import type { DiContainer } from "@ogre-tools/injectable"; +import { getInjectable } from "@ogre-tools/injectable"; import React from "react"; import { computed } from "mobx"; import type { RenderResult } from "@testing-library/react"; import { routeSpecificComponentInjectionToken } from "../renderer/routes/route-specific-component-injection-token"; import { observer } from "mobx-react"; -import { Route, routeInjectionToken } from "../common/front-end-routing/route-injection-token"; +import type { Route } from "../common/front-end-routing/route-injection-token"; +import { routeInjectionToken } from "../common/front-end-routing/route-injection-token"; import type { ApplicationBuilder } from "../renderer/components/test-utils/get-application-builder"; import { getApplicationBuilder } from "../renderer/components/test-utils/get-application-builder"; import currentRouteInjectable from "../renderer/routes/current-route.injectable"; diff --git a/src/behaviours/preferences/__snapshots__/closing-preferences.test.tsx.snap b/src/behaviours/preferences/__snapshots__/closing-preferences.test.tsx.snap index 2fc1f4410f..ec4585cb42 100644 --- a/src/behaviours/preferences/__snapshots__/closing-preferences.test.tsx.snap +++ b/src/behaviours/preferences/__snapshots__/closing-preferences.test.tsx.snap @@ -306,7 +306,7 @@ exports[`preferences - closing-preferences given accessing preferences directly