feat: use bun for tests and bundling

.github/workflows/release.yml

@@ -7,92 +7,58 @@ on:
 
 permissions: write-all
 
+# Cancel previously running workflows
+concurrency:
+  group: ci-${{ github.ref }}
+  cancel-in-progress: true
+
 jobs:
   install:
     name: Install dependencies
     runs-on: ubuntu-latest 
     steps:
-      - name: Cancel Previous Runs
-        uses: styfle/cancel-workflow-action@0.9.1
-        with:
-          access_token: ${{ secrets.GITHUB_TOKEN }}
-
-      - name: Checkout code
-        uses: actions/checkout@v3
-
-      - name: Download dependencies
-        uses: bahmutov/npm-install@v1
+      - uses: actions/checkout@v4
+      - uses: oven-sh/setup-bun@v1
+      - run: bun install
 
   test:
     name: Tests
     runs-on: ubuntu-latest 
     needs: [install]
     steps:
-      - name: Cancel Previous Runs
-        uses: styfle/cancel-workflow-action@0.9.1
-        with:
-          access_token: ${{ secrets.GITHUB_TOKEN }}
-
-      - name: Checkout code
-        uses: actions/checkout@v3
-
-      - name: Download dependencies
-        uses: bahmutov/npm-install@v1
-
-      - name: Run tests
-        run: npm test
+      - uses: actions/checkout@v4
+      - uses: oven-sh/setup-bun@v1
+      - run: bun install
+      - run: bun test
 
   lint:
     name: Lint
     runs-on: ubuntu-latest 
     needs: [install]
     steps:
-      - name: Cancel Previous Runs
-        uses: styfle/cancel-workflow-action@0.9.1
-        with:
-          access_token: ${{ secrets.GITHUB_TOKEN }}
-
-      - name: Checkout code
-        uses: actions/checkout@v3
-
-      - name: Download dependencies
-        uses: bahmutov/npm-install@v1
-
-      - name: Run linting
-        run: npm run lint
+      - uses: actions/checkout@v4
+      - uses: oven-sh/setup-bun@v1
+      - run: bun install
+      - run: bun lint
 
   typecheck:
     name: TypeScript
     needs: [install]
     runs-on: ubuntu-latest
     steps:
-      - name: Cancel Previous Runs
-        uses: styfle/cancel-workflow-action@0.9.1
-
-      - name: Checkout code
-        uses: actions/checkout@v3
-
-      - name: Download dependencies
-        uses: bahmutov/npm-install@v1
-
-      - name: Type check
-        run: npm run typecheck
+      - uses: actions/checkout@v4
+      - uses: oven-sh/setup-bun@v1
+      - run: bun install
+      - run: bun typecheck
 
   release:
     runs-on: ubuntu-latest
     needs: [test, lint, typecheck]
     steps:
-      - name: Cancel Previous Runs
-        uses: styfle/cancel-workflow-action@0.9.1
-
-      - name: Checkout code
-        uses: actions/checkout@v3
-
-      - name: Download dependencies
-        uses: bahmutov/npm-install@v1
-
-      - name: Build project
-        run: npm run build
+      - uses: actions/checkout@v4
+      - uses: oven-sh/setup-bun@v1
+      - run: bun install
+      - run: bun build:js
 
       - name: Semantic Release
         uses: cycjimmy/semantic-release-action@v2

lib/__tests__/index.spec.ts

@@ -1,4 +1,4 @@
-import { expect, test } from 'vitest'
+import { expect, test } from 'bun:test'
 import telefonnummer, {
   areaCode,
   areEqual,

lib/__tests__/numberingArea.spec.ts

@@ -1,4 +1,4 @@
-import { expect, test } from 'vitest'
+import { expect, test } from 'bun:test'
 import { numberingArea } from '../numberingArea'
 
 test('returns empty string if nothing is found', () => {

lib/__tests__/parser.spec.ts

@@ -1,4 +1,4 @@
-import { expect, test } from 'vitest'
+import { expect, test } from 'bun:test'
 import { phoneNumberParser } from '../parser'
 
 const parserHelper = (numbers: string[], expected: string): void => {
@@ -7,10 +7,6 @@ const parserHelper = (numbers: string[], expected: string): void => {
   })
 }
 
-test('handles null values', () => {
-  parserHelper([null, undefined], '')
-})
-
 test('parses voicemail', () => {
   const numbers = [
     '46888',
@@ -26,10 +22,6 @@ test('parses voicemail', () => {
   parserHelper(numbers, 'Röstbrevlåda')
 })
 
-test('handles non strings', () => {
-  expect(phoneNumberParser(46721834008)).toEqual('072-183 40 08')
-})
-
 test('parses normal mobile numbers', () => {
   expect(phoneNumberParser('46721834008')).toEqual('072-183 40 08')
   expect(phoneNumberParser('0721834008')).toEqual('072-183 40 08')

lib/__tests__/riktnummer.spec.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test } from 'vitest'
+import { describe, expect, test } from 'bun:test'
 import * as riktnummer from '../riktnummer'
 
 describe('#findMatchingAreaCode', () => {

lib/__tests__/validator.spec.ts

@@ -1,12 +1,12 @@
-import { expect, test } from 'vitest'
+import { expect, test } from 'bun:test'
 import { validator } from '../validator'
 
 test.each([
   ['070-abc de fg'],
   ['0701ggas234asdf567d'],
   ['070abcdefb'],
   ['abcdefg'],
-])('errors for non-numerical inputs', (phoneNumber) => {
+])('errors for non-numerical input %s', (phoneNumber) => {
   expect(validator(phoneNumber)).toBe(false)
 })
 

lib/utils/__tests__/areaCodeDigitCount.spec.ts

@@ -1,4 +1,4 @@
-import { expect, test } from 'vitest'
+import { expect, test } from 'bun:test'
 import { areaCodeDigitCount } from '../areaCodeDigitCount'
 
 const numberHelper = (numbers: string[], expected: number): void => {

lib/utils/__tests__/comparePhoneNumbers.spec.ts

@@ -1,4 +1,4 @@
-import { expect, test } from 'vitest'
+import { expect, test } from 'bun:test'
 import { phoneNumbersAreEqual } from '../comparePhoneNumbers'
 
 test('returns true if phone numbers are equal', () => {

lib/utils/__tests__/makeRegex.spec.ts

@@ -1,4 +1,4 @@
-import { expect, test } from 'vitest'
+import { expect, test } from 'bun:test'
 import { makeRegex } from '../makeRegex'
 
 test('handles undefined type (long implicitly)', () => {

lib/utils/__tests__/normalize.spec.ts

@@ -1,4 +1,4 @@
-import { expect, test } from 'vitest'
+import { expect, test } from 'bun:test'
 import { normalize } from '../normalize'
 
 test('should handle numbers with +46', () => {

lib/utils/__tests__/numberReplace.spec.ts

@@ -1,4 +1,4 @@
-import { expect, test } from 'vitest'
+import { expect, test } from 'bun:test'
 import { numberReplace } from '../numberReplace'
 
 test('handles undefined type (long implicitly)', () => {