feat(jest): upgrade jest testing and linting

Author: sam

.eslintignore

@@ -0,0 +1,2 @@
+tests_config/
+tests/

.eslintrc

@@ -0,0 +1,9 @@
+{
+  "env": {
+    "jest": true
+  },
+  "extends": ["airbnb-base", "prettier"],
+  "globals": {
+    "run_spec": false
+  }
+}

.github/workflows/nodejs.yml

@@ -9,7 +9,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: ["14.x"]
+        node-version: ['14.x']
 
     steps:
       - uses: actions/checkout@v2

.prettierignore

@@ -0,0 +1 @@
+*.html

.prettierrc.js

@@ -0,0 +1,24 @@
+/**
+ * @file Prettier configuration for Conformance
+ * @version 1.0.6
+ * @summary base config adapted from AirBNB to maximize performance
+ * @schema http://json.schemastore.org/prettierrc
+ */
+
+'use strict';
+
+module.exports = {
+  arrowParens: 'always',
+  bracketSpacing: true,
+  endOfLine: 'lf',
+  jsxBracketSameLine: false,
+  jsxSingleQuote: false,
+  printWidth: 100,
+  proseWrap: 'always',
+  quoteProps: 'as-needed',
+  semi: true,
+  singleQuote: true,
+  tabWidth: 2,
+  trailingComma: 'all',
+  useTabs: false,
+};

README.md

@@ -1,13 +1,13 @@
 # [prettier-solidity-config](#)
 
-## Quickstart 
+## Quickstart
 
 ```js
 /**
-* @file prettier.config.js
-* @version v1.0.7
-* @summary Prettier Configuration for Solidity
-*/
+ * @file prettier.config.js
+ * @version v1.0.7
+ * @summary Prettier Configuration for Solidity
+ */
 
 'use strict';
 
@@ -19,7 +19,7 @@ module.exports = prettierConfig;
 
 Prettier configuration for Solidity
 
-## Install 
+## Install
 
 #### Prettier
 
@@ -28,6 +28,7 @@ npm i -D prettier prettier-plugin-solidity --save-exact
 ```
 
 #### with SolHint
+
 ```bash
 npm install --save-dev solhint solhint-plugin-prettier prettier prettier-plugin-solidity --save-exact
 ```
@@ -43,26 +44,23 @@ npm install --save-dev solhint solhint-plugin-prettier prettier prettier-plugin-
 
 ## Abstract
 
-This configuration is optimized to reduce **diff churn** and improve AST results. 
+This configuration is optimized to reduce **diff churn** and improve AST results.
 
-The following rules are employed, with specific reasonings to their choice (source: airbnb style guide):
+The following rules are employed, with specific reasonings to their choice (source: airbnb style
+guide):
 
