ci: expand checks to include prettier and eslint

.github/workflows/ci.yml

@@ -0,0 +1,37 @@
+name: CI Checks
+on:
+  push:
+    branches: [main]
+  pull_request:
+    branches: [main]
+jobs:
+  code_checks:
+    timeout-minutes: 10
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
+        with:
+          node-version: 18
+
+      - name: Install dependencies
+        run: yarn
+
+      - name: Prettier
+        run: npm run prettier
+
+      - name: ESLint
+        run: npm run lint
+
+      - name: Install Playwright browsers
+        run: yarn playwright:install
+      - name: Run Playwright tests
+        run: yarn playwright:test
+        env:
+          VITE_PROJECT_ID: ${{ secrets.VITE_DEV_PROJECT_ID }}
+      - uses: actions/upload-artifact@v3
+        if: always()
+        with:
+          name: playwright-report
+          path: playwright-report/
+          retention-days: 30

playwright.config.ts

@@ -1,12 +1,12 @@
-import { defineConfig, devices } from '@playwright/test';
+import { defineConfig, devices } from '@playwright/test'
 
 /**
  * Read environment variables from file.
  * https://github.com/motdotla/dotenv
  */
 // require('dotenv').config();
 
-const baseURL = 'http://localhost:5173';
+const baseURL = 'http://localhost:5173'
 
 /**
  * See https://playwright.dev/docs/test-configuration.
@@ -16,7 +16,7 @@
   /* Run tests in files in parallel */
   fullyParallel: true,
   /* Fail the build on CI if you accidentally left test.only in the source code. */
   forbidOnly: !!process.env.CI,
   /* Retry on CI only */
   retries: process.env.CI ? 2 : 0,
   /* Opt out of parallel tests on CI. */
@@ -31,40 +31,40 @@
     /* Collect trace when retrying the failed test. See https://playwright.dev/docs/trace-viewer */
     trace: 'on-first-retry',
 
-    screenshot: 'only-on-failure',
+    screenshot: 'only-on-failure'
   },
 
   /* Configure projects for major browsers */
   projects: [
     {
       name: 'chromium',
-      use: { ...devices['Desktop Chrome'] },
+      use: { ...devices['Desktop Chrome'] }
     },
 
     {
       name: 'firefox',
-      use: { ...devices['Desktop Firefox'] },
+      use: { ...devices['Desktop Firefox'] }
     },
 
     {
       name: 'webkit',
-      use: { ...devices['Desktop Safari'] },
-    },
+      use: { ...devices['Desktop Safari'] }
+    }
 
     /* Test against mobile viewports. */
     // {
     //   name: 'Mobile Chrome',
     //   use: { ...devices['Pixel 5'] },
     // },
     // {
     //   name: 'Mobile Safari',
     //   use: { ...devices['iPhone 12'] },
     // },
 
     /* Test against branded browsers. */
     // {
     //   name: 'Microsoft Edge',
     //   use: { ...devices['Desktop Edge'], channel: 'msedge' },
     // },
     // {
     //   name: 'Google Chrome',
@@ -76,6 +76,6 @@
   webServer: {
     command: 'yarn dev',
     url: baseURL,
-    reuseExistingServer: !process.env.CI,
-  },
-});
+    reuseExistingServer: !process.env.CI
+  }
+})

src/constants/projects.ts

@@ -35,7 +35,8 @@ export const COMING_SOON_PROJECTS: Array<INotifyApp> = [
   {
     id: 'chainspot',
     name: 'Chainspot',
-    description: 'Bridge&swap across 27 chains at the best rates and research data about 100+ Web3 products.',
+    description:
+      'Bridge&swap across 27 chains at the best rates and research data about 100+ Web3 products.',
     url: 'https://app.chainspot.io/',
     isComingSoon: true,
     isVerified: false,

src/contexts/W3iContext/hooks/notifyHooks.ts

@@ -51,15 +51,15 @@ export const useNotifyState = (w3iProxy: Web3InboxProxy, proxyReady: boolean) =>
    * load in progress state using interval until it is
    */
   useEffect(() => {
-    if(watchSubscriptionsComplete) {
-      return noop;
+    if (watchSubscriptionsComplete) {
+      return noop
     }
     // Account for sync init
     const intervalId = setInterval(() => {
-    if (notifyClient?.hasFinishedInitialLoad()) {
-      setWatchSubscriptionsComplete(true)
-      return noop;
-    }
+      if (notifyClient?.hasFinishedInitialLoad()) {
+        setWatchSubscriptionsComplete(true)
+        return noop
+      }
       refreshNotifyState()
     }, 100)
 

tests/home.spec.ts

@@ -1,11 +1,11 @@
-import { test, expect } from '@playwright/test';
+import { expect, test } from '@playwright/test'
 
 test('has title', async ({ page }) => {
-  await page.goto("/");
-  await expect(page).toHaveTitle(/Web3Inbox/);
-});
+  await page.goto('/')
+  await expect(page).toHaveTitle(/Web3Inbox/)
+})
 
 test('welcome message', async ({ page }) => {
-  await page.goto("/");
-  await expect(page.getByText("Welcome to Web3Inbox")).toBeVisible();
-});
+  await page.goto('/')
+  await expect(page.getByText('Welcome to Web3Inbox')).toBeVisible()
+})