Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(storybook): added storybook 7 testing suite #15733

Closed
wants to merge 1 commit into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .github/workflows/e2e-matrix.yml
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,7 @@ jobs:
- e2e-web
- e2e-rollup
- e2e-storybook
- e2e-storybook-7
- e2e-storybook-angular
- e2e-vite
- e2e-webpack
Expand Down Expand Up @@ -141,6 +142,8 @@ jobs:
codeowners: 'S04SJ6PL98X'
- project: e2e-storybook
codeowners: 'S04SVQ8H0G5'
- project: e2e-storybook-7
codeowners: 'S04SVQ8H0G5'
- project: e2e-storybook-angular
codeowners: 'S04SVQ8H0G5'
- project: e2e-vite
Expand Down
3 changes: 3 additions & 0 deletions .github/workflows/e2e-windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@ jobs:
- e2e-web
- e2e-rollup
- e2e-storybook
- e2e-storybook-7
- e2e-storybook-angular
- e2e-vite
- e2e-webpack
Expand Down Expand Up @@ -118,6 +119,8 @@ jobs:
codeowners: 'S04SJ6PL98X'
- project: e2e-storybook
codeowners: 'S04SVQ8H0G5'
- project: e2e-storybook-7
codeowners: 'S04SVQ8H0G5'
- project: e2e-storybook-angular
codeowners: 'S04SVQ8H0G5'
- project: e2e-vite
Expand Down
24 changes: 24 additions & 0 deletions docs/shared/mental-model/large-tasks.json
Original file line number Diff line number Diff line change
Expand Up @@ -36283,6 +36283,30 @@
},
"dependencies": { "e2e-storybook:run-e2e-tests": [] }
},
"e2e-storybook-7:e2e": {
"roots": ["e2e-storybook-7:e2e"],
"tasks": {
"e2e-storybook-7:e2e": {
"id": "e2e-storybook-7:e2e",
"target": { "project": "e2e-storybook-7", "target": "e2e" },
"projectRoot": "e2e/storybook-7",
"overrides": {}
}
},
"dependencies": { "e2e-storybook-7:e2e": [] }
},
"e2e-storybook-7:run-e2e-tests": {
"roots": ["e2e-storybook-7:run-e2e-tests"],
"tasks": {
"e2e-storybook-7:run-e2e-tests": {
"id": "e2e-storybook-7:run-e2e-tests",
"target": { "project": "e2e-storybook-7", "target": "run-e2e-tests" },
"projectRoot": "e2e/storybook-7",
"overrides": {}
}
},
"dependencies": { "e2e-storybook-7:run-e2e-tests": [] }
},
"nx-dev:build": {
"roots": ["graph-client:build-base:release"],
"tasks": {
Expand Down
11 changes: 11 additions & 0 deletions e2e/storybook-7/jest.config.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
/* eslint-disable */
export default {
transform: {
'^.+\\.[tj]sx?$': ['ts-jest', { tsconfig: '<rootDir>/tsconfig.spec.json' }],
},
moduleFileExtensions: ['ts', 'tsx', 'js', 'jsx', 'html'],
maxWorkers: 1,
globals: {},
displayName: 'e2e-storybook-7',
preset: '../../jest.preset.js',
};
11 changes: 11 additions & 0 deletions e2e/storybook-7/project.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
{
"name": "e2e-storybook-7",
"$schema": "../../node_modules/nx/schemas/project-schema.json",
"sourceRoot": "e2e/storybook-7",
"projectType": "application",
"targets": {
"e2e": {},
"run-e2e-tests": {}
},
"implicitDependencies": ["storybook"]
}
129 changes: 129 additions & 0 deletions e2e/storybook-7/src/storybook-nested.test.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,129 @@
import {
checkFilesExist,
cleanupProject,
killPorts,
readJson,
runCLI,
runCommandUntil,
runCreateWorkspace,
tmpProjPath,
uniq,
} from '@nrwl/e2e/utils';
import { writeFileSync } from 'fs';

describe('Storybook generators and executors for standalone workspaces - using React + Vite', () => {
const wsName = uniq('react');
const appName = uniq('app');

beforeAll(() => {
// create a workspace with a single react app at the root
runCreateWorkspace(wsName, {
preset: 'react-standalone',
appName,
style: 'css',
bundler: 'vite',
});

runCLI(
`generate @nrwl/react:storybook-configuration --storybook7betaConfiguration ${appName} --generateStories --no-interactive --verbose`
);

runCLI(`report`);
});

afterAll(() => {
cleanupProject();
});

describe('Storybook generated files', () => {
it('should generate storybook files', () => {
checkFilesExist(
'.storybook/main.js',
'.storybook/preview.js',
'.storybook/tsconfig.json'
);
});

it('should edit root tsconfig.json', () => {
const tsconfig = readJson(`tsconfig.json`);
expect(tsconfig['ts-node']?.compilerOptions?.module).toEqual('commonjs');
});
});

describe('serve storybook', () => {
afterEach(() => killPorts());

it('should serve a React based Storybook setup that uses Vite', async () => {
const p = await runCommandUntil(`run ${appName}:storybook`, (output) => {
return /Storybook.*started/gi.test(output);
});
p.kill();
}, 200_000);
});

describe('build storybook', () => {
it('should build a React based storybook that uses Vite', () => {
runCLI(`run ${appName}:build-storybook --verbose`);
checkFilesExist(`dist/storybook/${appName}/index.html`);
}, 200_000);

// This test makes sure path resolution works
it('should build a React based storybook that references another lib and uses Vite', () => {
const reactLib = uniq('test-lib-react');
runCLI(`generate @nrwl/react:lib ${reactLib} --no-interactive`);
// create a React component we can reference
writeFileSync(
tmpProjPath(`${reactLib}/src/lib/mytestcmp.tsx`),
`
import React from 'react';

/* eslint-disable-next-line */
export interface MyTestCmpProps {}

export const MyTestCmp = (props: MyTestCmpProps) => {
return (
<div>
<h1>Welcome to test cmp!</h1>
</div>
);
};

export default MyTestCmp;
`
);
// update index.ts and export it
writeFileSync(
tmpProjPath(`${reactLib}/src/index.ts`),
`
export * from './lib/mytestcmp';
`
);

// create a story in the first lib to reference the cmp from the 2nd lib
writeFileSync(
tmpProjPath(`src/lib/myteststory.stories.tsx`),
`
import React from 'react';

import { MyTestCmp, MyTestCmpProps } from '@${wsName}/${reactLib}';

export default {
component: MyTestCmp,
title: 'MyTestCmp',
};

export const primary = () => {
/* eslint-disable-next-line */
const props: MyTestCmpProps = {};

return <MyTestCmp />;
};
`
);

// build React lib
runCLI(`run ${appName}:build-storybook --verbose`);
checkFilesExist(`dist/storybook/${appName}/index.html`);
}, 40000);
});
});
102 changes: 102 additions & 0 deletions e2e/storybook-7/src/storybook.test.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,102 @@
import {
checkFilesExist,
cleanupProject,
killPorts,
runCLI,
runCommandUntil,
tmpProjPath,
uniq,
newProject,
} from '@nrwl/e2e/utils';
import { writeFileSync } from 'fs';

describe('Storybook 7 generators and executors for monorepos', () => {
const reactStorybookLib = uniq('test-ui-lib-react');
let proj;
beforeAll(() => {
proj = newProject();
runCLI(`generate @nrwl/react:lib ${reactStorybookLib} --no-interactive`);
runCLI(
`generate @nrwl/react:storybook-configuration ${reactStorybookLib} --generateStories --no-interactive --bundler=webpack --storybook7betaConfiguration --verbose`
);
});

afterAll(() => {
cleanupProject();
});

describe('serve and build storybook', () => {
afterEach(() => killPorts());

it('should serve a React based Storybook setup that uses webpack', async () => {
// serve the storybook
const p = await runCommandUntil(
`run ${reactStorybookLib}:storybook`,
(output) => {
return /Storybook.*started/gi.test(output);
}
);
p.kill();
}, 200_000);

it('should build a React based storybook setup that uses webpack', () => {
// build
runCLI(`run ${reactStorybookLib}:build-storybook --verbose`);
checkFilesExist(`dist/storybook/${reactStorybookLib}/index.html`);
}, 200_000);

// This test makes sure path resolution works
it('should build a React based storybook that references another lib and uses webpack', () => {
const anotherReactLib = uniq('test-another-lib-react');
runCLI(`generate @nrwl/react:lib ${anotherReactLib} --no-interactive`);
// create a React component we can reference
writeFileSync(
tmpProjPath(`libs/${anotherReactLib}/src/lib/mytestcmp.tsx`),
`
export function MyTestCmp() {
return (
<div>
<h1>Welcome to OtherLib!</h1>
</div>
);
}

export default MyTestCmp;
`
);
// update index.ts and export it
writeFileSync(
tmpProjPath(`libs/${anotherReactLib}/src/index.ts`),
`
export * from './lib/mytestcmp';
`
);

// create a story in the first lib to reference the cmp from the 2nd lib
writeFileSync(
tmpProjPath(
`libs/${reactStorybookLib}/src/lib/myteststory.stories.tsx`
),
`
import type { Meta } from '@storybook/react';
import { MyTestCmp } from '@${proj}/${anotherReactLib}';

const Story: Meta<typeof MyTestCmp> = {
component: MyTestCmp,
title: 'MyTestCmp',
};
export default Story;

export const Primary = {
args: {},
};

`
);

// build React lib
runCLI(`run ${reactStorybookLib}:build-storybook --verbose`);
checkFilesExist(`dist/storybook/${reactStorybookLib}/index.html`);
}, 200_000);
});
});
13 changes: 13 additions & 0 deletions e2e/storybook-7/tsconfig.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
{
"extends": "../../tsconfig.base.json",
"compilerOptions": {
"types": ["node", "jest"]
},
"include": [],
"files": [],
"references": [
{
"path": "./tsconfig.spec.json"
}
]
}
20 changes: 20 additions & 0 deletions e2e/storybook-7/tsconfig.spec.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
{
"extends": "./tsconfig.json",
"compilerOptions": {
"outDir": "../../dist/out-tsc",
"module": "commonjs",
"types": ["jest", "node"]
},
"include": [
"**/*.test.ts",
"**/*.spec.ts",
"**/*.spec.tsx",
"**/*.test.tsx",
"**/*.spec.js",
"**/*.test.js",
"**/*.spec.jsx",
"**/*.test.jsx",
"**/*.d.ts",
"jest.config.ts"
]
}
Loading