-###  whitespace
+### whitespace
 
 [source@airbnb/javascript#whitespace--in-braces](https://github.com/airbnb/javascript#whitespace--in-braces)
 
 - 19.12 Add spaces inside curly braces. eslint: object-curly-spacing
 
 ```jsx
+// bad
+const foo = { clark: 'kent' };
 
-
-
-    // bad
-    const foo = {clark: 'kent'};
-
-    // good
-    const foo = { clark: 'kent' };
-
+// good
+const foo = { clark: 'kent' };
 ```
 
 #### arrow-parens
@@ -71,69 +69,72 @@ The following rules are employed, with specific reasonings to their choice (sour
 
 [source@airbnb/javascript#arrows--one-arg-parens](https://github.com/airbnb/javascript#arrows--one-arg-parens)
 
- > Why? Minimizes diff churn when adding or removing arguments.
+> Why? Minimizes diff churn when adding or removing arguments.
+
 ```jsx
-    // bad
-    [1, 2, 3].map(x => x * x);
-
-    // good
-    [1, 2, 3].map((x) => x * x);
-
-    // bad
-    [1, 2, 3].map(number => (
-      `A long string with the ${number}. It’s so long that we don’t want it to take up space on the .map line!`
-    ));
-
-    // good
-    [1, 2, 3].map((number) => (
-      `A long string with the ${number}. It’s so long that we don’t want it to take up space on the .map line!`
-    ));
-
-    // bad
-    [1, 2, 3].map(x => {
-      const y = x + 1;
-      return x * y;
-    });
-
-    // good
-    [1, 2, 3].map((x) => {
-      const y = x + 1;
-      return x * y;
-    });
+// bad
+[1, 2, 3].map((x) => x * x);
+
+// good
+[1, 2, 3].map((x) => x * x);
+
+// bad
+[1, 2, 3].map(
+  (number) =>
+    `A long string with the ${number}. It’s so long that we don’t want it to take up space on the .map line!`,
+);
+
+// good
+[1, 2, 3].map(
+  (number) =>
+    `A long string with the ${number}. It’s so long that we don’t want it to take up space on the .map line!`,
+);
+
+// bad
+[1, 2, 3].map((x) => {
+  const y = x + 1;
+  return x * y;
+});
+
+// good
+[1, 2, 3].map((x) => {
+  const y = x + 1;
+  return x * y;
+});
 ```
 
 #### one var
 
 - 13.2 Use one const or let declaration per variable or assignment. eslint: one-var
 
-> Why? It’s easier to add new variable declarations this way, and you never have to worry about swapping out a ; for a , 
-> or introducing punctuation-only diffs. You can also step through each declaration with the debugger, instead of jumping through all of them at once.
+> Why? It’s easier to add new variable declarations this way, and you never have to worry about
+> swapping out a ; for a , or introducing punctuation-only diffs. You can also step through each
+> declaration with the debugger, instead of jumping through all of them at once.
 
 ##### [ref:eslint/rules/one-var](https://eslint.org/docs/rules/one-var)
 
 ```jsx
-    // bad
-    const items = getItems(),
-        goSportsTeam = true,
-        dragonball = 'z';
-
-    // bad
-    // (compare to above, and try to spot the mistake)
-    const items = getItems(),
-        goSportsTeam = true;
-        dragonball = 'z';
-
-    // good
-    const items = getItems();
-    const goSportsTeam = true;
-    const dragonball = 'z';
+// bad
+const items = getItems(),
+  goSportsTeam = true,
+  dragonball = 'z';
+
+// bad
+// (compare to above, and try to spot the mistake)
+const items = getItems(),
+  goSportsTeam = true;
+dragonball = 'z';
+
+// good
+const items = getItems();
+const goSportsTeam = true;
+const dragonball = 'z';
 ```
 
 #### 20.2 Additional trailing comma: Yup. eslint: comma-dangle
 
     Why? This leads to cleaner git diffs. Also, transpilers like Babel will remove the additional trailing comma in the transpiled code which means you don’t have to worry about the trailing comma problem in legacy browsers.
-    
-    
+
 ```diff
 // bad - git diff without trailing comma
 const hero = {
@@ -154,19 +155,15 @@ const hero = {
 #### 8.6 Enforce the location of arrow function bodies with implicit returns. eslint: implicit-arrow-linebreak
 
 ```js
-    // bad
-    (foo) =>
-      bar;
-
-    (foo) =>
-      (bar);
-
-    // good
-    (foo) => bar;
-    (foo) => (bar);
-    (foo) => (
-       bar
-    )
+// bad
+(foo) => bar;
+
+(foo) => bar;
+
+// good
+(foo) => bar;
+(foo) => bar;
+(foo) => bar;
 ```
 
 ## License

index.spec.js

@@ -0,0 +1 @@
+run_spec('../tests/__snapshots__/jsfmt.spec.js.snap');

jest.config.js

@@ -7,24 +7,21 @@ module.exports = {
     '!scripts/generateIndexes.js',
     'src/**/*.js',
     '!<rootDir>/node_modules/',
-    '!src/prettier-comments/**/*.js'
+    '!src/prettier-comments/**/*.js',
   ],
   coverageDirectory: './coverage/',
   coverageThreshold: {
     global: {
       branches: 100,
       functions: 100,
       lines: 100,
-      statements: 100
-    }
+      statements: 100,
+    },
   },
   setupFiles: ['<rootDir>/tests_config/run_spec.js'],
   snapshotSerializers: ['<rootDir>/tests_config/raw-serializer.js'],
   testEnvironment: 'node',
   testRegex: 'jsfmt\\.spec\\.js$|__tests__/.*\\.js$|scripts/.*\\.test\\.js$',
   transform: {},
-  watchPlugins: [
-    'jest-watch-typeahead/filename',
-    'jest-watch-typeahead/testname'
-  ]
+  watchPlugins: ['jest-watch-typeahead/filename', 'jest-watch-typeahead/testname'],
 };