test(cache): update tests to use legacy hash for speed #656

2025-12-08T20:06:36Z

argoyle commented

2025-12-08 20:06:36 +00:00

(Migrated from gitlab.com)

Update test setup to leverage legacy hash for API keys in order
to avoid the performance impact of bcrypt. Replace the API key
based test with a user subscription-based test to enhance
concurrency and reliability. Replace OrganizationByAPIKey
with OrganizationsByUser to optimize the retrieval process.

Update test setup to leverage legacy hash for API keys in order to avoid the performance impact of bcrypt. Replace the API key based test with a user subscription-based test to enhance concurrency and reliability. Replace `OrganizationByAPIKey` with `OrganizationsByUser` to optimize the retrieval process.

argoyle commented

2025-12-08 20:21:17 +00:00

(Migrated from gitlab.com)

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 38.31%. Comparing base (1914211) to head (129cd8a).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #652   +/-   ##
=======================================
  Coverage   38.31%   38.31%           
=======================================
  Files          45       45           
  Lines        2691     2691           
=======================================
  Hits         1031     1031           
  Misses       1620     1620           
  Partials       40       40

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

## [Codecov](https://app.codecov.io/gl/unboundsoftware/schemas/pull/652?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=gitlab&utm_content=comment&utm_campaign=pr+comments&utm_term=unboundsoftware) Report :white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 38.31%. Comparing base ([`1914211`](https://app.codecov.io/gl/unboundsoftware/schemas/commit/1914211b8516e0ca8ec955c69f15f61f65b27cb6?dropdown=coverage&el=desc&utm_medium=referral&utm_source=gitlab&utm_content=comment&utm_campaign=pr+comments&utm_term=unboundsoftware)) to head ([`129cd8a`](https://app.codecov.io/gl/unboundsoftware/schemas/commit/129cd8aad1b54af4f359a31ef8f3c74a6abaea8d?dropdown=coverage&el=desc&utm_medium=referral&utm_source=gitlab&utm_content=comment&utm_campaign=pr+comments&utm_term=unboundsoftware)). <details><summary>Additional details and impacted files</summary> ```diff @@ Coverage Diff @@ ## main #652 +/- ## ======================================= Coverage 38.31% 38.31% ======================================= Files 45 45 Lines 2691 2691 ======================================= Hits 1031 1031 Misses 1620 1620 Partials 40 40 ``` </details> [:umbrella: View full report in Codecov by Sentry](https://app.codecov.io/gl/unboundsoftware/schemas/pull/652?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=gitlab&utm_content=comment&utm_campaign=pr+comments&utm_term=unboundsoftware). :loudspeaker: Have feedback on the report? [Share it here](https://gitlab.com/codecov-open-source/codecov-user-feedback/-/issues/4). <details><summary> :rocket: New features to boost your workflow: </summary> - :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems. </details>

argoyle (Migrated from gitlab.com) merged commit 6ae6a4d6cf into main

2025-12-08 20:31:03 +00:00

argoyle commented

2025-12-08 20:31:04 +00:00

(Migrated from gitlab.com)

mentioned in commit 6ae6a4d6cf

mentioned in commit 6ae6a4d6cf6d5941c1b41a90f1fdc464f2afce93

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: unboundsoftware/schemas#656