Merge pull request #3 from alenap93/add-CI

Create ci.yml

.github/workflows/ci.yml

@@ -0,0 +1,90 @@
+name: CI
+
+on:
+  push:
+    branches:
+     - main
+     - master
+     - next
+     - 'v*'
+    paths-ignore:
+      - 'docs/**'
+      - '*.md'
+  pull_request:
+    paths-ignore:
+      - 'docs/**'
+      - '*.md'
+
+jobs:
+  dependency-review:
+    name: Dependency Review
+    if: github.event_name == 'pull_request'
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+    steps:
+      - name: Check out repo
+        uses: actions/checkout@v4
+        with:
+          persist-credentials: false
+
+      - name: Dependency review
+        uses: actions/dependency-review-action@v4
+
+  linter:
+    name: Lint Code
+    if: >
+      !failure() &&
+      !cancelled()
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+    steps:
+      - name: Check out repo
+        uses: actions/checkout@v4
+        with:
+          persist-credentials: false
+
+      - name: Setup Node
+        uses: actions/setup-node@v4
+        with:
+          node-version: lts/*
+
+      - name: Install dependencies
+        run: npm i --ignore-scripts
+
+      - name: Lint code
+        run: npm run lint
+
+  test:
+    name: Test
+    runs-on: ${{ matrix.os }}
+    permissions:
+      contents: read
+    strategy:
+      matrix:
+        node-version: ["20", "22"]
+        os: [macos-latest, ubuntu-latest, windows-latest]
+        exclude:
+          - os: macos-latest
+            node-version: 14
+          - os: macos-latest
+            node-version: 16
+    steps:
+      - name: Check out repo
+        uses: actions/checkout@v4
+        with:
+          persist-credentials: false
+
+      - name: Setup Node ${{ matrix.node-version }}
+        uses: actions/setup-node@v4
+        with:
+          node-version: ${{ matrix.node-version }}
+
+      - name: Install dependencies
+        run: npm i
+
+      - name: Run tests
+        run: npm test
+        env:
+          NODE_AUTH_TOKEN: ${{secrets.npm_token}}

index.js

@@ -5,8 +5,6 @@ const fp = require('fastify-plugin')
 function fastifyKysely (fastify, options, next) {
   const { namespace, kysely } = options
 
-  const client = options.client || null
-
   if (!namespace) {
     return next(new Error('Namespace not defined'))
   }

test/index.test.js

@@ -57,12 +57,6 @@ test('fastify.kysely should throw error if kysely instance not defined', (t) =>
   t.plan(1)
   const fastify = Fastify()
 
-  const sqliteDialectOne = new SqliteDialect({
-    database: new Database(':memory:')
-  })
-
-  const kyselyInstanceOne = new Kysely({ dialect: sqliteDialectOne })
-
   fastify.register(fastifyKysely, {
     namespace: 'one'
   })