Skip to content

Commit

Permalink
style: reformat with prettier
Browse files Browse the repository at this point in the history
  • Loading branch information
eventualbuddha authored May 31, 2024
1 parent 85cdb0e commit b36db43
Show file tree
Hide file tree
Showing 6 changed files with 47 additions and 110 deletions.
23 changes: 9 additions & 14 deletions lib/printer.ts
Original file line number Diff line number Diff line change
Expand Up @@ -932,7 +932,7 @@ function genericPrintNoParens(path: any, options: any, print: any) {
return fromString(getPossibleRaw(n) || n.value + "m", options);

case "StringLiteral":
return fromString(nodeStr(n.value, options));
return fromString(nodeStr(n.value, options));

case "BooleanLiteral": // Babel 6 Literal split
case "Literal":
Expand Down Expand Up @@ -1474,10 +1474,7 @@ function genericPrintNoParens(path: any, options: any, print: any) {
return concat(parts);

case "ClassAccessorProperty": {
parts.push(
...printClassMemberModifiers(n),
"accessor ",
);
parts.push(...printClassMemberModifiers(n), "accessor ");

if (n.computed) {
parts.push("[", path.call(print, "key"), "]");
Expand Down Expand Up @@ -2236,7 +2233,7 @@ function genericPrintNoParens(path: any, options: any, print: any) {
return member.concat(";");
}
return member;
})
}),
);

if (members.isEmpty()) {
Expand Down Expand Up @@ -2289,8 +2286,7 @@ function genericPrintNoParens(path: any, options: any, print: any) {
return concat([path.call(print, "left"), ".", path.call(print, "right")]);

case "TSAsExpression":
case "TSSatisfiesExpression":
{
case "TSSatisfiesExpression": {
const expression = path.call(print, "expression");
parts.push(
expression,
Expand Down Expand Up @@ -2335,10 +2331,10 @@ function genericPrintNoParens(path: any, options: any, print: any) {
return concat(parts);

case "TSMethodSignature":
if (n.kind === 'get') {
parts.push('get ')
} else if (n.kind === 'set') {
parts.push('set ')
if (n.kind === "get") {
parts.push("get ");
} else if (n.kind === "set") {
parts.push("set ");
}

if (n.computed) {
Expand Down Expand Up @@ -2484,7 +2480,7 @@ function genericPrintNoParens(path: any, options: any, print: any) {
return element.concat(";");
}
return element;
})
}),
);
if (lines.isEmpty()) {
return fromString("{}", options);
Expand Down Expand Up @@ -2591,7 +2587,6 @@ function genericPrintNoParens(path: any, options: any, print: any) {
return concat(parts);
}


// https://github.com/babel/babel/pull/10148
case "V8IntrinsicIdentifier":
return concat(["%", path.call(print, "name")]);
Expand Down
28 changes: 15 additions & 13 deletions test/babel.ts
Original file line number Diff line number Diff line change
Expand Up @@ -454,27 +454,32 @@ describe("Babel", function () {
["class A {", " declare public readonly x;", "}"].join(eol),
);
});

it("should keep braces in !(a && b)", function () {
const code = '(options || !options.bidirectional) ? false : true;';
const code = "(options || !options.bidirectional) ? false : true;";
const ast = recast.parse(code, parseOptions);

ast.program.body[0].expression = b.unaryExpression('!', ast.program.body[0].expression.test);

ast.program.body[0].expression = b.unaryExpression(
"!",
ast.program.body[0].expression.test,
);

assert.strictEqual(
recast.print(ast).code,
'!(options || !options.bidirectional);',
"!(options || !options.bidirectional);",
);
});

it("should use single quotes", function () {
const code = 'const a = 1;';
const code = "const a = 1;";
const ast = recast.parse(code, parseOptions);

ast.program.body.unshift(b.expressionStatement(b.stringLiteral('use strict')));

ast.program.body.unshift(
b.expressionStatement(b.stringLiteral("use strict")),
);

assert.strictEqual(
recast.print(ast, {quote: 'single'}).code,
recast.print(ast, { quote: "single" }).code,
`'use strict';\nconst a = 1;`,
);
});
Expand All @@ -490,9 +495,6 @@ describe("Babel", function () {
];
const ast = recast.parse(code.join(eol), parseOptions);

assert.strictEqual(
recast.prettyPrint(ast).code,
code.join(eol),
);
assert.strictEqual(recast.prettyPrint(ast).code, code.join(eol));
});
});
58 changes: 11 additions & 47 deletions test/printer.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1174,11 +1174,7 @@ describe("printer", function () {
});

