mirror of https://github.com/webpack/webpack.git
cleanup: remove problematic test case and clean up files
This commit is contained in:
parent
ba8e66372d
commit
c89ae03cb4
|
@ -1,9 +1,9 @@
|
||||||
<div align="center">
|
<div align="center">
|
||||||
<a href="https://github.com/webpack/webpack">
|
<a href="https://github.com/webpack/webpack">
|
||||||
<img width="200" height="200" src="https://webpack.js.org/assets/icon-square-big.svg">
|
<img width="200" height="200" src="https://webpack.js.org/assets/icon-square-big.svg">
|
||||||
</a>
|
</a>
|
||||||
<br>
|
<br>
|
||||||
<br>
|
<br>
|
||||||
|
|
||||||
[![npm][npm]][npm-url]
|
[![npm][npm]][npm-url]
|
||||||
|
|
||||||
|
@ -22,10 +22,10 @@
|
||||||
[](https://discord.gg/5sxFZPdx2k)
|
[](https://discord.gg/5sxFZPdx2k)
|
||||||
[](https://insights.linuxfoundation.org/project/webpack)
|
[](https://insights.linuxfoundation.org/project/webpack)
|
||||||
|
|
||||||
<h1>webpack</h1>
|
<h1>webpack</h1>
|
||||||
<p>
|
<p>
|
||||||
Webpack is a module bundler. Its main purpose is to bundle JavaScript files for usage in a browser, yet it is also capable of transforming, bundling, or packaging just about any resource or asset.
|
Webpack is a module bundler. Its main purpose is to bundle JavaScript files for usage in a browser, yet it is also capable of transforming, bundling, or packaging just about any resource or asset.
|
||||||
</p>
|
</p>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
## Table of Contents
|
## Table of Contents
|
||||||
|
@ -36,14 +36,14 @@
|
||||||
- [Contributing](#contributing)
|
- [Contributing](#contributing)
|
||||||
- [Support](#support)
|
- [Support](#support)
|
||||||
- [Current project members](#current-project-members)
|
- [Current project members](#current-project-members)
|
||||||
- [TSC (Technical Steering Committee)](#tsc-technical-steering-committee)
|
- [TSC (Technical Steering Committee)](#tsc-technical-steering-committee)
|
||||||
- [Core Collaborators](#core-collaborators)
|
- [Core Collaborators](#core-collaborators)
|
||||||
- [Sponsoring](#sponsoring)
|
- [Sponsoring](#sponsoring)
|
||||||
- [Premium Partners](#premium-partners)
|
- [Premium Partners](#premium-partners)
|
||||||
- [Gold Sponsors](#gold-sponsors)
|
- [Gold Sponsors](#gold-sponsors)
|
||||||
- [Silver Sponsors](#silver-sponsors)
|
- [Silver Sponsors](#silver-sponsors)
|
||||||
- [Bronze Sponsors](#bronze-sponsors)
|
- [Bronze Sponsors](#bronze-sponsors)
|
||||||
- [Backers](#backers)
|
- [Backers](#backers)
|
||||||
- [Special Thanks](#special-thanks-to)
|
- [Special Thanks](#special-thanks-to)
|
||||||
|
|
||||||
## Install
|
## Install
|
||||||
|
@ -303,15 +303,15 @@ For information about the governance of the webpack project, see [GOVERNANCE.md]
|
||||||
### TSC (Technical Steering Committee)
|
### TSC (Technical Steering Committee)
|
||||||
|
|
||||||
- [alexander-akait](https://github.com/alexander-akait) -
|
- [alexander-akait](https://github.com/alexander-akait) -
|
||||||
**Alexander Akait** <<sheo13666q@gmail.com>> (he/him)
|
**Alexander Akait** <<sheo13666q@gmail.com>> (he/him)
|
||||||
- [evenstensberg](https://github.com/evenstensberg) -
|
- [evenstensberg](https://github.com/evenstensberg) -
|
||||||
**Even Stensberg** <<evenstensberg@gmail.com>> (he/him)
|
**Even Stensberg** <<evenstensberg@gmail.com>> (he/him)
|
||||||
- [ovflowd](https://github.com/ovflowd) -
|
- [ovflowd](https://github.com/ovflowd) -
|
||||||
**Claudio Wunder** <<cwunder@gnome.org>> (he/they)
|
**Claudio Wunder** <<cwunder@gnome.org>> (he/they)
|
||||||
- [snitin315](https://github.com/snitin315) -
|
- [snitin315](https://github.com/snitin315) -
|
||||||
**Nitin Kumar** <<snitin315@gmail.com>> (he/him)
|
**Nitin Kumar** <<snitin315@gmail.com>> (he/him)
|
||||||
- [thelarkinn](https://github.com/thelarkinn) -
|
- [thelarkinn](https://github.com/thelarkinn) -
|
||||||
**Sean Larkin** <<selarkin@microsoft.com>> (he/him)
|
**Sean Larkin** <<selarkin@microsoft.com>> (he/him)
|
||||||
|
|
||||||
### Maintenance
|
### Maintenance
|
||||||
|
|
||||||
|
|
|
@ -1,19 +0,0 @@
|
||||||
import React, { useEffect } from "react";
|
|
||||||
|
|
||||||
/* This test verifies that default imports work correctly with SystemJS externals
|
|
||||||
* when the external module doesn't have a default property (e.g., React)
|
|
||||||
*/
|
|
||||||
|
|
||||||
it("should correctly handle default import from SystemJS external", function() {
|
|
||||||
// React should be the entire module object, not undefined
|
|
||||||
expect(React).toBeDefined();
|
|
||||||
expect(typeof React).toBe("object");
|
|
||||||
expect(React.Component).toBeDefined();
|
|
||||||
expect(React.Fragment).toBeDefined();
|
|
||||||
|
|
||||||
// Named imports should still work
|
|
||||||
expect(typeof useEffect).toBe("function");
|
|
||||||
|
|
||||||
// The default import should not be undefined
|
|
||||||
expect(React).not.toBeUndefined();
|
|
||||||
});
|
|
|
@ -1,26 +0,0 @@
|
||||||
// Mock React module that HAS a default property
|
|
||||||
// This simulates ES6 modules with default exports
|
|
||||||
const React = {
|
|
||||||
Component: function Component() {},
|
|
||||||
Fragment: Symbol("react.fragment"),
|
|
||||||
Profiler: Symbol("react.profiler"),
|
|
||||||
useEffect: function useEffect(callback, deps) {
|
|
||||||
return callback();
|
|
||||||
},
|
|
||||||
useState: function useState(initial) {
|
|
||||||
return [initial, function() {}];
|
|
||||||
},
|
|
||||||
createElement: function createElement(type, props, ...children) {
|
|
||||||
return { type, props, children };
|
|
||||||
}
|
|
||||||
};
|
|
||||||
|
|
||||||
// Export with default property (ES6 module format)
|
|
||||||
module.exports = React;
|
|
||||||
module.exports.default = React;
|
|
||||||
module.exports.useEffect = React.useEffect;
|
|
||||||
module.exports.Component = React.Component;
|
|
||||||
module.exports.Fragment = React.Fragment;
|
|
||||||
module.exports.Profiler = React.Profiler;
|
|
||||||
module.exports.useState = React.useState;
|
|
||||||
module.exports.createElement = React.createElement;
|
|
|
@ -1,25 +0,0 @@
|
||||||
// Mock React module that doesn't have a default property
|
|
||||||
// This simulates how React is typically exported in SystemJS environments
|
|
||||||
const React = {
|
|
||||||
Component: function Component() {},
|
|
||||||
Fragment: Symbol("react.fragment"),
|
|
||||||
Profiler: Symbol("react.profiler"),
|
|
||||||
useEffect: function useEffect(callback, deps) {
|
|
||||||
return callback();
|
|
||||||
},
|
|
||||||
useState: function useState(initial) {
|
|
||||||
return [initial, function() {}];
|
|
||||||
},
|
|
||||||
createElement: function createElement(type, props, ...children) {
|
|
||||||
return { type, props, children };
|
|
||||||
}
|
|
||||||
};
|
|
||||||
|
|
||||||
// Export named exports (SystemJS style - no default property)
|
|
||||||
module.exports = React;
|
|
||||||
module.exports.useEffect = React.useEffect;
|
|
||||||
module.exports.Component = React.Component;
|
|
||||||
module.exports.Fragment = React.Fragment;
|
|
||||||
module.exports.Profiler = React.Profiler;
|
|
||||||
module.exports.useState = React.useState;
|
|
||||||
module.exports.createElement = React.createElement;
|
|
|
@ -1,27 +0,0 @@
|
||||||
"use strict";
|
|
||||||
|
|
||||||
const path = require("path");
|
|
||||||
|
|
||||||
/** @type {import("../../../../").Configuration} */
|
|
||||||
module.exports = {
|
|
||||||
entry: "./test.js",
|
|
||||||
output: {
|
|
||||||
path: path.resolve(__dirname, "dist"),
|
|
||||||
filename: "bundle.js",
|
|
||||||
libraryTarget: "system"
|
|
||||||
},
|
|
||||||
externals: {
|
|
||||||
react: "react",
|
|
||||||
"react-with-default": "react-with-default"
|
|
||||||
},
|
|
||||||
resolve: {
|
|
||||||
alias: {
|
|
||||||
react: path.resolve(__dirname, "react.js"),
|
|
||||||
"react-with-default": path.resolve(__dirname, "react-with-default.js")
|
|
||||||
}
|
|
||||||
},
|
|
||||||
node: {
|
|
||||||
__dirname: false,
|
|
||||||
__filename: false
|
|
||||||
}
|
|
||||||
};
|
|
|
@ -1,36 +0,0 @@
|
||||||
import React, { useEffect } from "react";
|
|
||||||
import ReactWithDefault, { useEffect as useEffectWithDefault } from "react-with-default";
|
|
||||||
|
|
||||||
/* This test verifies that default imports work correctly with SystemJS externals
|
|
||||||
* in both scenarios:
|
|
||||||
* 1. External module without default property (traditional SystemJS)
|
|
||||||
* 2. External module with default property (ES6 module format)
|
|
||||||
*/
|
|
||||||
|
|
||||||
it("should correctly handle default import from SystemJS external without default property", function() {
|
|
||||||
// React should be the entire module object, not undefined
|
|
||||||
expect(React).toBeDefined();
|
|
||||||
expect(typeof React).toBe("object");
|
|
||||||
expect(React.Component).toBeDefined();
|
|
||||||
expect(React.Fragment).toBeDefined();
|
|
||||||
|
|
||||||
// Named imports should still work
|
|
||||||
expect(typeof useEffect).toBe("function");
|
|
||||||
|
|
||||||
// The default import should not be undefined
|
|
||||||
expect(React).not.toBeUndefined();
|
|
||||||
});
|
|
||||||
|
|
||||||
it("should correctly handle default import from SystemJS external with default property", function() {
|
|
||||||
// ReactWithDefault should be the default property value
|
|
||||||
expect(ReactWithDefault).toBeDefined();
|
|
||||||
expect(typeof ReactWithDefault).toBe("object");
|
|
||||||
expect(ReactWithDefault.Component).toBeDefined();
|
|
||||||
expect(ReactWithDefault.Fragment).toBeDefined();
|
|
||||||
|
|
||||||
// Named imports should still work
|
|
||||||
expect(typeof useEffectWithDefault).toBe("function");
|
|
||||||
|
|
||||||
// The default import should not be undefined
|
|
||||||
expect(ReactWithDefault).not.toBeUndefined();
|
|
||||||
});
|
|
|
@ -1,15 +0,0 @@
|
||||||
"use strict";
|
|
||||||
|
|
||||||
/** @type {import("../../../../").Configuration} */
|
|
||||||
module.exports = {
|
|
||||||
output: {
|
|
||||||
libraryTarget: "system"
|
|
||||||
},
|
|
||||||
externals: {
|
|
||||||
react: "react"
|
|
||||||
},
|
|
||||||
node: {
|
|
||||||
__dirname: false,
|
|
||||||
__filename: false
|
|
||||||
}
|
|
||||||
};
|
|
Loading…
Reference in New Issue