Feature: Add jest-playwright as an ESLint environment

[playwright-community]

Feature: Add jest-playwright as an ESLint environment

[mxschmitt]

@mxschmitt I verified that it works. A question though. The implementation has this:

module.exports = {
  configs: {
    recommended: {
      env: {
        node: true,
        jest: true,
      },
      globals: {
        browser: true,
        context: true,
        page: true,
        jestPlaywright: true,
      }
    }
  },
}

Isn't this more proper?

module.exports = {
  configs: {
    recommended: {
      env: {
        shared-node-browser: true,
        jest: true,
      },
      globals: {
        browser: true,
        context: true,
        page: true,
        jestPlaywright: true,
      }
    }
  },
}

Reason being that you can just document. in many of the Playwright functions that runs inside the browser.

good catch, will adjust this in a bit! :)

[mmarkelov]

@thernstig that's make sense

[mxschmitt]

@thernstig could you try https://github.com/mxschmitt/eslint-plugin-jest-playwright?
Should fit for your needs, if yes then we can refer to this in the README.

[thernstig]

@mxschmitt I verified that it works. A question though. The implementation has this:

module.exports = {
  configs: {
    recommended: {
      env: {
        node: true,
        jest: true,
      },
      globals: {
        browser: true,
        context: true,
        page: true,
        jestPlaywright: true,
      }
    }
  },
}

Isn't this more proper?

module.exports = {
  configs: {
    recommended: {
      env: {
        shared-node-browser: true,
        jest: true,
      },
      globals: {
        browser: true,
        context: true,
        page: true,
        jestPlaywright: true,
      }
    }
  },
}

Reason being that you can just document. in many of the Playwright functions that runs inside the browser.