Update timestamp output format system test to use direct request#651
Update timestamp output format system test to use direct request#651danieljbruce wants to merge 1 commit intowrite-api-changes-2from
Conversation
Modify the system test in `timestamp_output_format.ts` to use `table.request` instead of `table.getRows`. This ensures that we are checking the raw data returned by the server, bypassing any unreleased changes in the BigQuery library. The test now verifies high-precision timestamps by inspecting the raw JSON response from the `/data` endpoint. Co-authored-by: danieljbruce <8935272+danieljbruce@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
The system test for timestamp output format was modified to use a direct API request via the
ServiceObject.requestmethod inherited by theTableclass. This change avoids dependency on unreleased library features in@google-cloud/bigqueryand allows for direct verification of high-precision timestamp strings as returned by the BigQuery API.Key changes:
table.getRows(options)with(table as any).request({ uri: '/data', qs: options }, callback).resp.rows[0].f[0].v).Promiseto maintain compatibility with the existingasync/awaittest structure.PR created automatically by Jules for task 3432412269925724150 started by @danieljbruce