Skip to content

Enable contextual debug logging for integration tests #9012

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MarkDuckworth
Copy link
Contributor

Contextual debug logging for integration tests will log the debug log for the 100 debug or warning messages leading up to a logged error message. This is intended to give context to flakes seen in the CI, without logging an excessive amount which would make the CI logs too large to easily review.

Copy link

changeset-bot bot commented May 9, 2025

⚠️ No Changeset found

Latest commit: badc177

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
Contributor

github-actions bot commented May 9, 2025

Vertex AI Mock Responses Check ⚠️

A newer major version of the mock responses for Vertex AI unit tests is available. update_vertexai_responses.sh should be updated to clone the latest version of the responses: v13.0

@google-oss-bot
Copy link
Contributor

Size Report 1

Affected Products

  • @firebase/firestore

    TypeBase (88584fd)Merge (f47a0d1)Diff
    browser385 kB386 kB+1.27 kB (+0.3%)
    main595 kB597 kB+1.90 kB (+0.3%)
    module385 kB386 kB+1.27 kB (+0.3%)
    react-native385 kB386 kB+1.28 kB (+0.3%)
  • @firebase/firestore-lite

    TypeBase (88584fd)Merge (f47a0d1)Diff
    browser114 kB115 kB+1.26 kB (+1.1%)
    main157 kB159 kB+1.90 kB (+1.2%)
    module114 kB115 kB+1.26 kB (+1.1%)
    react-native114 kB115 kB+1.26 kB (+1.1%)
  • bundle

    15 size changes

    TypeBase (88584fd)Merge (f47a0d1)Diff
    firestore (CSI Auto Indexing Disable and Delete)275 kB275 kB+113 B (+0.0%)
    firestore (CSI Auto Indexing Enable)275 kB275 kB+113 B (+0.0%)
    firestore (Persistence)306 kB306 kB+113 B (+0.0%)
    firestore (Query Cursors)252 kB252 kB+102 B (+0.0%)
    firestore (Query)249 kB250 kB+102 B (+0.0%)
    firestore (Read data once)237 kB237 kB+102 B (+0.0%)
    firestore (Read Write w Persistence)331 kB331 kB+113 B (+0.0%)
    firestore (Realtime updates)239 kB240 kB+102 B (+0.0%)
    firestore (Transaction)216 kB217 kB+102 B (+0.0%)
    firestore (Write data)216 kB216 kB+102 B (+0.0%)
    firestore-lite (Query Cursors)105 kB105 kB+102 B (+0.1%)
    firestore-lite (Query)101 kB101 kB+102 B (+0.1%)
    firestore-lite (Read data once)76.3 kB76.4 kB+102 B (+0.1%)
    firestore-lite (Transaction)102 kB102 kB+108 B (+0.1%)
    firestore-lite (Write data)85.9 kB86.0 kB+102 B (+0.1%)

  • firebase

    TypeBase (88584fd)Merge (f47a0d1)Diff
    firebase-compat.js797 kB798 kB+1.03 kB (+0.1%)
    firebase-firestore-compat.js342 kB343 kB+1.03 kB (+0.3%)
    firebase-firestore-lite.js133 kB134 kB+1.24 kB (+0.9%)
    firebase-firestore.js443 kB445 kB+1.23 kB (+0.3%)

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/XaA5bcDgaX.html

@google-oss-bot
Copy link
Contributor

Size Analysis Report 1

This report is too large (296,025 characters) to be displayed here in a GitHub comment. Please use the below link to see the full report on Google Cloud Storage.

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/yG8mdJqVDS.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants