Skip to content

Commit

Permalink
feat(github-actions): extract deploy & e2e job for deployable flag
Browse files Browse the repository at this point in the history
  • Loading branch information
mildronize committed May 6, 2024
1 parent 3d0a624 commit 75717e5
Show file tree
Hide file tree
Showing 5 changed files with 167 additions and 59 deletions.
145 changes: 145 additions & 0 deletions .github/workflows/e2e.backup.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,145 @@
name: 'E2E Tests (backup)'

on:
workflow_dispatch:
# push:
# branches: [v3.x-bundle]
# paths-ignore:
# - '**/*.md'
# pull_request:
# branches: [ main ]
# paths-ignore:
# - '**/*.md'

env:
pnpm_version: 8
node_version: 20
debug_identifier: nammatham:*

RESOURCE_IDENTIFIER_NODE18_LINUX_X64: ${{ secrets.RESOURCE_IDENTIFIER_NODE18_LINUX_X64 }}

jobs:
get-matrix:
runs-on: ubuntu-latest
timeout-minutes: 4
outputs:
matrix: ${{ steps.deploy-matrix.outputs.matrix }}
deployable_matrix: ${{ steps.deploy-matrix.outputs.deployable_matrix }}
steps:
- uses: actions/checkout@v4
- uses: oven-sh/setup-bun@v1

- name: Use Node.js ${{ env.node_version }}
uses: actions/setup-node@v4
with:
node-version: ${{env.node_version }}
- uses: pnpm/action-setup@v3
name: Install pnpm
with:
version: ${{ env.pnpm_version }}

- run: pnpm install
- name: Export the matrix for testing
id: deploy-matrix
run: pnpm exec nx run @infra/azure-functions:github-actions

build:
runs-on: ${{ matrix.os }}
needs: get-matrix
timeout-minutes: 10
strategy:
matrix:
include: ${{fromJson(needs.get-matrix.outputs.matrix)}}

steps:
- uses: actions/checkout@v4
- uses: oven-sh/setup-bun@v1

- name: Use Node.js ${{ matrix.version }}
if: matrix.runtime == 'node'
uses: actions/setup-node@v4
with:
node-version: ${{ matrix.version }}

- uses: pnpm/action-setup@v3
name: Install pnpm
with:
version: ${{ env.pnpm_version }}
- run: pnpm install

- run: pnpm build

- name: pnpm install again for ensure the cli is available
run: pnpm install

- name: Build the project
run: pnpm exec nx run @examples/with-${{ matrix.runtime }}:build
env:
DEBUG: ${{ env.debug_identifier }}

- name: Upload Artifact
uses: actions/upload-artifact@v4
with:
name: ${{ matrix.resource_identifier_key }}
path: examples/with-${{ matrix.runtime }}/.nmt/dist
retention-days: 1

deploy:
runs-on: ubuntu-latest
needs:
- build
- get-matrix
timeout-minutes: 10
strategy:
matrix:
include: ${{fromJson(needs.get-matrix.outputs.deployable_matrix)}}

steps:
- name: Azure Login
uses: azure/login@v2
with:
creds: ${{ secrets.AZURE_CREDENTIALS_E2E_TESTS }}

- name: Download Artifact
uses: actions/download-artifact@v4
with:
name: ${{ matrix.resource_identifier_key }}
path: examples/with-${{ matrix.runtime }}/.nmt/dist

- name: Deploy to Azure Functions
uses: Azure/functions-action@v1
with:
app-name: nmt-e2e-${{ matrix.target }}-${{ secrets[matrix.resource_identifier_key] }}
package: .

e2e:
runs-on: ubuntu-latest
needs:
- build
- deploy
- get-matrix
timeout-minutes: 10
strategy:
matrix:
include: ${{fromJson(needs.get-matrix.outputs.deployable_matrix)}}

steps:
- uses: actions/checkout@v4
- uses: oven-sh/setup-bun@v1

- name: Use Node.js ${{ env.node_version }}
uses: actions/setup-node@v4
with:
node-version: ${{ env.node_version }}

- uses: pnpm/action-setup@v3
name: Install pnpm
with:
version: ${{ env.pnpm_version }}
- run: pnpm install

