From 2e5e5b1b251a08f5f35488de03e613ee948ea726 Mon Sep 17 00:00:00 2001
From: Titus Wormer <tituswormer@gmail.com>
Date: Thu, 5 Aug 2021 16:16:07 +0200
Subject: [PATCH] Refactor code-style

---
 index.js      |  2 +-
 package.json  | 12 ++---------
 test/index.js | 60 +++++++++++++++++++++++++--------------------------
 3 files changed, 32 insertions(+), 42 deletions(-)

diff --git a/index.js b/index.js
index 6ed4a8b..4014423 100644
--- a/index.js
+++ b/index.js
@@ -5,7 +5,7 @@ import {
 } from 'mdast-util-frontmatter'
 
 export default function remarkFrontmatter(options) {
-  var data = this.data()
+  const data = this.data()
   add('micromarkExtensions', frontmatter(options))
   add('fromMarkdownExtensions', frontmatterFromMarkdown(options))
   add('toMarkdownExtensions', frontmatterToMarkdown(options))
diff --git a/package.json b/package.json
index bd57c22..f64fa9d 100644
--- a/package.json
+++ b/package.json
@@ -46,7 +46,7 @@
     "tape": "^5.0.0",
     "to-vfile": "^7.0.0",
     "unified": "^10.0.0",
-    "xo": "^0.37.0"
+    "xo": "^0.39.0"
   },
   "scripts": {
     "format": "remark . -qfo --ignore-pattern test/ && prettier . -w --loglevel warn && xo --fix",
@@ -64,14 +64,6 @@
   },
   "xo": {
     "prettier": true,
-    "esnext": false,
-    "rules": {
-      "no-var": "off",
-      "prefer-arrow-callback": "off",
-      "unicorn/no-array-callback-reference": "off",
-      "unicorn/prefer-optional-catch-binding": "off",
-      "guard-for-in": "off"
-    },
     "ignores": [
       "types/"
     ]
@@ -79,7 +71,7 @@
   "remarkConfig": {
     "plugins": [
       [
-        "./",
+        "./index.js",
         [
           "yaml",
           "toml"
diff --git a/test/index.js b/test/index.js
index daa00c4..b39e481 100644
--- a/test/index.js
+++ b/test/index.js
@@ -7,22 +7,22 @@ import {remark} from 'remark'
 import {isHidden} from 'is-hidden'
 import remarkFrontmatter from '../index.js'
 
-var join = path.join
-var read = fs.readFileSync
-var write = fs.writeFileSync
-var dir = fs.readdirSync
+const join = path.join
+const read = fs.readFileSync
+const write = fs.writeFileSync
+const dir = fs.readdirSync
 
-test('remarkFrontmatter', function (t) {
-  t.doesNotThrow(function () {
+test('remarkFrontmatter', (t) => {
+  t.doesNotThrow(() => {
     remark().use(remarkFrontmatter).freeze()
   }, 'should not throw if not passed options')
 
-  t.doesNotThrow(function () {
+  t.doesNotThrow(() => {
     unified().use(remarkFrontmatter).freeze()
   }, 'should not throw if without parser or compiler')
 
   t.throws(
-    function () {
+    () => {
       unified().use(remarkFrontmatter, [1]).freeze()
     },
     /^Error: Expected matter to be an object, not `1`/,
@@ -30,7 +30,7 @@ test('remarkFrontmatter', function (t) {
   )
 
   t.throws(
-    function () {
+    () => {
       unified().use(remarkFrontmatter, ['jsonml']).freeze()
     },
     /^Error: Missing matter definition for `jsonml`/,
@@ -38,7 +38,7 @@ test('remarkFrontmatter', function (t) {
   )
 
   t.throws(
-    function () {
+    () => {
       unified()
         .use(remarkFrontmatter, [{marker: '*'}])
         .freeze()
@@ -48,7 +48,7 @@ test('remarkFrontmatter', function (t) {
   )
 
   t.throws(
-    function () {
+    () => {
       unified()
         .use(remarkFrontmatter, [{type: 'jsonml'}])
         .freeze()
@@ -60,41 +60,39 @@ test('remarkFrontmatter', function (t) {
   t.end()
 })
 
-test('fixtures', function (t) {
-  var base = join('test', 'fixtures')
-  var entries = dir(base).filter((d) => !isHidden(d))
+test('fixtures', (t) => {
+  const base = join('test', 'fixtures')
+  const entries = dir(base).filter((d) => !isHidden(d))
+  let index = -1
 
   t.plan(entries.length)
 
-  entries.forEach(each)
-
-  function each(fixture) {
-    t.test(fixture, function (st) {
-      var input = readSync(join(base, fixture, 'input.md'))
-      var treePath = join(base, fixture, 'tree.json')
-      var outputPath = join(base, fixture, 'output.md')
-      var output
-      var actual
-      var expected
-      var config
-      var proc
+  while (++index < entries.length) {
+    const fixture = entries[index]
+    t.test(fixture, (st) => {
+      const input = readSync(join(base, fixture, 'input.md'))
+      const treePath = join(base, fixture, 'tree.json')
+      const outputPath = join(base, fixture, 'output.md')
+      let output
+      let expected
+      let config
 
       try {
         config = JSON.parse(read(join(base, fixture, 'config.json')))
-      } catch (_) {}
+      } catch {}
 
-      proc = remark().use(remarkFrontmatter, config)
-      actual = JSON.parse(JSON.stringify(proc.parse(input)))
+      const proc = remark().use(remarkFrontmatter, config)
+      const actual = JSON.parse(JSON.stringify(proc.parse(input)))
 
       try {
         output = read(outputPath, 'utf8')
-      } catch (_) {
+      } catch {
         output = String(input)
       }
 
       try {
         expected = JSON.parse(read(treePath))
-      } catch (_) {
+      } catch {
         // New fixture.
         write(treePath, JSON.stringify(actual, 0, 2) + '\n')
         expected = actual