Skip to content

Commit d2363f9

Browse files
committed
fix: include dev files in format:check and format:fix scripts
Signed-off-by: Denis Golovin <[email protected]>
1 parent c46e702 commit d2363f9

File tree

5 files changed

+26
-15
lines changed

5 files changed

+26
-15
lines changed

package.json

+2-2
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,8 @@
3434
"scripts": {
3535
"build": "rollup --bundleConfigAsCjs --config rollup.config.js --compact --environment BUILD:production && node ./scripts/build.js",
3636
"watch": "rollup --bundleConfigAsCjs --config rollup.config.js -w",
37-
"format:check": "prettier --end-of-line auto --cache --check \"{src,types,scripts}/**/*.{ts,js}\"",
38-
"format:fix": "prettier --cache --write \"{src,types,scripts}/**/*.{ts,js}\"",
37+
"format:check": "prettier --end-of-line auto --cache --check \"**/*.{ts,js}\"",
38+
"format:fix": "prettier --cache --write \"**/*.{ts,js}\"",
3939
"lint:clean": "rimraf .eslintcache",
4040
"lint:fix": "node --max-old-space-size=6144 node_modules/eslint/bin/eslint.js --cache . --fix --ext js,ts",
4141
"lint:check": "node --max-old-space-size=6144 node_modules/eslint/bin/eslint.js --cache . --ext js,ts",

rollup.config.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,6 @@ export default {
3434
typescript(),
3535
commonjs({ extensions: ['.js', '.ts'] }), // the ".ts" extension is required],
3636
json(),
37-
nodeResolve({preferBuiltins: true}),
37+
nodeResolve({ preferBuiltins: true }),
3838
],
3939
};

tests/src/setupFiles/extended-hooks.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
* SPDX-License-Identifier: Apache-2.0
1717
***********************************************************************/
1818

19-
import { type RunnerTestContext,takeScreenshotHook } from '@podman-desktop/tests-playwright';
19+
import { type RunnerTestContext, takeScreenshotHook } from '@podman-desktop/tests-playwright';
2020
import { afterEach } from 'vitest';
2121

2222
afterEach(async (context: RunnerTestContext) => {

tests/src/sso-extension.spec.ts

+20-10
Original file line numberDiff line numberDiff line change
@@ -15,11 +15,17 @@
1515
*
1616
* SPDX-License-Identifier: Apache-2.0
1717
***********************************************************************/
18-
import type { Page} from '@playwright/test';
18+
import type { Page } from '@playwright/test';
1919
import { expect as playExpect } from '@playwright/test';
20-
import type { RunnerTestContext} from '@podman-desktop/tests-playwright';
21-
import { AuthenticationPage, ExtensionCardPage, NavigationBar, PodmanDesktopRunner, WelcomePage } from '@podman-desktop/tests-playwright';
22-
import { afterAll, beforeAll, beforeEach,describe, test } from 'vitest';
20+
import type { RunnerTestContext } from '@podman-desktop/tests-playwright';
21+
import {
22+
AuthenticationPage,
23+
ExtensionCardPage,
24+
NavigationBar,
25+
PodmanDesktopRunner,
26+
WelcomePage,
27+
} from '@podman-desktop/tests-playwright';
28+
import { afterAll, beforeAll, beforeEach, describe, test } from 'vitest';
2329

2430
import { SSOExtensionPage } from './model/pages/sso-extension-page';
2531

@@ -35,7 +41,6 @@ const authProviderName = 'Red Hat SSO';
3541
const activeExtensionStatus = 'ACTIVE';
3642
const disabledExtensionStatus = 'DISABLED';
3743

38-
3944
beforeEach<RunnerTestContext>(async ctx => {
4045
ctx.pdRunner = pdRunner;
4146
});
@@ -56,7 +61,6 @@ afterAll(async () => {
5661
});
5762

5863
describe('Red Hat Authentication extension verification', async () => {
59-
6064
test('Go to extensions and check if extension is already installed', async () => {
6165
const extensions = await navBar.openExtensions();
6266
if (await extensions.extensionIsInstalled(extensionLabel)) {
@@ -99,8 +103,12 @@ describe('Red Hat Authentication extension verification', async () => {
99103
const authPage = await settingsBar.openTabPage(AuthenticationPage);
100104
await playExpect(authPage.heading).toHaveText('Authentication');
101105
const provider = authPage.getProvider(authProviderName);
102-
await playExpect(provider.getByLabel('Provider Information').getByLabel('Provider Name')).toHaveText(authProviderName);
103-
await playExpect(provider.getByLabel('Provider Information').getByLabel('Provider Status')).toHaveText('Logged out');
106+
await playExpect(provider.getByLabel('Provider Information').getByLabel('Provider Name')).toHaveText(
107+
authProviderName,
108+
);
109+
await playExpect(provider.getByLabel('Provider Information').getByLabel('Provider Status')).toHaveText(
110+
'Logged out',
111+
);
104112
await playExpect(provider.getByLabel('Provider Actions').getByRole('button')).toContainText('Sign in');
105113
});
106114

@@ -132,7 +140,7 @@ describe('Red Hat Authentication extension verification', async () => {
132140
const authPage = await settingsBar.openTabPage(AuthenticationPage);
133141
await playExpect(authPage.heading).toHaveText('Authentication');
134142
await playExpect(authPage.getProvider(authProviderName)).toHaveCount(1);
135-
});
143+
});
136144
});
137145

138146
test('SSO extension can be removed', async () => {
@@ -145,5 +153,7 @@ async function removeExtension(): Promise<void> {
145153
const extensionCard = await extensions.getInstalledExtension(extensionLabelName, extensionLabel);
146154
await extensionCard.disableExtension();
147155
await extensionCard.removeExtension();
148-
await playExpect.poll(async () => await extensions.extensionIsInstalled(extensionLabel), { timeout: 15000 }).toBeFalsy();
156+
await playExpect
157+
.poll(async () => await extensions.extensionIsInstalled(extensionLabel), { timeout: 15000 })
158+
.toBeFalsy();
149159
}

vitest.config.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,8 @@ const excludeArray = [
3737
];
3838

3939
export function coverageConfig(packageRoot, packageName) {
40-
const obj = { coverage: {
40+
const obj = {
41+
coverage: {
4142
all: true,
4243
clean: true,
4344
src: [packageRoot],

0 commit comments

Comments
 (0)