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.

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

Jump to bottom

Avoid resolveTargets call if browsers is an empty array #14294

Merged
merged 15 commits into from Mar 14, 2022
Merged

Avoid resolveTargets call if browsers is an empty array #14294

Changes from 2 commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
bdc7fdf
fix: remove `resolveTargets` call if browsers is empty
dev-itsheng Feb 22, 2022
bba433a
Merge branch 'babel:main' into patch-1
dev-itsheng Mar 1, 2022
c03d061
fix: added validation for `browsers` values `[]` and `undefined`
dev-itsheng Mar 2, 2022
32c3aa7
Merge branch 'patch-1' of https://github.com/dev-itsheng/babel
dev-itsheng Mar 2, 2022
0183f3f
Merge branch 'babel:main' into patch-1
dev-itsheng Mar 4, 2022
8e9453f
fix: added some comment of changed code.
dev-itsheng Mar 4, 2022
426d2c7
Merge branch 'patch-1' of https://github.com/dev-itsheng/babel
dev-itsheng Mar 4, 2022
3eea970
fix: fixed some test preconditions, and added a test to see if a func…
dev-itsheng Mar 13, 2022
6944235
Merge branch 'babel:main' into patch-1
dev-itsheng Mar 13, 2022
6a4dab1
Merge branch 'patch-1' of https://github.com/dev-itsheng/babel
dev-itsheng Mar 13, 2022
57669c6
fix: Fix test error
dev-itsheng Mar 13, 2022
a3fe073
fix: Fix code style
dev-itsheng Mar 13, 2022
2d531e5
fix: Fix ESLint error of code style
dev-itsheng Mar 13, 2022
6527dc0
fix: Refactoring test code using Jest's `toThrow` syntax.
dev-itsheng Mar 13, 2022
5a5af58
fix: Re-fix ESLint error of code style
dev-itsheng Mar 14, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
21 changes: 15 additions & 6 deletions packages/babel-helper-compilation-targets/test/targets-parser.spec.js
View file
Open in desktop
Expand Up @@ -293,18 +293,14 @@ describe("getTargets", () => {
},
);

(process.env.BABEL_8_BREAKING ? it.skip : it)(
"'browsers' option will have no effect if it is an empty array - Babel 7",
() => {
it("'browsers' option will have no effect if it is an empty array - Babel 7", () => {
expect(getTargets({ esmodules: "intersect", browsers: [] })).toEqual(
getTargets({ esmodules: "intersect" }),
);
},
);

(process.env.BABEL_8_BREAKING ? it.skip : it)(
"The final 'browsers' handled variable will have no effect if it is an empty array - Babel 7",
() => {
it("The final 'browsers' handled variable will have no effect if it is an empty array", () => {
expect(getTargets({ esmodules: "intersect", browsers: [] })).toEqual(
getTargets(
{ esmodules: "intersect" },
Expand All @@ -314,6 +310,19 @@ describe("getTargets", () => {
},
);

it("'resolveTargets' will be called rightly if 'browsers' is an array with some value", () => {
let x = 0;

// 'test' is an unknown browser query, so methods of 'browserslist' library will throw an error
try {
getTargets({ esmodules: "intersect", browsers: ["test"] });
} catch {
x++;
}

expect(x).toBe(1);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Jest has native support for this:

expect(() => getTargets({ esmodules: "intersect", browsers: ["test"] })).toThrow();

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great idea, I've fixed it.

});

(process.env.BABEL_8_BREAKING ? it : it.skip)(
"'intersect' behaves like no-op if no browsers are specified",
() => {
Expand Down