Skip to content

Fix UserType response fields#552

Open
jychp wants to merge 1 commit into
okta:masterfrom
jychp:codex-fix-user-type-fields
Open

Fix UserType response fields#552
jychp wants to merge 1 commit into
okta:masterfrom
jychp:codex-fix-user-type-fields

Conversation

@jychp

@jychp jychp commented Jun 7, 2026

Copy link
Copy Markdown

Summary

  • add the missing UserType response fields already present in the local OpenAPI schema
  • preserve UserType(id=...) serialization for existing embedded user-type requests
  • add focused coverage for UserType response deserialization

Source check

  • No pull request template is present in this repository under .github
  • UserType fields were taken from openapi/api.yaml and cross-checked against openapi/management.yaml
  • The public Okta User Types API reference lists the same response fields

Validation

  • .venv/bin/python -m pytest tests/test_user_type.py tests/integration/test_user_types_it.py
  • git diff --check
  • .venv/bin/python -m pytest: 599 passed, 44 skipped, 1 unrelated existing failure in test_header_mutation.py because the top-level async test is not marked for an async pytest plugin

Fixes #535

@jychp jychp marked this pull request as ready for review June 7, 2026 15:44
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.

UserType response model discards all fields except id

1 participant