(revert!) Add logs to debug test in CI

vercel · Oct 11, 2024 · d767b48 · d767b48
1 parent 968d330
commit d767b48
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 1 deletion.
diff --git a/run-tests.js b/run-tests.js
@@ -461,7 +461,7 @@ ${ENDGROUP}`)
         '--runInBand',
         '--forceExit',
         '--verbose',
-        '--silent',
+        // '--silent',
         ...(isTestJob
           ? ['--json', `--outputFile=${test.file}${RESULTS_EXT}`]
           : []),

diff --git a/test/e2e/cancel-request/pages/api/edge-api.ts b/test/e2e/cancel-request/pages/api/edge-api.ts
@@ -8,6 +8,7 @@ export const config = {
 let streamable: ReturnType<typeof Streamable> | undefined
 
 export default async function handler(req: NextRequest): Promise<Response> {
+  console.log('handler', req.url)
   if (req.nextUrl.searchParams.has('compile')) {
     // The request just wants to trigger compilation.
     return new Response(null, { status: 204 })
@@ -22,13 +23,15 @@ export default async function handler(req: NextRequest): Promise<Response> {
   if (write) {
     const s = (streamable = Streamable(+write))
 
+    console.log('aborted:', req.signal.aborted)
     // The request was aborted before the response was returned.
     if (req.signal.aborted) {
       s.abort()
       return new Response(null, { status: 204 })
     }
 
     req.signal.onabort = () => {
+      console.log('onabort')
       s.abort()
     }
 

diff --git a/test/e2e/cancel-request/stream-cancel.test.ts b/test/e2e/cancel-request/stream-cancel.test.ts
@@ -10,18 +10,22 @@ describe('streaming responses cancel inner stream after disconnect', () => {
   function prime(url: string, noData?: boolean) {
     return new Promise<void>((resolve, reject) => {
       url = new URL(url, next.url).href
+      console.log('prime', url)
 
       // There's a bug in node-fetch v2 where aborting the fetch will never abort
       // the connection, because the body is a transformed stream that doesn't
       // close the connection stream.
       // https://github.com/node-fetch/node-fetch/pull/670
       const req = get(url, async (res) => {
+        console.log('response')
         while (true) {
           const value = res.read(1)
+          console.log('received value:', value)
           if (value) break
           await sleep(5)
         }
 
+        console.log('res.destroy()')
         res.destroy()
         resolve()
       })

diff --git a/test/e2e/cancel-request/streamable.ts b/test/e2e/cancel-request/streamable.ts
@@ -11,6 +11,7 @@ export function Streamable(write: number) {
     finished: Promise.all([cleanedUp.promise, aborted.promise]).then(() => i),
 
     abort() {
+      console.log('abort, startedConsuming:', startedConsuming)
       aborted.resolve()
 
       if (!startedConsuming) {
@@ -19,6 +20,7 @@ export function Streamable(write: number) {
     },
     stream: new ReadableStream({
       async pull(controller) {
+        console.log('pull')
         startedConsuming = true
 
         if (i >= write) {
@@ -29,6 +31,7 @@ export function Streamable(write: number) {
         controller.enqueue(encoder.encode(String(i++)))
       },
       cancel() {
+        console.log('cancel, startedConsuming:', startedConsuming)
         cleanedUp.resolve()
       },
     }),