suyanhanx commented on code in PR #3619:
URL: 
https://github.com/apache/incubator-opendal/pull/3619#discussion_r1404780447


##########
bindings/nodejs/tests/suites/async.suite.mjs:
##########
@@ -19,15 +19,31 @@
 
 import { randomUUID } from 'node:crypto'
 import { test } from 'vitest'
+import { generateBytes } from '../utils.mjs'
 
-export function run(operator) {
+export function run(op) {
   test('async stat not exist files', async () => {
     const filename = `random_file_${randomUUID()}`
 
     try {
-      await operator.stat(filename)
+      await op.stat(filename)
     } catch (error) {
       assert.include(error.message, 'NotFound')
     }
   })
+
+  test('async writer', async () => {

Review Comment:
   ~~All other Storage services are fine. s3 keeps reporting that the size is 
too small on close. 
   I have changed the size to 5MB and it still reports the same error.~~
   
   The chunk size was set too small. Fixed.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to