Update to next.js and typescript (#152)

Author: dlebech

.codecov.yml

@@ -5,9 +5,9 @@ name: Node.js CI
 
 on:
   push:
-    branches: [ master ]
+    branches: [ main ]
   pull_request:
-    branches: [ master ]
+    branches: [ main ]
 
 jobs:
   build:
@@ -16,15 +16,17 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [14.x, 16.x, 18.x]
+        node-version: [24.x]
 
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v6
     - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v2
+      uses: actions/setup-node@v6
       with:
         node-version: ${{ matrix.node-version }}
         cache: npm
     - run: npm ci
     - run: npm test -- --silent --coverage --coverageReporters text lcov
-    - run: bash <(curl -s https://codecov.io/bash)
+    - run: npm run lint
+    - run: npm run build
+

.github/workflows/node.js.yml

@@ -28,4 +28,9 @@ test_examples/
 !.yarn/plugins
 !.yarn/sdks
 !.yarn/versions
-.pnp.*
+.pnp.*
+
+.next/
+*.tsbuildinfo
+next-env.d.ts
+.claude/settings.local.json

.gitignore

@@ -0,0 +1,28 @@
+import fs from 'fs';
+import path from 'path';
+import axios from 'axios';
+import MockAdapter from 'axios-mock-adapter';
+import { GET } from '../src/app/api/currencies/route';
+import { NextRequest } from 'next/server';
+
+const daily = fs.readFileSync(path.join(__dirname, './daily.zip'));
+
+describe('currencies API', () => {
+  let mock: MockAdapter;
+  
+  beforeEach(() => {
+    mock = new MockAdapter(axios);
+    mock.onGet(/.*eurofxref\.zip$/).reply(200, daily);
+  });
+
+  afterEach(() => mock.restore());
+
+  it('should return supported currencies', async () => {
+    const request = new NextRequest('http://localhost:3000/api/currencies');
+    const response = await GET(request);
+    const currencies = await response.json();
+    
+    expect(response.status).toBe(200);
+    expect(currencies).toEqual(['USD', 'JPY']);
+  });
+});

.nvmrc

@@ -2,13 +2,15 @@ import fs from 'fs';
 import path from 'path';
 import axios from 'axios';
 import MockAdapter from 'axios-mock-adapter';
-import { handler } from '../currencyRates';
+import { GET } from '../src/app/api/currencyRates/route';
+import { NextRequest } from 'next/server';
 
 const daily = fs.readFileSync(path.join(__dirname, './daily.zip'));
 const historical = fs.readFileSync(path.join(__dirname, './hist.zip'));
 
-describe('currencyRates', () => {
-  let mock;
+describe('currencyRates API', () => {
+  let mock: MockAdapter;
+  
   beforeEach(() => {
     mock = new MockAdapter(axios);
     mock.onGet(/.*eurofxref\.zip$/).reply(200, daily);
@@ -18,8 +20,11 @@ describe('currencyRates', () => {
   afterEach(() => mock.restore());
 
   it('should return daily rates', async () => {
-    const res = await handler();
-    const rates = {
+    const request = new NextRequest('http://localhost:3000/api/currencyRates');
+    const response = await GET(request);
+    const rates = await response.json();
+    
+    const expectedRates = {
       '2018-09-17': {
         USD: 1.1691,
         JPY: 130.77
@@ -34,19 +39,16 @@ describe('currencyRates', () => {
       }
     };
 
-    expect(res).toEqual({
-      statusCode: 200,
-      body: JSON.stringify(rates)
-    });
+    expect(response.status).toBe(200);
+    expect(rates).toEqual(expectedRates);
   });
 
   it('should return specific currencies', async () => {
-    const res = await handler({
-      queryStringParameters: {
-        currencies: ['USD']
-      }
-    });
-    const rates = {
+    const request = new NextRequest('http://localhost:3000/api/currencyRates?currencies=USD');
+    const response = await GET(request);
+    const rates = await response.json();
+    
+    const expectedRates = {
       '2018-09-17': {
         USD: 1.1691
       },
@@ -58,9 +60,7 @@ describe('currencyRates', () => {
       }
     };
 
-    expect(res).toEqual({
-      statusCode: 200,
-      body: JSON.stringify(rates)
-    });
+    expect(response.status).toBe(200);
+    expect(rates).toEqual(expectedRates);
   });
-});
+});

__tests__/currencies.test.ts

@@ -0,0 +1,18 @@
+import { defineConfig, globalIgnores } from "eslint/config";
+import nextVitals from "eslint-config-next/core-web-vitals";
+import nextTs from "eslint-config-next/typescript";
+
+const eslintConfig = defineConfig([
+  ...nextVitals,
+  ...nextTs,
+  // Override default ignores of eslint-config-next.
+  globalIgnores([
+    // Default ignores of eslint-config-next:
+    ".next/**",
+    "out/**",
+    "build/**",
+    "next-env.d.ts",
+  ]),
+]);
+
+export default eslintConfig;

src/lambda/__tests__/currencyRates.js __tests__/currencyRates.test.tssrc/lambda/__tests__/currencyRates.js renamed to __tests__/currencyRates.test.ts

@@ -0,0 +1,31 @@
+import type { Config } from 'jest'
+import nextJest from 'next/jest.js'
+import { createRequire } from 'module'
+
+const require = createRequire(import.meta.url)
+
+const createJestConfig = nextJest({
+  dir: './',
+})
+
+const config: Config = {
+  coverageProvider: 'v8',
+  testEnvironment: 'jsdom',
+  setupFilesAfterEnv: ['<rootDir>/jest.setup.ts'],
+  moduleNameMapper: {
+    '^@/(.*)$': '<rootDir>/src/$1',
+    '^uuid$': require.resolve('uuid'),
+  },
+  testPathIgnorePatterns: ['<rootDir>/.next/', '<rootDir>/node_modules/'],
+  transformIgnorePatterns: [
+    'node_modules/(?!(uuid|d3-array|d3-scale-chromatic|react-dates)/)',
+  ],
+  extensionsToTreatAsEsm: ['.ts', '.tsx'],
+  globals: {
+    'ts-jest': {
+      useESM: true,
+    },
+  },
+}
+
+export default createJestConfig(config)