@lvce-editor/eslint-plugin-e2e 17.8.0 → 18.1.0

package/dist/index.js

@@ -1,4 +1,4 @@
-const meta$9 = {
+const meta$a = {
   docs: {
     description: 'Disallow direct click calls in E2E tests'
   },
@@ -10,7 +10,7 @@ const meta$9 = {
 const isDirectClickCall = node => {
   return node.callee.type === 'MemberExpression' && node.callee.property.type === 'Identifier' && node.callee.property.name === 'click';
 };
-const create$9 = context => {
+const create$a = context => {
   return {
     CallExpression(node) {
       if (!isDirectClickCall(node)) {
@@ -26,12 +26,12 @@ const create$9 = context => {
 
 const noDirectClick = {
   __proto__: null,
-  create: create$9,
-  meta: meta$9
+  create: create$a,
+  meta: meta$a
 };
 
 const allowedTypeImportSource = '@lvce-editor/test-with-playwright';
-const meta$8 = {
+const meta$9 = {
   docs: {
     description: 'Disallow imports in e2e tests except type imports from @lvce-editor/test-with-playwright'
   },
@@ -49,7 +49,7 @@ const isAllowedTypeImport = node => {
   }
   return node.specifiers.length > 0 && node.specifiers.every(specifier => specifier.importKind === 'type');
 };
-const create$8 = context => {
+const create$9 = context => {
   return {
     ImportDeclaration(node) {
       if (isAllowedTypeImport(node)) {
@@ -65,11 +65,11 @@ const create$8 = context => {
 
 const noImports = {
   __proto__: null,
-  create: create$8,
-  meta: meta$8
+  create: create$9,
+  meta: meta$9
 };
 
-const meta$7 = {
+const meta$8 = {
   docs: {
     description: 'Disallow inline locator expressions inside expect calls'
   },
@@ -126,7 +126,7 @@ const containsInlineLocatorCall = node => {
 const isExpectCall$2 = node => {
   return node.callee.type === 'Identifier' && node.callee.name === 'expect';
 };
-const create$7 = context => {
+const create$8 = context => {
   return {
     CallExpression(node) {
       if (!isExpectCall$2(node)) {
@@ -146,11 +146,11 @@ const create$7 = context => {
 
 const noInlineLocatorInExpect = {
   __proto__: null,
-  create: create$7,
-  meta: meta$7
+  create: create$8,
+  meta: meta$8
 };
 
-const meta$6 = {
+const meta$7 = {
   docs: {
     description: 'Disallow inline nth locator expressions inside expect calls'
   },
@@ -213,7 +213,7 @@ const containsInlineNthCall = node => {
 const isExpectCall$1 = node => {
   return node.callee.type === 'Identifier' && node.callee.name === 'expect';
 };
-const create$6 = context => {
+const create$7 = context => {
   return {
     CallExpression(node) {
       if (!isExpectCall$1(node)) {
@@ -233,11 +233,11 @@ const create$6 = context => {
 
 const noInlineNthInExpect = {
   __proto__: null,
-  create: create$6,
-  meta: meta$6
+  create: create$7,
+  meta: meta$7
 };
 
-const meta$5 = {
+const meta$6 = {
   docs: {
     description: 'Disallow lazy nth-based variable names for locator variables'
   },
@@ -298,7 +298,7 @@ const containsNthCall = node => {
 const hasLazyNthVariableName = node => {
   return isIdentifierNode$1(node) && lazyNthVariableNamePattern.test(node.name);
 };
-const create$5 = context => {
+const create$6 = context => {
   return {
     VariableDeclarator(node) {
       if (!isVariableDeclaratorNode$1(node)) {
@@ -319,6 +319,63 @@ const create$5 = context => {
 };
 
 const noLazyNthVariableName = {
+  __proto__: null,
+  create: create$6,
+  meta: meta$6
+};
+
+const meta$5 = {
+  docs: {
+    description: 'Disallow exporting skip with a value of zero'
+  },
+  fixable: 'code',
+  messages: {
+    noSkipZero: 'Omit the skip export when the test should run.'
+  },
+  type: 'suggestion'
+};
+const isSkipZeroDeclaration = node => {
+  const {
+    declaration
+  } = node;
+  if (declaration?.type !== 'VariableDeclaration' || declaration.kind !== 'const' || declaration.declarations.length !== 1) {
+    return false;
+  }
+  const [{
+    id,
+    init
+  }] = declaration.declarations;
+  return id.type === 'Identifier' && id.name === 'skip' && init?.type === 'Literal' && init.value === 0;
+};
+const getRemovalRange = (node, sourceCode) => {
+  if (!node.range) {
+    return undefined;
+  }
+  const trailingLineBreaks = /^(?:[\t ]*\r?\n)+/.exec(sourceCode.text.slice(node.range[1]))?.[0].length ?? 0;
+  return [node.range[0], node.range[1] + trailingLineBreaks];
+};
+const create$5 = context => {
+  const {
+    sourceCode
+  } = context;
+  return {
+    ExportNamedDeclaration(node) {
+      if (!isSkipZeroDeclaration(node)) {
+        return;
+      }
+      context.report({
+        fix(fixer) {
+          const range = getRemovalRange(node, sourceCode);
+          return range ? fixer.removeRange(range) : fixer.remove(node);
+        },
+        messageId: 'noSkipZero',
+        node
+      });
+    }
+  };
+};
+
+const noSkipZero = {
   __proto__: null,
   create: create$5,
   meta: meta$5
@@ -596,6 +653,7 @@ const plugin = {
     'no-inline-locator-in-expect': noInlineLocatorInExpect,
     'no-inline-nth-in-expect': noInlineNthInExpect,
     'no-lazy-nth-variable-name': noLazyNthVariableName,
+    'no-skip-zero': noSkipZero,
     'prefer-direct-api-destructuring': preferDirectApiDestructuring,
     'prefer-execute-extension-command': preferExecuteExtensionCommand,
     'prefer-filesystem-set-files': preferFileSystemSetFiles,
@@ -614,6 +672,7 @@ const recommended = [{
     'e2e/no-inline-locator-in-expect': 'error',
     'e2e/no-inline-nth-in-expect': 'error',
     'e2e/no-lazy-nth-variable-name': 'error',
+    'e2e/no-skip-zero': 'error',
     'e2e/prefer-direct-api-destructuring': 'error',
     'e2e/prefer-execute-extension-command': 'error',
     'e2e/prefer-filesystem-set-files': 'error',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lvce-editor/eslint-plugin-e2e",
-  "version": "17.8.0",
+  "version": "18.1.0",
   "main": "dist/index.js",
   "repository": {
     "type": "git",