fix: added linting

Author: Ben Monro

.gitignore

@@ -1,3 +1,5 @@
 .DS_Store
 node_modules
 public/bundle.*
+coverage
+dist

.prettierrc

@@ -0,0 +1,11 @@
+{
+  "printWidth": 80,
+  "tabWidth": 2,
+  "useTabs": false,
+  "semi": false,
+  "singleQuote": true,
+  "trailingComma": "all",
+  "bracketSpacing": false,
+  "jsxBracketSameLine": false,
+  "proseWrap": "always"
+}

jest.config.js

@@ -1,16 +1,16 @@
-const { jest: jestConfig } = require("kcd-scripts/config");
+const { jest: jestConfig } = require('kcd-scripts/config');
 
 const config = Object.assign(jestConfig, {
-  roots: ["tests"],
-  testMatch: ["/**/*.spec.js"],
+  roots: ['tests'],
+  testMatch: ['/**/*.spec.js'],
   transform: {
     ...jestConfig.transform,
-    "\\.svelte$": "./tests/transform"
+    '\\.svelte$': './tests/transform'
   },
   transformIgnorePatterns: [
     ...jestConfig.transformIgnorePatterns,
-    "/node_modules/(?!svelte).+\\.js$"
+    '/node_modules/(?!svelte).+\\.js$'
   ],
-  moduleFileExtensions: [...jestConfig.moduleFileExtensions, "svelte"]
+  moduleFileExtensions: [...jestConfig.moduleFileExtensions, 'svelte']
 });
 module.exports = config;

package-lock.json

@@ -7,6 +7,7 @@
     "dom-testing-library": "^4.0.0",
     "esm": "^3.2.22",
     "jest": "^24.7.1",
+    "jest-dom": "^3.1.2",
     "kcd-scripts": "^1.2.1",
     "npm-run-all": "^4.1.5",
     "rollup": "^1.10.1",
@@ -20,14 +21,31 @@
     "svelte-jest": "^0.2.0"
   },
   "scripts": {
-    "build": "rollup -c",
-    "autobuild": "rollup -c -w",
+    "lint": "kcd-scripts lint",
+    "validate": "kcd-scripts validate",
+    "build": "kcd-scripts build",
+    "build:example": "rollup -c",
+    "autobuild:example": "rollup -c -w",
     "dev": "run-p start:dev autobuild",
     "start": "sirv public",
     "start:dev": "sirv public --dev",
     "test": "kcd-scripts test --no-watch --config=jest.config.js"
   },
   "dependencies": {
     "@babel/runtime": "^7.4.3"
-  }
+  },
+  "eslintConfig": {
+    "extends": "./node_modules/kcd-scripts/eslint.js",
+    "rules": {
+      "react/prop-types": "off",
+      "import/no-unassigned-import": "off",
+      "import/named": "off"
+    }
+  },
+  "eslintIgnore": [
+    "node_modules",
+    "coverage",
+    "dist",
+    "public"
+  ]
 }

package.json

@@ -1,25 +1,26 @@
-import { getQueriesForElement } from "dom-testing-library";
+import {getQueriesForElement} from 'dom-testing-library'
 
-export * from "dom-testing-library";
-let mountedContainers = new Set();
+export * from 'dom-testing-library'
+const mountedContainers = new Set()
 export const render = (Component, props) => {
-  const container = document.body.appendChild(document.createElement("div"));
+  const container = document.body.appendChild(document.createElement('div'))
 
-  let rendered = new Component({
+  const rendered = new Component({
     target: container,
-    props
-  });
+    props,
+  })
 
-  mountedContainers.add(rendered);
+  mountedContainers.add(rendered)
   return {
-    ...getQueriesForElement(document.body)
-  };
-};
-export const cleanup = () => {
-  mountedContainers.forEach(cleanupAtContainer);
-};
+    ...getQueriesForElement(document.body),
+  }
+}
 
 const cleanupAtContainer = container => {
-  container.$destroy();
-  mountedContainers.delete(container);
-};
+  container.$destroy()
+  mountedContainers.delete(container)
+}
+
+export const cleanup = () => {
+  mountedContainers.forEach(cleanupAtContainer)
+}

src/index.js

@@ -1,22 +1,22 @@
-const rollup = require("rollup");
-const esm = require("esm");
+const rollup = require('rollup')
+const esm = require('esm')
 
-const config = esm(module)("../rollup.config");
+const config = esm(module)('../rollup.config')
 
 function generateOptions(filePath, name) {
   return {
-    input: { input: filePath, plugins: config.default.plugins },
+    input: {input: filePath, plugins: config.default.plugins},
     output: {
       file: `./dist/test/${name}.js`,
-      format: "cjs"
-    }
-  };
+      format: 'cjs',
+    },
+  }
 }
 
 function compile(filePath, name) {
-  const { input, output } = generateOptions(filePath, name);
+  const {input, output} = generateOptions(filePath, name)
 
-  return rollup.rollup(input).then(bundle => bundle.generate(output));
+  return rollup.rollup(input).then(bundle => bundle.generate(output))
 }
 
-module.exports = { compile };
+module.exports = {compile}

tests/compile.js

@@ -1,21 +1,22 @@
-import App from "./example/App.svelte";
-import { render, fireEvent, waitForElement, cleanup } from "../src";
+import {render, fireEvent, waitForElement, cleanup} from '../src'
+import App from './example/App.svelte'
+import 'jest-dom/extend-expect'
 
-afterEach(cleanup);
-describe("queries", () => {
-  test("getByText", () => {
-    const { getByText } = render(App, { name: "world" });
+afterEach(cleanup)
+describe('queries', () => {
+  test('getByText', () => {
+    const {getByText} = render(App, {name: 'world'})
 
-    expect(getByText("Hello world!"));
-  });
+    expect(getByText('Hello world!')).toBeInTheDocument()
+  })
 
-  test("click button", async () => {
-    const { getByText } = render(App, { name: "world" });
+  test('click button', async () => {
+    const {getByText} = render(App, {name: 'world'})
 
-    fireEvent.click(getByText("Button Text"));
+    fireEvent.click(getByText('Button Text'))
 
-    const button = await waitForElement(() => getByText("Button Clicked"));
+    const button = await waitForElement(() => getByText('Button Clicked'))
 
-    expect(button);
-  });
-});
+    expect(button).toBeInTheDocument()
+  })
+})

tests/queries.spec.js

@@ -1,14 +1,13 @@
-const deasync = require("deasync");
-const { compile } = require("./compile");
+const deasync = require('deasync')
+const {compile} = require('./compile')
 
 module.exports.process = (_s, filePath, _c, _o) => {
-  let code, map;
-  compile(filePath, "App", {}).then(comp => {
+  let code, map
+  compile(filePath, 'App', {}).then(comp => {
+    code = comp.output[0].code
+    map = comp.output[0].map
+  })
+  deasync.loopWhile(() => !code && !map)
 
-    code = comp.output[0].code;
-    map = comp.output[0].map;
-  });
-  deasync.loopWhile(() => !code && !map);
-
-  return { code, map };
-};
+  return {code, map}
+}