refactor: Update test environments and lint configuration

.eslintignore

@@ -1,10 +1,9 @@
 .git
-packages/docsify-server-renderer/build.js
-node_modules
+**/*.md
 build
-server.js
+docs
 lib
+node_modules
+packages/docsify-server-renderer/build.js
+server.js
 themes
-build
-docs/
-**/*.md

.eslintrc.js

@@ -1,24 +1,28 @@
+const prettierConfig = require('./.prettierrc');
+
 module.exports = {
  root: true,
- parser: 'babel-eslint',
+ extends: [
+ 'eslint:recommended',
+ 'plugin:import/recommended',
+ 'plugin:prettier/recommended', // Must be last
+ ],
+ parser: '@babel/eslint-parser',
  parserOptions: {
  sourceType: 'module',
  ecmaVersion: 2019,
  },
+ plugins: ['prettier', 'import'],
  env: {
  browser: true,
- node: true,
  es6: true,
+ node: true,
  },
- plugins: ['prettier', 'import'],
- extends: ['eslint:recommended', 'plugin:import/recommended'],
  settings: {
  'import/ignore': ['node_modules', '.json$'],
  },
  rules: {
- 'prettier/prettier': ['error'],
  camelcase: ['warn'],
- 'no-useless-escape': ['warn'],
  curly: ['error', 'all'],
  'dot-notation': ['error'],
  eqeqeq: ['error'],
@@ -33,9 +37,11 @@ module.exports = {
  'no-proto': ['error'],
  'no-return-assign': ['error'],
  'no-self-compare': ['error'],
- 'no-shadow': ['warn'],
  'no-shadow-restricted-names': ['error'],
+ 'no-shadow': ['warn'],
+ 'no-unused-vars': ['error', { args: 'none' }],
  'no-useless-call': ['error'],
+ 'no-useless-escape': ['warn'],
  'no-var': ['error'],
  'no-void': ['error'],
  'no-with': ['error'],
@@ -46,18 +52,21 @@ module.exports = {
 
  // Import rules
  // Search way how integrate with `lerna`
- 'import/no-unresolved': 'off',
  'import/imports-first': ['error'],
  'import/newline-after-import': ['error'],
  'import/no-duplicates': ['error'],
  'import/no-mutable-exports': ['error'],
- 'import/no-named-as-default': ['error'],
  'import/no-named-as-default-member': ['error'],
+ 'import/no-named-as-default': ['error'],
+ 'import/no-unresolved': 'off',
  'import/order': ['warn'],
+
+ // Prettier (Must be last)
+ 'prettier/prettier': ['warn', prettierConfig],
  },
  globals: {
- Docsify: 'writable',
  $docsify: 'writable',
+ Docsify: 'writable',
  dom: 'writable',
  },
 };

.github/workflows/lint.yml

@@ -2,29 +2,25 @@ name: Lint
 
 on:
  push:
- branches:
- - master
- - develop
+ branches: [master, develop]
  pull_request:
- branches:
- - master
- - develop
+ branches: [master, develop]
 
 jobs:
- build:
+ lint:
  runs-on: ubuntu-latest
  strategy:
  fail-fast: false
  matrix:
- node-version: [12.x, 14.x]
+ node-version: [14.x, 16.x]
 
  steps:
  - uses: actions/checkout@v2
  - name: Use Node.js ${{ matrix.node-version }}
- uses: actions/setup-node@v1
+ uses: actions/setup-node@v2
  with:
  node-version: ${{ matrix.node-version }}
- - name: Install
+ - name: Install dependencies
  run: npm i --ignore-scripts
  - name: Lint
  run: npm run lint

.github/workflows/test.yml

@@ -2,41 +2,69 @@ name: Build & Test
 
 on:
  push:
- branches:
- - master
- - develop
+ branches: [master, develop]
  pull_request:
- branches:
- - master
- - develop
+ branches: [master, develop]
 
 jobs:
- build:
+ install:
+ runs-on: ubuntu-latest
+ strategy:
+ matrix:
+ node-version: [16]
+ steps:
+ - uses: actions/checkout@v2
+ - name: Use Node.js ${{ matrix.node-version }}
+ uses: actions/setup-node@v2
+ with:
+ node-version: ${{ matrix.node-version }}
+ cache: 'npm'
+ - run: npm install
+
+ lint:
+ needs: install
+ runs-on: ubuntu-latest
+ strategy:
+ matrix:
+ node-version: [16]
+ steps:
+ - uses: actions/checkout@v2
+ - name: Use Node.js ${{ matrix.node-version }}
+ uses: actions/setup-node@v2
+ with:
+ node-version: ${{ matrix.node-version }}
+ cache: 'npm'
+ - name: Lint
+ run: npm run lint
+
+ test:
+ needs: install
  runs-on: ${{ matrix.os }}
  strategy:
- fail-fast: false
  matrix:
- node-version: [12.x, 14.x]
+ node-version: [16]
  os: ['macos-latest', 'ubuntu-latest', 'windows-latest']
-
  steps:
  - uses: actions/checkout@v2
  - name: Use Node.js ${{ matrix.node-version }}
- uses: actions/setup-node@v1
+ uses: actions/setup-node@v2
  with:
  node-version: ${{ matrix.node-version }}
- - name: bootstrap
+ cache: 'npm'
+ - name: Bootstrap
  run: npm run bootstrap
- - name: unit tests
+ - name: Unit Tests (Jest)
  run: npm run test:unit -- -ci --runInBand
- - name: integration tests
+ - name: Integration Tests (Jest)
  run: npm run test:integration -- -ci --runInBand
- - uses: microsoft/playwright-github-action@v1
- - name: e2e tests
-  run: npm run test:e2e -- --ci --runInBand
- - name: Upload artifacts (diff output)
-   uses: actions/upload-artifact@v2
+ - name: Install Playwright
+  run: npx playwright install --with-deps
+ - name: E2E Tests (Playwright)
+  run: npm run test:e2e
+ - uses: actions/upload-artifact@v2
  if: failure()
  with:
- name: ${{ matrix.os }}-${{ matrix.node-version }}-diff-output
- path: ${{ github.workspace }}/test/**/__diff_output__/*
+ name: ${{ matrix.os }}-${{ matrix.node-version }}-artifacts
+ path: |
+ _playwright-results/
+ _playwright-report/

.gitignore

@@ -1,10 +1,11 @@
-*.log
 .DS_Store
 .idea
-__diff_output__
-lib/
+*.log
+_playwright-report
+_playwright-results
+lib
 node_modules
-themes/
+themes
 
 # exceptions
 !.gitkeep

.npmignore

@@ -1,5 +1,5 @@
 .eslintignore
 .eslintrc
-.github/
+.github
 .gitignore
 .travis.yml

.prettierrc.js

@@ -1,4 +1,4 @@
 module.exports = {
+ arrowParens: 'avoid',
  singleQuote: true,
- trailingComma: 'es5',
 };

.vscode/settings.json

@@ -1,3 +1,4 @@
 {
- "editor.defaultFormatter": "esbenp.prettier-vscode"
+ "editor.defaultFormatter": "esbenp.prettier-vscode",
+ "cSpell.words": ["coverpage"]
 }

jest.config.js

@@ -2,60 +2,28 @@ const { TEST_HOST } = require('./test/config/server.js');
 
 const sharedConfig = {
  errorOnDeprecated: true,
- globals: {
- TEST_HOST,
- },
  globalSetup: './test/config/jest.setup.js',
  globalTeardown: './test/config/jest.teardown.js',
  resetModules: true,
  restoreMocks: true,
+ setupFilesAfterEnv: ['<rootDir>/test/config/jest.setup-tests.js'],
+ testEnvironment: 'jsdom',
+ testURL: `${TEST_HOST}/_blank.html`,
 };
 
 module.exports = {
- // Adding globals to config root for easier importing into .eslint.js, but
- // as of Jest 26.4.2 these globals need to be added to each project config
- // as well.
- globals: sharedConfig.globals,
  projects: [
- // Unit Tests (Jest)
+ // Unit Tests
  {
- ...sharedConfig,
  displayName: 'unit',
- setupFilesAfterEnv: ['<rootDir>/test/config/jest.setup-tests.js'],
+ ...sharedConfig,
  testMatch: ['<rootDir>/test/unit/*.test.js'],
- testURL: `${TEST_HOST}/_blank.html`,
  },
- // Integration Tests (Jest)
+ // Integration Tests
  {
- ...sharedConfig,
  displayName: 'integration',
- setupFilesAfterEnv: ['<rootDir>/test/config/jest.setup-tests.js'],
- testMatch: ['<rootDir>/test/integration/*.test.js'],
- testURL: `${TEST_HOST}/_blank.html`,
- },
- // E2E Tests (Jest + Playwright)
- {
  ...sharedConfig,
- displayName: 'e2e',
- preset: 'jest-playwright-preset',
- setupFilesAfterEnv: [
- '<rootDir>/test/config/jest-playwright.setup-tests.js',
- ],
- testEnvironmentOptions: {
- 'jest-playwright': {
- // prettier-ignore
- browsers: [
- 'chromium',
- 'firefox',
- 'webkit',
- ],
- launchOptions: {
- // headless: false,
- // devtools: true,
- },
- },
- },
- testMatch: ['<rootDir>/test/e2e/*.test.js'],
+ testMatch: ['<rootDir>/test/integration/*.test.js'],
  },
  ],
 };