Skip to content

Conversation

@nulmete
Copy link
Member

@nulmete nulmete commented Feb 3, 2026

Related issue: Resolves #38554

Checklist for submitter

  • Changes file added for user-visible changes in changes/, orbit/changes/ or ee/fleetd-chrome/changes.
    See Changes files for more information.

Testing

  • Added/updated automated tests

There's already an integration test for MaybeAssociateHostWithScimUser which is the function call I added as a fix. See

func testMaybeAssociateHostWithScimUser(t *testing.T, ds *Datastore) {

  • QA'd all new/changed functionality manually

Enrolled physical Android device and verified that Full name (IdP) and Groups (IdP) are populated.

Screenshot 2026-02-03 at 3 31 16 PM

@codecov
Copy link

codecov bot commented Feb 3, 2026

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 66.18%. Comparing base (b25c952) to head (ae09ef3).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
server/mdm/android/service/pubsub.go 0.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #39254      +/-   ##
==========================================
- Coverage   66.18%   66.18%   -0.01%     
==========================================
  Files        2424     2424              
  Lines      194205   194207       +2     
  Branches     8559     8559              
==========================================
  Hits       128543   128543              
- Misses      53974    53975       +1     
- Partials    11688    11689       +1     
Flag Coverage Δ
backend 67.95% <0.00%> (-0.01%) ⬇️

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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

IdP host vitals not automatically populated for Android hosts

4 participants