fix(storage): multipart upload is firing on 0 bytes data (#13927)

Author: HuiSF

packages/storage/__tests__/providers/s3/apis/uploadData/index.test.ts

@@ -180,6 +180,22 @@ describe('uploadData with path', () => {
 			},
 		);
 
+		it('should use putObject for 0 bytes data (e.g. create a folder)', () => {
+			const testInput = {
+				path: 'test-path',
+				data: '', // 0 bytes
+			};
+
+			uploadData(testInput);
+
+			expect(mockPutObjectJob).toHaveBeenCalledWith(
+				testInput,
+				expect.any(AbortSignal),
+				expect.any(Number),
+			);
+			expect(mockGetMultipartUploadHandlers).not.toHaveBeenCalled();
+		});
+
 		it('should use uploadTask', async () => {
 			mockPutObjectJob.mockReturnValueOnce('putObjectJob');
 			mockCreateUploadTask.mockReturnValueOnce('uploadTask');

packages/storage/src/providers/s3/apis/uploadData/index.ts

@@ -135,7 +135,7 @@ export function uploadData(input: UploadDataInput | UploadDataWithPathInput) {
 		StorageValidationErrorCode.ObjectIsTooLarge,
 	);
 
-	if (dataByteLength && dataByteLength <= DEFAULT_PART_SIZE) {
+	if (dataByteLength !== undefined && dataByteLength <= DEFAULT_PART_SIZE) {
 		// Single part upload
 		const abortController = new AbortController();