apitest icon indicating copy to clipboard operation
apitest copied to clipboard

Multipart should use MapFS and MapFile from testing package in standard lib instead of OS

Open tjad opened this issue 2 years ago • 1 comments

Is your feature request related to a problem? Please describe. This is an improvement request Multipart should use MapFS and MapFile from testing package instead of OS. This would be more inline with testing as the file would be in memory and not be required to exist on disk

Describe the solution you'd like Use https://pkg.go.dev/testing/fstest#MapFS And https://pkg.go.dev/testing/fstest#MapFile

Describe alternatives you've considered None - using the current implementation of Multipart in #118 #119 Is almost good enough, but this improvement would make it more complete - we don't want to use the file system during testing. The test data is binary, and storing it in git is not a particularly good way to go. Storing the bytes as integers in a file and writing to disk to be read is inefficient, messy and unnecessary, it is also not a good way to go as it would require cleanup etc etc etc.

tjad avatar Jan 09 '24 07:01 tjad

Would be happy to take a PR for this, sounds like a good idea.

steinfletcher avatar May 08 '24 12:05 steinfletcher