it("prints class property initializers with type annotations correctly", function () {
const code = [
"class A {",
" foo = (a: b): void => {};",
"}",
].join(eol);
const code = ["class A {", " foo = (a: b): void => {};", "}"].join(eol);

const arg = b.identifier("a");
arg.typeAnnotation = b.typeAnnotation(
Expand All @@ -1204,11 +1200,7 @@ describe("printer", function () {
});

it("prints ClassProperty correctly", function () {
const code = [
"class A {",
" foo: Type = Bar;",
"}",
].join(eol);
const code = ["class A {", " foo: Type = Bar;", "}"].join(eol);

const ast = b.program([
b.classDeclaration(
Expand All @@ -1234,11 +1226,7 @@ describe("printer", function () {
});

it("prints 'definite' ClassProperty correctly", function () {
const code = [
"class A {",
" foo!: string;",
"}",
].join(eol);
const code = ["class A {", " foo!: string;", "}"].join(eol);

const ast = b.program([
b.classDeclaration(
Expand All @@ -1265,11 +1253,7 @@ describe("printer", function () {
});

it("prints static ClassProperty correctly", function () {
const code = [
"class A {",
" static foo = Bar;",
"}",
].join(eol);
const code = ["class A {", " static foo = Bar;", "}"].join(eol);

const ast = b.program([
b.classDeclaration(
Expand All @@ -1289,11 +1273,7 @@ describe("printer", function () {
});

it("prints ClassAccessorProperty correctly", function () {
const code = [
"class A {",
" accessor foo: Type = Bar;",
"}",
].join(eol);
const code = ["class A {", " accessor foo: Type = Bar;", "}"].join(eol);

const ast = b.program([
b.classDeclaration(
Expand All @@ -1304,8 +1284,8 @@ describe("printer", function () {
value: b.identifier("Bar"),
typeAnnotation: b.tsTypeAnnotation(
b.tsTypeReference(b.identifier("Type")),
)
})
),
}),
]),
),
]);
Expand All @@ -1319,11 +1299,7 @@ describe("printer", function () {
});

it("prints 'definite' ClassAccessorProperty correctly", function () {
const code = [
"class A {",
" accessor foo!: string;",
"}",
].join(eol);
const code = ["class A {", " accessor foo!: string;", "}"].join(eol);

const ast = b.program([
b.classDeclaration(
Expand All @@ -1347,11 +1323,7 @@ describe("printer", function () {
});

it("prints static ClassAccessorProperty correctly", function () {
const code = [
"class A {",
" static accessor foo = Bar;",
"}",
].join(eol);
const code = ["class A {", " static accessor foo = Bar;", "}"].join(eol);

const ast = b.program([
b.classDeclaration(
Expand All @@ -1375,11 +1347,7 @@ describe("printer", function () {
});

it("prints abstract ClassAccessorProperty correctly", function () {
const code = [
"class A {",
" abstract accessor foo = Bar;",
"}",
].join(eol);
const code = ["class A {", " abstract accessor foo = Bar;", "}"].join(eol);

const ast = b.program([
b.classDeclaration(
Expand All @@ -1403,11 +1371,7 @@ describe("printer", function () {
});

it("prints override ClassAccessorProperty correctly", function () {
const code = [
"class A {",
" override accessor foo = Bar;",
"}",
].join(eol);
const code = ["class A {", " override accessor foo = Bar;", "}"].join(eol);

const ast = b.program([
b.classDeclaration(
Expand Down
2 changes: 1 addition & 1 deletion test/syntax.ts
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ const nodeMajorVersion = parseInt(process.versions.node, 10);
},
});

Object.keys(types.namedTypes).forEach(name => {
Object.keys(types.namedTypes).forEach((name) => {
it(name, () => {
assert.ok(hasOwn.call(typeNames, name), "unhandled type: " + name);
assert.strictEqual(name, typeNames[name]);
Expand Down
41 changes: 10 additions & 31 deletions test/typescript.ts
Original file line number Diff line number Diff line change
Expand Up @@ -285,17 +285,13 @@ const nodeMajorVersion = parseInt(process.versions.node, 10);
check([
"class Dog extends Animal {",
" protected override getSound() {",
" return \"bark\";",
' return "bark";',
" }",
"}",
])

check([
"export interface S {",
" i(s: string): boolean;",
"}"
]);

check(["export interface S {", " i(s: string): boolean;", "}"]);

check([
"namespace Validation {",
" export interface S {",
Expand All @@ -304,17 +300,9 @@ const nodeMajorVersion = parseInt(process.versions.node, 10);
"}",
]);

check([
"export interface S {",
" i(j: string): boolean;",
"}",
]);
check(["export interface S {", " i(j: string): boolean;", "}"]);

check([
"declare namespace D3 {",
" export const f: number;",
"}",
]);
check(["declare namespace D3 {", " export const f: number;", "}"]);

check(["declare function foo<K, V>(arg: T = getDefault()): R"]);

Expand All @@ -324,13 +312,7 @@ const nodeMajorVersion = parseInt(process.versions.node, 10);
"}",
]);

check([
"function myFunction(",
" {",
" param1",
" }: Params",
") {}",
]);
check(["function myFunction(", " {", " param1", " }: Params", ") {}"]);

check([
'const unqualified: import("package") = 1;',
Expand Down Expand Up @@ -361,8 +343,8 @@ const nodeMajorVersion = parseInt(process.versions.node, 10);
check([
"type alias = boolean;",
"const value = 0;",
"export { type alias, value };"
])
"export { type alias, value };",
]);
});

it("InterfaceBody: duplicate semicolon", function () {
Expand Down Expand Up @@ -468,11 +450,8 @@ const nodeMajorVersion = parseInt(process.versions.node, 10);

const ast = recast.parse(code, { parser });

assert.strictEqual(
recast.prettyPrint(ast).code,
code
);
})
assert.strictEqual(recast.prettyPrint(ast).code, code);
});
});

testReprinting(
Expand Down
5 changes: 1 addition & 4 deletions tsconfig.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,5 @@
"importHelpers": true,
"stripInternal": true
},
"exclude": [
"node_modules",
"test/data"
]
"exclude": ["node_modules", "test/data"]
}

0 comments on commit b36db43

Please sign in to comment.