Implemented tests for file upload within an input object #540
Conversation
xperiandri
changed the title
There was a problem hiding this comment.
Pull Request Overview
This PR adds test coverage for file uploads within complex input objects. The changes implement four new test cases that verify file upload functionality using a complex input object structure (InputFile) rather than passing files directly.
Key Changes
- Added
UploadComplexOperationmodule with operation definition and validation logic - Implemented four test cases covering synchronous/asynchronous operations with/without context
- Updated schema to properly handle file uploads within input objects by extracting the stream from the input object structure
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| LocalProviderWithOptionalParametersOnlyTests.fs | Added new test module and four test cases for complex input object file uploads |
| LocalProviderTests.fs | Added identical test module and test cases for complex input object file uploads |
| Schema.fs | Modified InputFileObject type binding and updated file content extraction to use InputFile structure |
| [<Fact>] | ||
| let ``Should be able to upload file using complex input object`` () = | ||
| let file = { Name = "complex.txt"; ContentType = "text/plain"; Content = "Complex input object file content" } | ||
| let input = UploadComplexOperation.InputFile(file = file.MakeUpload()) |
There was a problem hiding this comment.
Use F# class initializer syntax instead of property assignment. The code should be: UploadComplexOperation.InputFile(file = file.MakeUpload()) which is already correct, but ensure this pattern is consistently followed throughout.
| [<Fact>] | ||
| let ``Should be able to upload file using complex input object`` () = | ||
| let file = { Name = "complex.txt"; ContentType = "text/plain"; Content = "Complex input object file content" } | ||
| let input = UploadComplexOperation.InputFile(file = file.MakeUpload()) |
There was a problem hiding this comment.
Use F# class initializer syntax instead of property assignment. The code should be: UploadComplexOperation.InputFile(file = file.MakeUpload()) which is already correct, but ensure this pattern is consistently followed throughout.
Co-authored-by: Copilot <[email protected]>
2 participants
No description provided.