Skip to content

Improve profile and plan stub tests#705

Draft
robsdedude wants to merge 2 commits into6.xfrom
summary-profile-optional-children
Draft

Improve profile and plan stub tests#705
robsdedude wants to merge 2 commits into6.xfrom
summary-profile-optional-children

Conversation

@robsdedude
Copy link
Member

@robsdedude robsdedude commented Mar 6, 2026

  • Allow missing children key, null children value, empty list children value are now considered equivalent.
  • Add tests for bolt 6.0 where the structure of profiles and plans has changed a fair bit. Most prominently will children be omitted by the bolt server when it's empty.
  • Adding tests under new feature flag "Feature:API:Summary:Profile:OptionalStats":
    Drivers should be able to differentiate missing profile stats (e.g., dbHits = 0 has a different meaning that absent dbHits)

Closes: DRIVERS-281

Depends on:

  * Allow missing children key, null children value, empty list children value
    are now considered equivalent.
  * Add tests for bolt 6.0 where the structure of profiles and plans has changed
    a fair bit. Most prominently will `children` be omitted by the bolt server
    when it's empty.
@robsdedude robsdedude force-pushed the summary-profile-optional-children branch from 0ac43bd to ce6c367 Compare March 6, 2026 14:06
 * Adding a new feature flag `"Feature:API:Summary:Profile:OptionalStats"`:
   Drivers should be able to differentiate missing profile stats
   (e.g., `dbHits = 0` has a different meaning that absent `dbHits`).
 * For list fields `summary.notifications` and `profile.children`, drivers may
   now either send an empty list, None, or omit the key.
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.

1 participant