Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix: Use new orgs fetched from GH resync #3081

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

fix: Use new orgs fetched from GH resync #3081

wants to merge 1 commit into from

Conversation

calvin-codecov
Copy link
Contributor

@calvin-codecov calvin-codecov commented Jul 29, 2024 •
edited
Loading

Description

Closes codecov/engineering-team#2171

Code Example

Notable Changes

Screenshots

Link to Sample Entry

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. In 2022 this entity acquired Codecov and as result Sentry is going to need some rights from me in order to utilize my contributions in this PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

@calvin-codecov calvin-codecov marked this pull request as draft July 29, 2024 22:08
@codecov-staging
Copy link

codecov-staging bot commented Jul 29, 2024 •
edited
Loading

Test Failures Detected: Due to failing tests, we cannot provide coverage reports at this time.

❌ Failed Test Results:

Completed 3016 tests with 1 failed, 3015 passed and 0 skipped.

View the full list of failed tests

useResyncUser

  • Class name: useResyncUser when the hook is called and the syncing is in progress calls invalidate queries on each sync
    Test name: useResyncUser when the hook is called and the syncing is in progress calls invalidate queries on each sync

    Error: expect(jest.fn()).toHaveBeenCalledTimes(expected)

    Expected number of calls: 2
    Received number of calls: 3

    Ignored nodes: comments, script, style
    <html>
      <head />
      <body>
        <div />
      </body>
    </html>
        at callback (.../services/user/useResyncUser.spec.js:123:56)
        at runWithExpensiveErrorDiagnosticsDisabled (.../gazebo/gazebo/node_modules/@.../dom/dist/config.js:47:12)
        at checkCallback (.../gazebo/gazebo/node_modules/@.../dom/dist/wait-for.js:124:76)
        at .../gazebo/gazebo/node_modules/@.../dom/dist/wait-for.js:82:9

@codecov-qa
Copy link

codecov-qa bot commented Jul 29, 2024 •
edited
Loading

Test Failures Detected: Due to failing tests, we cannot provide coverage reports at this time.

❌ Failed Test Results:

Completed 2423 tests with 1 failed, 2422 passed and 0 skipped.

View the full list of failed tests

useResyncUser

  • Class name: useResyncUser when the hook is called and the syncing is in progress calls invalidate queries on each sync
    Test name: useResyncUser when the hook is called and the syncing is in progress calls invalidate queries on each sync

    Error: expect(jest.fn()).toHaveBeenCalledTimes(expected)

    Expected number of calls: 2
    Received number of calls: 3

    Ignored nodes: comments, script, style
    <html>
      <head />
      <body>
        <div />
      </body>
    </html>
        at callback (.../services/user/useResyncUser.spec.js:123:56)
        at runWithExpensiveErrorDiagnosticsDisabled (.../gazebo/gazebo/node_modules/@.../dom/dist/config.js:47:12)
        at checkCallback (.../gazebo/gazebo/node_modules/@.../dom/dist/wait-for.js:124:76)
        at .../gazebo/gazebo/node_modules/@.../dom/dist/wait-for.js:82:9

@codecov-public-qa Codecov Public QA
Copy link

Test Failures Detected: Due to failing tests, we cannot provide coverage reports at this time.

❌ Failed Test Results:

Completed 1245 tests with 1 failed, 1244 passed and 0 skipped.

View the full list of failed tests

useResyncUser

  • Class name: useResyncUser when the hook is called and the syncing is in progress calls invalidate queries on each sync
    Test name: useResyncUser when the hook is called and the syncing is in progress calls invalidate queries on each sync

    Error: expect(jest.fn()).toHaveBeenCalledTimes(expected)

    Expected number of calls: 2
    Received number of calls: 3

    Ignored nodes: comments, script, style
    <html>
      <head />
      <body>
        <div />
      </body>
    </html>
        at callback (.../services/user/useResyncUser.spec.js:123:56)
        at runWithExpensiveErrorDiagnosticsDisabled (.../gazebo/gazebo/node_modules/@.../dom/dist/config.js:47:12)
        at checkCallback (.../gazebo/gazebo/node_modules/@.../dom/dist/wait-for.js:124:76)
        at .../gazebo/gazebo/node_modules/@.../dom/dist/wait-for.js:82:9

@codecov-staging
Copy link

Bundle Report

Changes will increase total bundle size by 177 bytes ⬆️

Bundle name Size Change
gazebo-staging-array-push 5.84MB 177 bytes ⬆️

@codecov Codecov
Copy link

codecov bot commented Jul 29, 2024

Bundle Report

Changes will increase total bundle size by 177 bytes ⬆️

Bundle name Size Change
gazebo-production-array-push 5.84MB 177 bytes ⬆️

@codecov-releaser
Copy link
Contributor

✅ Deploy preview for gazebo ready!

Previews expire after 1 month automatically.

Storybook

Commit Created Cloud Enterprise
7ab3239 Mon, 29 Jul 2024 22:15:25 GMT Cloud Enterprise

@codecov Codecov
Copy link

codecov bot commented Jul 30, 2024

Test Failures Detected: Due to failing tests, we cannot provide coverage reports at this time.

❌ Failed Test Results:

Completed 3643 tests with 1 failed, 3642 passed and 0 skipped.

View the full list of failed tests

useResyncUser

  • Class name: useResyncUser when the hook is called and the syncing is in progress calls invalidate queries on each sync
    Test name: useResyncUser when the hook is called and the syncing is in progress calls invalidate queries on each sync

    Error: expect(jest.fn()).toHaveBeenCalledTimes(expected)

    Expected number of calls: 2
    Received number of calls: 3

    Ignored nodes: comments, script, style
    <html>
      <head />
      <body>
        <div />
      </body>
    </html>
        at callback (.../services/user/useResyncUser.spec.js:123:56)
        at runWithExpensiveErrorDiagnosticsDisabled (.../gazebo/gazebo/node_modules/@.../dom/dist/config.js:47:12)
        at checkCallback (.../gazebo/gazebo/node_modules/@.../dom/dist/wait-for.js:124:76)
        at .../gazebo/gazebo/node_modules/@.../dom/dist/wait-for.js:82:9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

New Org Onboarding Resync Isn't Updating
2 participants
@calvin-codecov @codecov-releaser