Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: add all cases for update endpoint #182

Conversation

Orkuncakilkaya
Copy link
Contributor

No description provided.

Copy link
Contributor

github-actions bot commented Jan 9, 2024

This PR will create a major release 🚀

2.0.0 (2024-01-09)

⚠ BREAKING CHANGES

  • use node 20

Features

  • add endpoints structure and error handlers (f5b3709)
  • add event and ctx types (6f72caf)
  • add lambda function update (2b2a2c4)
  • add settings secret, update mgmt lambda permissions (#162) (d84e6df)
  • improve error handling in mgmt-lambda (529f9d4)
  • increase timeout for Fingerprint lambda (a5662fa)
  • introduce deployment settings (850ca61)
  • introduce mgmt-token scheme for authorization (#176) (97cec56)
  • mgmt-lambda-update: introduce error codes (d38f5f1)
  • pass AWS clients into handlers (d84dd9e)
  • reworked getting env, updated tests (2f7ceb6)
  • update lambda existence check (d816e8f)
  • update secrets manager to V3, retrieve secret in mgmt-lambda (64e1e39)
  • upgrade Lambda functions runtime to Node 20 (cabf2f5)
  • use AWS SDK v3 Client mock for testing (2eb947c)

Bug Fixes

  • remove aws-sdk v2 usage (9f76031)
  • remove CodePipeline client (f98c8d5)
  • remove CodePipeline part from mgmt code (01a3739)
  • set correct type for public URL events (baba28d)
  • update handleResult tests with comparing hrefs (0c53e8f)
  • update logging (3b6c713)

Copy link
Contributor

github-actions bot commented Jan 9, 2024

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements
95.59% (+0.92% 🔼)
520/544
🟢 Branches
91.28% (+3.36% 🔼)
136/149
🟢 Functions 92.79% 103/111
🟢 Lines
95.72% (+0.97% 🔼)
492/514

Test suite run success

119 tests passing in 18 suites.

Report generated by 🧪jest coverage report action from 34746f5

Show full coverage report
St File % Stmts % Branch % Funcs % Lines Uncovered Line #s
🟢 All files 95.58 91.27 92.79 95.71
🟢  mgmt-lambda 98.82 93.33 100 98.82
🟢   DefaultSettings.ts 100 100 100 100
🟢   app.ts 97.82 95 100 97.82 25
🟢   auth.ts 100 100 100 100
🟢   exceptions.ts 100 66.66 100 100 15
🟢  mgmt-lambda/handlers 98.76 91.66 100 98.75
🟢   errorHandlers.ts 100 71.42 100 100 22,41
🟢   statusHandler.ts 85.71 100 100 85.71 21
🟢   updateHandler.ts 100 100 100 100
🟢  proxy 100 71.42 100 100
🟢   logger.ts 100 71.42 100 100 10,17
🟢  proxy/handlers 87.35 85.71 89.65 87.95
🟢   handleAgentDowloading.ts 96 66.66 100 95.83 26
🟡   handleResult.ts 76.19 100 76.92 77.5 88-103
🟢   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 98.49 88.57 96.29 98.36
🟢   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
🟢   request.ts 100 100 100 100
🟢   routing.ts 100 100 100 100
🟢   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 89.7 100 84.61 90.9
🔴   normalize-secret.ts 0 100 0 0 1-4
🟢   retrieve-secret.ts 100 100 100 100
🟢   secrets-manager-variables.ts 88 100 100 87.5 33,55-60
🟢   validate-secret.ts 100 100 100 100

@Orkuncakilkaya Orkuncakilkaya merged commit b98dec0 into feature/rollout-as-code Jan 9, 2024
7 checks passed
@Orkuncakilkaya Orkuncakilkaya deleted the feature/inter-480-more-test-cases-for-update-endpoint branch January 9, 2024 11:38
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.

3 participants