Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Validate python intergration tests. #5617

Merged
merged 9 commits into from
Oct 18, 2024
Merged

Conversation

andrueastman
Copy link
Member

Validates intergration tests after typing infor updates in microsoft/kiota-python#384

baywet
baywet previously approved these changes Oct 16, 2024
@andrueastman andrueastman force-pushed the andrueastman/intergrationtests branch from 7ab3ea8 to 2dd9628 Compare October 16, 2024 15:56
baywet
baywet previously approved these changes Oct 16, 2024
@andrueastman
Copy link
Member Author

andrueastman commented Oct 16, 2024

There's still a couple of things to resolved with the integration tests.

  • Fixing type information with composed type serialization in generated code(deserializer functions need to handle None cases that the parsenode can return. as well as issues with referencing properties in wrapper types that were not showing up before.)
  • Handling of openApi descriptions with paths that return Enum (currently the code calls send_async which expects a Parsable and a factory which is being flagged as an Enum is not a parsbable)

Copy link

@andrueastman andrueastman marked this pull request as ready for review October 18, 2024 12:23
@andrueastman andrueastman requested a review from a team as a code owner October 18, 2024 12:23
@andrueastman andrueastman requested a review from baywet October 18, 2024 12:25
@andrueastman andrueastman self-assigned this Oct 18, 2024
@andrueastman andrueastman merged commit f1e60b7 into main Oct 18, 2024
209 checks passed
@andrueastman andrueastman deleted the andrueastman/intergrationtests branch October 18, 2024 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants