Skip to content

Conversation

@kailash-b
Copy link
Contributor

@kailash-b kailash-b commented Jun 17, 2025

Changes

  • Fixes an un-intended .ToLower() while generating the query parameter in Clients.GetAllAsync()

References

Testing

  • This change adds unit test coverage

  • This change adds integration test coverage

  • This change has been tested on the latest version of the platform/language or why not

Checklist

@kailash-b kailash-b requested a review from a team as a code owner June 17, 2025 10:58
@codecov
Copy link

codecov bot commented Jun 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.74%. Comparing base (1655daf) to head (c3a3476).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #820      +/-   ##
==========================================
- Coverage   75.78%   75.74%   -0.04%     
==========================================
  Files         445      445              
  Lines        5843     5843              
  Branches      366      366              
==========================================
- Hits         4428     4426       -2     
- Misses       1254     1256       +2     
  Partials      161      161              
Flag Coverage Δ
authIntTests 30.68% <0.00%> (ø)
mgmtIntTests 57.91% <100.00%> (-0.04%) ⬇️
unittests 5.89% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

@kailash-b kailash-b enabled auto-merge (squash) June 18, 2025 09:19
@kailash-b kailash-b merged commit a7b25c3 into master Jun 19, 2025
12 of 13 checks passed
@kailash-b kailash-b deleted the feature/SDK-6086 branch June 19, 2025 17:13
@kailash-b kailash-b mentioned this pull request Jun 23, 2025
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