eslint-plugin-putout 10.8.0 → 11.1.0

package/README.md

@@ -73,6 +73,7 @@ Then configure the rules you want to use under the rules section.
 - [No unresolved](/packages/eslint-plugin-putout/lib/no-unresolved)
 - [Evaluate](/packages/eslint-plugin-putout/lib/evaluate)
 - [Tape: add new line before assertion]('/packages/eslint-plugin-putout/lib/tape-add-new-line-before-assertion)
+- [Tape: add new line between tests]('/packages/eslint-plugin-putout/lib/tape-add-new-line-between-tests)
 
 ### Safe mode
 
@@ -93,6 +94,7 @@ List of disabled `putout` rules:
 
 - [remove-empty](https://github.com/coderaiser/putout/tree/v20.0.0/packages/plugin-remove-empty);
 - [remove-unused-variables](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-unused-variables);
+- [remove-unused-types](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-unused-types);
 - [remove-unused-for-of-variables](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-unused-for-of-variables);
 - [remove-unused-expressions](https://github.com/coderaiser/putout/tree/v20.0.0/packages);
 - [remove-skip](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-skip);

package/lib/index.js

@@ -31,6 +31,7 @@ module.exports.rules = {
     ...getWrapRule('no-unresolved'),
     ...getWrapRule('evaluate'),
     ...getWrapRule('tape-add-newline-before-assertion'),
+    ...getWrapRule('tape-add-newline-between-tests'),
     ...getRule('putout'),
 };
 
@@ -59,6 +60,7 @@ const recommended = {
         'putout/no-unresolved': 'error',
         'putout/evaluate': 'error',
         'putout/tape-add-newline-before-assertion': 'error',
+        'putout/tape-add-newline-between-tests': 'error',
         'putout/putout': 'error',
         
         'node/no-unsupported-features/es-syntax': 'off',
@@ -83,11 +85,12 @@ const safe = {
         'putout/putout': ['error', {
             rules: {
                 'remove-empty': 'off',
+                'remove-unused-types': 'off',
                 'remove-unused-variables': 'off',
                 'remove-unused-expressions': 'off',
                 'remove-unused-for-of-variables': 'off',
-                'remove-skip': 'off',
-                'remove-only': 'off',
+                'tape/remove-skip': 'off',
+                'tape/remove-only': 'off',
                 'remove-console': 'off',
                 'remove-debugger': 'off',
                 'convert-for-to-for-of': 'off',

package/lib/tape-add-newline-between-tests/README.md

@@ -0,0 +1,42 @@
+# Add new line between tests (add-newline-between-tests)
+
+Add new line between tests, for [supertape](https://github.com/coderaiser/supertape).
+
+## Rule Details
+
+This rule aims to add new line between tests.
+
+Examples of **incorrect** code for this rule:
+
+```js
+test('lint: do some check', (t) => {
+    const result = 1 + 2;
+    
+    t.equal(result, 3);
+    t.end();
+});
+test('lint: do some check', (t) => {
+    const result = 1 + 2;
+    
+    t.equal(result, 3);
+    t.end();
+});
+```
+
+Examples of **correct** code for this rule:
+
+```js
+test('lint: do some check', (t) => {
+    const result = 1 + 2;
+    
+    t.equal(result, 3);
+    t.end();
+});
+
+test('lint: do some check', (t) => {
+    const result = 1 + 2;
+    
+    t.equal(result, 3);
+    t.end();
+});
+```

package/lib/tape-add-newline-between-tests/index.js

@@ -0,0 +1,27 @@
+'use strict';
+
+module.exports.category = 'tape';
+module.exports.report = () => 'Add new line between tests';
+
+module.exports.filter = ({text, node, getText, getCommentsBefore}) => {
+    if (!/^test(\.only|\.skip)?\(/.test(text))
+        return false;
+    
+    const comments = getCommentsBefore(node);
+    
+    if (comments.length)
+        return false;
+    
+    const [a] = getText(node, 2);
+    
+    return a === ';';
+};
+
+module.exports.fix = ({text}) => {
+    return `\n${text}`;
+};
+
+module.exports.include = () => [
+    'CallExpression ',
+];
+

package/lib/ts.js

@@ -12,6 +12,7 @@ module.exports = [{
     rules: {
         'no-undef': 'off',
         'no-var': 'off',
+        'putout/no-unresolved': 'off',
     },
 }];
 

package/lib/wrap.js

@@ -6,6 +6,7 @@ const prepare = (plugin, context, options) => (node) => {
     const source = context.getSourceCode();
     const filename = context.getFilename();
     const getText = source.getText.bind(source);
+    const getCommentsBefore = source.getCommentsBefore.bind(source);
     
     const text = getText(node);
     
@@ -14,6 +15,7 @@ const prepare = (plugin, context, options) => (node) => {
         node,
         options,
         getText,
+        getCommentsBefore,
         filename,
     });
     

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-putout",
-  "version": "10.8.0",
+  "version": "11.1.0",
   "description": "eslint plugin for putout",
   "release": false,
   "tag": false,
@@ -24,7 +24,7 @@
     "watch:test": "madrun watch:test",
     "lint": "madrun lint",
     "lint:all": "madrun lint:all",
-    "lint:ide": "madrun lint:ide",
+    "lint:safe": "madrun lint:safe",
     "fresh:lint": "madrun fresh:lint",
     "lint:fresh": "madrun lint:fresh",
     "fix:lint": "madrun fix:lint",
@@ -58,8 +58,8 @@
     "node": ">=14"
   },
   "peerDependencies": {
-    "eslint": ">=8.0.0-beta.0",
-    "putout": ">=20"
+    "eslint": ">=8.0.0",
+    "putout": ">=21"
   },
   "license": "MIT",
   "publishConfig": {