@putout/plugin-tape 14.0.1 → 14.2.0

package/README.md

@@ -24,7 +24,6 @@ npm i @putout/plugin-tape -D
         "tape/jest": "on",
         "tape/apply-stub": "on",
         "tape/apply-destructuring": "on",
-        "tape/apply-stub-operator": "on",
         "tape/apply-with-name": "on",
         "tape/add-t-end": "on",
         "tape/add-stop-all": "on",
@@ -403,26 +402,6 @@ const c = stub().rejects(Error('hello'));
 const d = stub().rejects(Error('hello'));
 ```
 
-## apply-stub-operator
-
-### ❌ Example of incorrect code
-
-```js
-test('some message', (t) => {
-    t.ok(fn.calledWith(a));
-    t.end();
-});
-```
-
-### ✅ Example of correct code
-
-```js
-test('some message', (t) => {
-    t.calledWith(fn, [a]);
-    t.end();
-});
-```
-
 ## apply-with-name
 
 ### ❌ Example of incorrect code

package/lib/convert-ok-to-called-with/index.js

@@ -1,7 +1,22 @@
 'use strict';
 
-module.exports.report = () => `Use 't.calledWith()' instead of 't.ok()'`;
+const {operator} = require('putout');
+const {compare} = operator;
+
+module.exports.report = (path) => {
+    if (compare(path, 't.ok(__args)'))
+        return `Use 't.calledWith()' instead of 't.ok()'`;
+    
+    return `Use 't.notCalledWith()' instead of 't.notOk()'`;
+};
 
 module.exports.replace = () => ({
     't.ok(__a.calledWith(__args))': 't.calledWith(__a, [__args])',
+    't.ok(__a.calledWith(__args), __b)': 't.calledWith(__a, [__args], __b)',
+    
+    't.ok(__a.calledWith(...__b))': 't.calledWith(__a, __b)',
+    't.ok(__a.calledWith(...__b), __c)': 't.calledWith(__a, __b, __c)',
+    
+    't.notOk(__a.called, __b)': 't.notCalled(__a, __b)',
+    't.ok(__a.calledWith(), __b)': 't.calledWithNoArgs(__a, __b)',
 });

package/lib/index.js

@@ -27,7 +27,6 @@ const declare = require('./declare');
 const removeDefaultMessages = require('./remove-default-messages');
 const removeUselessNotCalledArgs = require('./remove-useless-not-called-args');
 const switchExpectedWithResult = require('./switch-expected-with-result');
-const applyStubOperator = require('./apply-stub-operator');
 const addTEnd = require('./add-t-end');
 const addStopAll = require('./add-stop-all');
 const removeStopAll = require('./remove-stop-all');
@@ -66,7 +65,6 @@ module.exports.rules = {
     'remove-default-messages': removeDefaultMessages,
     'remove-useless-not-called-args': removeUselessNotCalledArgs,
     'switch-expected-with-result': switchExpectedWithResult,
-    'apply-stub-operator': applyStubOperator,
     'add-t-end': addTEnd,
     'add-stop-all': addStopAll,
     'remove-stop-all': removeStopAll,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-tape",
-  "version": "14.0.1",
+  "version": "14.2.0",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin helps with tests",

package/lib/apply-stub-operator/index.js

@@ -1,14 +0,0 @@
-'use strict';
-
-module.exports.report = () => 'Use Stub operator';
-
-module.exports.replace = () => ({
-    't.ok(__a.calledWith(__b))': 't.calledWith(__a, [__b])',
-    't.ok(__a.calledWith(__b, __c))': 't.calledWith(__a, [__b, __c])',
-    't.ok(__a.calledWith(...__b))': 't.calledWith(__a, __b)',
-    't.ok(__a.calledWith(__b), __c)': 't.calledWith(__a, [__b], __c)',
-    't.ok(__a.calledWith(__b, __c), __d)': 't.calledWith(__a, [__b, __c], __d)',
-    't.ok(__a.calledWith(...__b), __c)': 't.calledWith(__a, __b, __c)',
-    't.notOk(__a.called, __b)': 't.notCalled(__a, __b)',
-    't.ok(__a.calledWith(), __b)': 't.calledWithNoArgs(__a, __b)',
-});