Skip to content

Fixes webpack-manifest-plugin issue #3

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.

Sign up for GitHub

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

Closed
wants to merge 20 commits into from
Closed

Fixes webpack-manifest-plugin issue #3

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
493a379
Map (css|sass|scss) modules to identity-obj-proxy in jest (#4419)
kusold May 16, 2018
966ae83
Merge pull request #1 from facebook/next
May 16, 2018
543ae53
disable module concatenation
May 16, 2018
2cbb719
Merge branch 'webpack4' of https://github.com/bugzpodder/create-react…
May 16, 2018
2983c23
fix cleanup
May 17, 2018
918f549
fix optimization
May 17, 2018
327f334
restore tmp removal
May 17, 2018
5528eda
preserve
May 19, 2018
ed43e5c
fix css tests
May 19, 2018
8074dbf
fix test run
May 19, 2018
f583cdf
fix svg test
May 19, 2018
9692972
revert tmp dir
May 19, 2018
9c98866
fix import
May 20, 2018
ad59f59
fix import
May 20, 2018
886d21f
help debugging
May 20, 2018
e231801
fix tests
May 20, 2018
6e83944
fix sass test
May 20, 2018
2c5be47
restore cleanup
May 20, 2018
d72678f
- [x] Upgrade to webpack 4.8.X (#4077)
andriijas May 20, 2018
52f08e6
Merge branch 'next' into webpack4
gaearon May 20, 2018
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
Prev Previous commit
Next Next commit
fix css tests
  • Loading branch information
Jack Zhao committed May 19, 2018
commit ed43e5c0f3950bd6494875d5b6f30a43fe11ed77
19 changes: 6 additions & 13 deletions packages/react-scripts/fixtures/kitchensink/integration/initDOM.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ const path = require('path');
const { expect } = require('chai');

let getMarkup;
let resourceLoader;
export let resourceLoader;

if (process.env.E2E_FILE) {
const file = path.isAbsolute(process.env.E2E_FILE)
Expand Down Expand Up @@ -47,25 +47,18 @@ if (process.env.E2E_FILE) {

resourceLoader = (resource, callback) => resource.defaultFetch(callback);
} else {
it.only(
'can run jsdom (at least one of "E2E_FILE" or "E2E_URL" environment variables must be provided)',
() => {
expect(
new Error("This isn't the error you are looking for.")
).to.be.undefined();
}
);
it.only('can run jsdom (at least one of "E2E_FILE" or "E2E_URL" environment variables must be provided)', () => {
expect(
new Error("This isn't the error you are looking for.")
).to.be.undefined();
});
}

export default feature =>
new Promise(async resolve => {
const markup = await getMarkup();
const host = process.env.E2E_URL || 'http://www.example.org/spa:3000';
const doc = jsdom.jsdom(markup, {
features: {
FetchExternalResources: ['script', 'css'],
ProcessExternalResources: ['script'],
},
created: (_, win) =>
win.addEventListener('ReactFeatureDidMount', () => resolve(doc), true),
deferClose: true,
Expand Down
110 changes: 63 additions & 47 deletions packages/react-scripts/fixtures/kitchensink/integration/webpack.test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,77 +6,89 @@
*/

import { expect } from 'chai';
import initDOM from './initDOM';
import initDOM, { resourceLoader } from './initDOM';

describe('Integration', () => {
describe('Webpack plugins', () => {
it('css inclusion', async () => {
const doc = await initDOM('css-inclusion');

expect(
doc.getElementsByTagName('style')[0].textContent.replace(/\s/g, '')
).to.match(/html\{/);
expect(
doc.getElementsByTagName('style')[1].textContent.replace(/\s/g, '')
).to.match(/#feature-css-inclusion\{background:.+;color:.+}/);
resourceLoader(
{ url: url.parse(doc.getElementsByTagName('link')[1].href) },
(_, textContent) => {
expect(textContent).to.match(/html\{/);
expect(textContent).to.match(
/#feature-css-inclusion\{background:.+;color:.+}/
);
}
);
});

it('css modules inclusion', async () => {
const doc = await initDOM('css-modules-inclusion');

expect(
doc.getElementsByTagName('style')[0].textContent.replace(/\s/g, '')
).to.match(/.+style_cssModulesInclusion__.+\{background:.+;color:.+}/);
expect(
doc.getElementsByTagName('style')[1].textContent.replace(/\s/g, '')
).to.match(
/.+assets_cssModulesIndexInclusion__.+\{background:.+;color:.+}/
resourceLoader(
{ url: url.parse(doc.getElementsByTagName('link')[2].href) },
(_, textContent) => {
expect(textContent).to.match(
/.+style_cssModulesInclusion__.+\{background:.+;color:.+}/
);
expect(textContent).to.match(
/.+assets_cssModulesIndexInclusion__.+\{background:.+;color:.+}/
);
}
);
});

it('scss inclusion', async () => {
const doc = await initDOM('scss-inclusion');

expect(
doc.getElementsByTagName('style')[0].textContent.replace(/\s/g, '')
).to.match(/#feature-scss-inclusion\{background:.+;color:.+}/);
resourceLoader(
{ url: url.parse(doc.getElementsByTagName('link')[2].href) },
(_, textContent) => {
expect(textContent).to.match(
/#feature-scss-inclusion\{background:.+;color:.+}/
);
}
);
});

it('scss modules inclusion', async () => {
const doc = await initDOM('scss-modules-inclusion');

expect(
doc.getElementsByTagName('style')[0].textContent.replace(/\s/g, '')
).to.match(
/.+scss-styles_scssModulesInclusion.+\{background:.+;color:.+}/
);
expect(
doc.getElementsByTagName('style')[1].textContent.replace(/\s/g, '')
).to.match(
/.+assets_scssModulesIndexInclusion.+\{background:.+;color:.+}/
resourceLoader(
{ url: url.parse(doc.getElementsByTagName('link')[2].href) },
(_, textContent) => {
expect(textContent).to.match(
/.+scss-styles_scssModulesInclusion.+\{background:.+;color:.+}/
);
expect(textContent).to.match(
/.+assets_scssModulesIndexInclusion.+\{background:.+;color:.+}/
);
}
);
});

it('sass inclusion', async () => {
const doc = await initDOM('sass-inclusion');

expect(
doc.getElementsByTagName('style')[0].textContent.replace(/\s/g, '')
).to.match(/#feature-sass-inclusion\{background:.+;color:.+}/);
resourceLoader(
{ url: url.parse(doc.getElementsByTagName('link')[2].href) },
(_, textContent) => {
expect(textContent).to.match(
/#feature-sass-inclusion\{background:.+;color:.+}/
);
}
);
});

it('sass modules inclusion', async () => {
const doc = await initDOM('sass-modules-inclusion');

expect(
doc.getElementsByTagName('style')[0].textContent.replace(/\s/g, '')
).to.match(
/.+sass-styles_sassModulesInclusion.+\{background:.+;color:.+}/
);
expect(
doc.getElementsByTagName('style')[1].textContent.replace(/\s/g, '')
).to.match(
/.+assets_sassModulesIndexInclusion.+\{background:.+;color:.+}/
resourceLoader(
{ url: url.parse(doc.getElementsByTagName('link')[2].href) },
(_, textContent) => {
expect(textContent).to.match(
/.+sass-styles_sassModulesInclusion.+\{background:.+;color:.+}/
);
expect(textContent).to.match(
/.+assets_sassModulesIndexInclusion.+\{background:.+;color:.+}/
);
}
);
});

Expand Down Expand Up @@ -124,9 +136,13 @@ describe('Integration', () => {

it('svg inclusion', async () => {
const doc = await initDOM('svg-inclusion');

expect(doc.getElementById('feature-svg-inclusion').src).to.match(
/\/static\/media\/logo\..+\.svg$/
resourceLoader(
{ url: url.parse(doc.getElementsByTagName('link')[2].href) },
(_, textContent) => {
expect(doc.getElementById('feature-svg-inclusion').src).to.match(
/\/static\/media\/logo\..+\.svg$/
);
}
);
});

Expand Down