- name: Run E2E tests
run: pnpm exec nx run @infra/azure-functions:test
env:
AZURE_FUNCTIONS_URL: https://nmt-e2e-${{ matrix.target }}-${{ secrets[matrix.resource_identifier_key] }}.azurewebsites.net
AZURE_FUNCTIONS_API_KEY: ${{ secrets.AZURE_FUNCTIONS_HOST_KEY }}
50 changes: 6 additions & 44 deletions .github/workflows/e2e.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,10 +20,8 @@ env:
jobs:
get-matrix:
runs-on: ubuntu-latest
timeout-minutes: 4
outputs:
matrix: ${{ steps.deploy-matrix.outputs.matrix }}
deployable_matrix: ${{ steps.deploy-matrix.outputs.deployable_matrix }}
steps:
- uses: actions/checkout@v4
- uses: oven-sh/setup-bun@v1
Expand All @@ -45,7 +43,6 @@ jobs:
build:
runs-on: ${{ matrix.os }}
needs: get-matrix
timeout-minutes: 10
strategy:
matrix:
include: ${{fromJson(needs.get-matrix.outputs.matrix)}}
Expand Down Expand Up @@ -76,61 +73,26 @@ jobs:
env:
DEBUG: ${{ env.debug_identifier }}

- name: Upload Artifact
uses: actions/upload-artifact@v4
with:
name: ${{ matrix.resource_identifier_key }}
path: examples/with-${{ matrix.runtime }}/.nmt/dist
retention-days: 1

e2e:
runs-on: ubuntu-latest
needs:
- build
- get-matrix
timeout-minutes: 10
strategy:
matrix:
include: ${{fromJson(needs.get-matrix.outputs.deployable_matrix)}}

steps:
- uses: actions/checkout@v4
- uses: oven-sh/setup-bun@v1

- name: Use Node.js ${{ matrix.version }}
if: matrix.runtime == 'node'
uses: actions/setup-node@v3
with:
node-version: ${{ matrix.version }}

- uses: pnpm/action-setup@v2
name: Install pnpm
with:
version: ${{ env.pnpm_version }}
- run: pnpm install

- name: Azure Login
uses: azure/login@v2
if: matrix.is_deployable == true
with:
creds: ${{ secrets.AZURE_CREDENTIALS_E2E_TESTS }}

- name: Download Artifact
uses: actions/download-artifact@v4
with:
name: ${{ matrix.resource_identifier_key }}
path: examples/with-${{ matrix.runtime }}/.nmt/dist

- name: Deploy to Azure Functions
uses: Azure/functions-action@v1
if: matrix.is_deployable == true
with:
app-name: nmt-e2e-${{ matrix.target }}-${{ secrets[matrix.resource_identifier_key] }}
package: .
package: examples/with-${{ matrix.runtime }}/.nmt/dist

- name: Wait for the deployment to finish
if: matrix.is_deployable == true
run: sleep 15

- name: Run E2E tests
if: matrix.is_deployable == true
run: pnpm exec nx run @infra/azure-functions:test
env:
AZURE_FUNCTIONS_URL: https://nmt-e2e-${{ matrix.target }}-${{ secrets[matrix.resource_identifier_key] }}.azurewebsites.net
AZURE_FUNCTIONS_API_KEY: ${{ secrets.AZURE_FUNCTIONS_HOST_KEY }}
AZURE_FUNCTIONS_API_KEY: ${{ secrets.AZURE_FUNCTIONS_HOST_KEY }}
6 changes: 3 additions & 3 deletions examples/with-node/nammatham.config.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -28,9 +28,9 @@ const nammathamConfig = {
excludedTypes: 'Request',
},
},
// logLevel: {
// default: 'Trace',
// },
logLevel: {
default: 'Trace',
},
},
},
};
Expand Down
22 changes: 11 additions & 11 deletions infra/azure-functions/src/config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,17 +13,17 @@ export const infraConfigs = createInfraConfig(
runtime: 'node18',
isDeployable: true,
},
{
platform: 'win',
arch: 'x64',
runtime: 'node18',
isDeployable: true,
},
{
platform: 'macos',
arch: 'arm64',
runtime: 'node18',
},
// {
// platform: 'win',
// arch: 'x64',
// runtime: 'node18',
// isDeployable: true,
// },
// {
// platform: 'macos',
// arch: 'arm64',
// runtime: 'node18',
// },
// --- Bun ----
// {
// platform: 'linux',
Expand Down
3 changes: 2 additions & 1 deletion infra/azure-functions/src/e2e.test.ts
Original file line number Diff line number Diff line change
@@ -1,11 +1,12 @@
import { test, expect } from 'bun:test';
import supertest from 'supertest';

const url = process.env.AZURE_FUNCTIONS_URL;
const apiKey = process.env.AZURE_FUNCTIONS_API_KEY;
if (!url) throw new Error('AZURE_FUNCTIONS_URL not set');
if (!apiKey) throw new Error('AZURE_FUNCTIONS_API_KEY not set');

test('e2e', async () => {
const response = await fetch(new URL(`/api/SimpleHttpTrigger?code=${apiKey}`, url).toString());
const response = await supertest(url).get(`/api/SimpleHttpTrigger?code=${apiKey}`);
expect(response.status).toBe(200);
});

0 comments on commit 75717e5

Please sign in to comment.