Skip to content

Conversation

@TheUnderScorer
Copy link
Contributor

@TheUnderScorer TheUnderScorer commented Apr 10, 2025

Changes highlight:

  • Migrate infrastructure provision logic from Pulumi to Terraform
  • Use GitHub Actions for running E2E tests
  • Remove cloudfrontWithoutVariables test cases, since they are no longer needed.
  • E2E tests now re-use the same CloudFront distribution, greatly reducing the deployment time.
  • E2E tests run as part of terraform apply so that we can leverage the Terraform State locking to avoid conflicts with multiple pull requests.
  • Release E2E tests run using separate Terraform State, to ensure no conflicts with pull requests.
  • Use Vite instead of Webpack for building website used in E2E tests

@TheUnderScorer TheUnderScorer self-assigned this Apr 10, 2025
@TheUnderScorer TheUnderScorer force-pushed the tests/INTER-1193-e2e-tests-infra-terraform branch 2 times, most recently from 98d7022 to 9d262bd Compare April 28, 2025 12:13
@github-actions
Copy link
Contributor

github-actions bot commented Apr 28, 2025

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements 92.58% 649/701
🟢 Branches 86.78% 151/174
🟢 Functions 91.79% 123/134
🟢 Lines 92.76% 615/663

Test suite run success

170 tests passing in 18 suites.

Report generated by 🧪jest coverage report action from f145ccc

Show full coverage report
St File % Stmts % Branch % Funcs % Lines Uncovered Line #s
🟢 All files 92.58 86.78 91.79 92.76
🟢  mgmt-lambda 98.93 93.33 100 98.93
🟢   DefaultSettings.ts 100 100 100 100
🟢   app.ts 97.91 95 100 97.91 26
🟢   auth.ts 100 100 100 100
🟢   exceptions.ts 100 66.66 100 100 20
🟢   routing.ts 100 100 100 100
🟢  mgmt-lambda/handlers 87.22 72.5 93.33 87.15
🟢   errorHandlers.ts 100 71.42 100 100 22,41
🟡   statusHandler.ts 80 50 100 80 76-80,84-89
🟢   updateHandler.ts 87.21 75.86 87.5 87.12 50-51,67-68,139-142,214,219-228,290-291,319
🟢  mgmt-lambda/utils 81.25 88.88 66.66 100
🟢   cloudfrontUtils.ts 100 88.88 100 100 6
🔴   delay.ts 40 100 0 100
🟢  proxy/handlers 87.91 86.95 90.32 88.63
🟢   handleAgentDowloading.ts 96.29 66.66 100 96.15 31
🟡   handleResult.ts 76.74 100 76.92 78.04 90-105
🟢   handleStatus.ts 100 100 100 100
🟡  proxy/test 77.77 100 50 71.42
🟡   aws.ts 77.77 100 50 71.42 4-5
🟢  proxy/test/utils/customer-variables 100 100 100 100
🟢   in-memory-customer-variables.ts 100 100 100 100
🟢  proxy/utils 94.57 85.71 94.28 94.11
🟢   buffer.ts 100 100 100 100
🟢   cache-control.ts 100 100 100 100
🟢   cookie.ts 100 100 100 100
🟢   headers.ts 100 100 100 100
🔴   is-blob.ts 0 0 0 0 6-7
🟢   log.ts 85.71 33.33 100 83.33 11
🟢   request.ts 100 100 100 100
🟢   routing.ts 100 100 100 100
🔴   string.ts 14.28 100 0 14.28 2-8
🟢   traffic.ts 100 100 100 100
🟢  proxy/utils/customer-variables 100 100 100 100
🟢   customer-variables.ts 100 100 100 100
🟢   defaults.ts 100 100 100 100
🟢   header-customer-variables.ts 100 100 100 100
🟢   maybe-obfuscate-variable.ts 100 100 100 100
🟢   selectors.ts 100 100 100 100
🟢   types.ts 100 100 100 100
🟢  proxy/utils/customer-variables/secrets-manager 95.58 100 100 95.52
🟢   normalize-secret.ts 100 100 100 100
🟢   retrieve-secret.ts 100 100 100 100
🟢   secrets-manager-variables.ts 86.95 100 100 86.95 28,50-55
🟢   validate-secret.ts 100 100 100 100

@TheUnderScorer TheUnderScorer force-pushed the tests/INTER-1193-e2e-tests-infra-terraform branch 2 times, most recently from ccbf170 to 3a4bec1 Compare April 30, 2025 14:24
@github-actions

This comment has been minimized.

@TheUnderScorer TheUnderScorer force-pushed the tests/INTER-1193-e2e-tests-infra-terraform branch 4 times, most recently from 6351ce5 to 11f5948 Compare May 8, 2025 11:24
@github-actions

This comment has been minimized.

@TheUnderScorer TheUnderScorer force-pushed the tests/INTER-1193-e2e-tests-infra-terraform branch 3 times, most recently from 28bceb2 to be5a81d Compare May 9, 2025 14:37
@TheUnderScorer TheUnderScorer marked this pull request as ready for review May 13, 2025 10:18
necipallef
necipallef previously approved these changes May 15, 2025
@TheUnderScorer TheUnderScorer force-pushed the tests/INTER-1193-e2e-tests-infra-terraform branch from 749175d to 802c4d5 Compare May 19, 2025 11:51
@github-actions
Copy link
Contributor

github-actions bot commented Jun 4, 2025

This PR will create a minor release 🚀

2.1.0 (2025-06-04)

Features

  • add nonce-based csp to style (e2b5f6c)

Build System

  • deps: update dependencies (9242e69)

@TheUnderScorer TheUnderScorer merged commit bfe05b8 into rc Jun 4, 2025
8 checks passed
@TheUnderScorer TheUnderScorer deleted the tests/INTER-1193-e2e-tests-infra-terraform branch June 4, 2025 13:07
erayaydin pushed a commit that referenced this pull request Jun 10, 2025
…-infra-terraform

Migrate tests to Terraform
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants