style: fix stylistics problems

Author: alexander-akait

src/utils.js

@@ -662,9 +662,7 @@ function getExportCode(exports, replacements, options) {
 
 async function resolveRequests(resolve, context, possibleRequests) {
   return resolve(context, possibleRequests[0])
-    .then((result) => {
-      return result;
-    })
+    .then((result) => result)
     .catch((error) => {
       const [, ...tailPossibleRequests] = possibleRequests;
 

test/helpers/compile.js

@@ -1,5 +1,5 @@
-export default (compiler) => {
-  return new Promise((resolve, reject) => {
+export default (compiler) =>
+  new Promise((resolve, reject) => {
     compiler.run((error, stats) => {
       if (error) {
         return reject(error);
@@ -8,4 +8,3 @@ export default (compiler) => {
       return resolve(stats);
     });
   });
-};

test/helpers/getErrors.js

@@ -1,5 +1,4 @@
 import normalizeErrors from "./normalizeErrors";
 
-export default (stats, shortError, type) => {
-  return normalizeErrors(stats.compilation.errors, shortError, type).sort();
-};
+export default (stats, shortError, type) =>
+  normalizeErrors(stats.compilation.errors, shortError, type).sort();

test/helpers/getWarnings.js

@@ -1,5 +1,3 @@
 import normalizeErrors from "./normalizeErrors";
 
-export default (stats) => {
-  return normalizeErrors(stats.compilation.warnings).sort();
-};
+export default (stats) => normalizeErrors(stats.compilation.warnings).sort();

test/helpers/normalizeErrors.js

@@ -27,8 +27,8 @@ function removeCWD(str) {
     .replace(new RegExp(cwd, "g"), "");
 }
 
-export default (errors, shortError, type) => {
-  return errors.map((error) => {
+export default (errors, shortError, type) =>
+  errors.map((error) => {
     let errorMessage = error.toString();
 
     if (shortError) {
@@ -50,4 +50,3 @@ export default (errors, shortError, type) => {
 
     return removeCWD(errorMessage.split("\n").slice(0, 12).join("\n"));
   });
-};

test/modules-option.test.js

@@ -676,9 +676,7 @@ describe('"modules" option', () => {
   it("issue #1063 throw error", async () => {
     const compiler = getCompiler("./modules/issue-1063/issue-1063.js", {
       modules: {
-        mode: () => {
-          return "not local, global or pure";
-        },
+        mode: () => "not local, global or pure",
       },
     });
     const stats = await compile(compiler);

test/runtime/api.test.js

@@ -27,19 +27,15 @@ describe("api", () => {
   });
 
   it("should toString a single module", () => {
-    const m = api(function (i) {
-      return i[1];
-    });
+    const m = api((i) => i[1]);
 
     m.push([1, "body { a: 1; }", ""]);
 
     expect(m.toString()).toMatchSnapshot();
   });
 
   it("should toString multiple modules", () => {
-    const m = api(function (i) {
-      return i[1];
-    });
+    const m = api((i) => i[1]);
 
     m.push([2, "body { b: 2; }", ""]);
     m.push([1, "body { a: 1; }", ""]);
@@ -48,9 +44,7 @@ describe("api", () => {
   });
 
   it("should toString with media query", () => {
-    const m = api(function (i) {
-      return i[1];
-    });
+    const m = api((i) => i[1]);
 
     const m1 = [1, "body { a: 1; }", "screen"];
     const m2 = [2, "body { b: 2; }", ""];
@@ -66,9 +60,7 @@ describe("api", () => {
   });
 
   it("should import modules", () => {
-    const m = api(function (i) {
-      return i[1];
-    });
+    const m = api((i) => i[1]);
     const m1 = [1, "body { a: 1; }", "(orientation:landscape)"];
     const m2 = [2, "body { b: 2; }", ""];
     const m3 = [3, "body { c: 3; }", ""];
@@ -84,9 +76,7 @@ describe("api", () => {
   });
 
   it("should import named modules", () => {
-    const m = api(function (i) {
-      return i[1];
-    });
+    const m = api((i) => i[1]);
     const m1 = ["./module1", "body { a: 1; }", "screen"];
     const m2 = ["./module2", "body { b: 2; }", ""];
     const m3 = ["./module3", "body { c: 3; }", ""];
@@ -156,9 +146,7 @@ describe("api", () => {
   });
 
   it("should import modules with dedupe", () => {
-    const m = api(function (i) {
-      return i[1];
-    });
+    const m = api((i) => i[1]);
 
     const m1 = [null, "body { b: 1; }", ""];
     const m2 = ["./module2", "body { b: 2; }", ""];
@@ -175,9 +163,7 @@ describe("api", () => {
   });
 
   it("should import modules when module string", () => {
-    const m = api(function (i) {
-      return i[1];
-    });
+    const m = api((i) => i[1]);
 
     m.i(".button { b: 2; }");
     m.i("");