Skip to content

wip#3025

Draft
jas-valgotar wants to merge 1 commit intomainfrom
jas/addOpenapiSchemaToFT
Draft

wip#3025
jas-valgotar wants to merge 1 commit intomainfrom
jas/addOpenapiSchemaToFT

Conversation

@jas-valgotar
Copy link
Contributor

@jas-valgotar jas-valgotar commented Feb 13, 2026

This pull request adds new functionality to convert JSON Schema strings to FormulaType objects and introduces comprehensive unit tests to validate this feature. The main focus is on enhancing the usability and robustness of schema-to-type conversion, ensuring support for a wide range of JSON Schema constructs and error scenarios.

New JSON Schema to FormulaType conversion:

  • Added two new overloads to OpenApiExtensions for converting a JSON Schema string to a FormulaType, one with default settings and one allowing custom ConnectorSettings. These methods handle various schema types and error conditions.

Expanded unit test coverage:

  • Introduced extensive unit tests in OpenApiExtensionTests to verify the new conversion methods, including:
    • Primitive and formatted string types (e.g., string, number, date-time)
    • Objects, nested objects, arrays of objects/primitives, and file types
    • Handling of unsupported types, invalid input, and exceptions
    • End-to-end test that parses a complex nested schema, deserializes JSON data, and evaluates a Power Fx expression using the generated type
  • Added System.Text.Json import to support JSON parsing in tests.

@jas-valgotar jas-valgotar force-pushed the jas/addOpenapiSchemaToFT branch from 52710e9 to 8aafb9a Compare February 13, 2026 18:00
@jas-valgotar
Copy link
Contributor Author

✅ No public API change.

@jas-valgotar jas-valgotar force-pushed the jas/addOpenapiSchemaToFT branch from 8aafb9a to 47cdd87 Compare February 13, 2026 21:38
@jas-valgotar
Copy link
Contributor Author

✅ No public API change.

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

Comments