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.

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

TEST #3469

Closed
wants to merge 4 commits into from
Closed

TEST #3469

wants to merge 4 commits into from

Conversation

RulaKhaled
Copy link
Contributor

Description

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.

@codecov-staging
Copy link

codecov-staging bot commented Nov 6, 2024

Bundle Report

Bundle size has no change ✅

Copy link

codecov bot commented Nov 6, 2024

Bundle Report

Bundle size has no change ✅

@codecov-staging
Copy link

codecov-staging bot commented Nov 6, 2024

❌ 2 Tests Failed:

Tests completed Failed Passed Skipped
2999 2 2997 0
View the top 2 failed tests by shortest run time
src/pages/RepoPage/hooks/pathMatchersHooks.test.tsx > useMatchCoverageOnboardingPath > when on a coverage onboarding path > returns true when on /new/other-ci path
Stack Traces | 0.00618s run time
AssertionError: expected true not to be true // Object.is equality
 ❯ .../RepoPage/hooks/pathMatchersHooks.test.tsx:88:34
src/pages/RepoPage/hooks/useJSorTSPendoTracking.test.tsx > useJSorTSPendoTracking > pendo is not present in the window > does not call pendo.updateOptions
Stack Traces | 1.02s run time
AssertionError: expected "spy" to be called at least once

Ignored nodes: comments, script, style
<html>
  <head />
  <body>
    <div>
      <a
        href="/:provider/second-owner/second-repo"
      >
        ClickMe
      </a>
    </div>
  </body>
</html>
 ❯ .../RepoPage/hooks/useJSorTSPendoTracking.test.tsx:244:53
 ❯ runWithExpensiveErrorDiagnosticsDisabled node_modules/@.../dom/dist/config.js:47:12
 ❯ checkCallback node_modules/@.../dom/dist/wait-for.js:124:77
 ❯ Timeout.checkRealTimersCallback node_modules/@.../dom/dist/wait-for.js:118:16

To view more test analytics go to the Test Analytics Dashboard
Got feedback? Let us know on Github

Copy link

codecov-public-qa bot commented Nov 6, 2024

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

❌ Failed Test Results:

Completed 2386 tests with 2 failed, 2384 passed and 0 skipped.

View the full list of failed tests

src/pages/RepoPage/hooks/pathMatchersHooks.test.tsx

  • Class name: src/pages/RepoPage/hooks/pathMatchersHooks.test.tsx
    Test name: useMatchCoverageOnboardingPath > when on a coverage onboarding path > returns true when on /new/other-ci path

    AssertionError: expected true not to be true // Object.is equality
    ❯ .../RepoPage/hooks/pathMatchersHooks.test.tsx:88:34

src/pages/RepoPage/hooks/useJSorTSPendoTracking.test.tsx

  • Class name: src/pages/RepoPage/hooks/useJSorTSPendoTracking.test.tsx
    Test name: useJSorTSPendoTracking > pendo is not present in the window > does not call pendo.updateOptions

    AssertionError: expected "spy" to be called at least once

    Ignored nodes: comments, script, style
    <html>
    <head />
    <body>
    <div>
    <a
    href="/:provider/second-owner/second-repo"
    >
    ClickMe
    </a>
    </div>
    </body>
    </html>
    ❯ .../RepoPage/hooks/useJSorTSPendoTracking.test.tsx:244:53
    ❯ runWithExpensiveErrorDiagnosticsDisabled node_modules/@.../dom/dist/config.js:47:12
    ❯ checkCallback node_modules/@.../dom/dist/wait-for.js:124:77
    ❯ Timeout.checkRealTimersCallback node_modules/@.../dom/dist/wait-for.js:118:16

@codecov-qa
Copy link

codecov-qa bot commented Nov 6, 2024

❌ 2 Tests Failed:

Tests completed Failed Passed Skipped
2386 2 2384 0
View the top 2 failed tests by shortest run time
src/pages/RepoPage/hooks/pathMatchersHooks.test.tsx > useMatchCoverageOnboardingPath &gt; when on a coverage onboarding path &gt; returns true when on /new/other-ci path
Stack Traces | 0.00618s run time
AssertionError: expected true not to be true // Object.is equality
 ❯ .../RepoPage/hooks/pathMatchersHooks.test.tsx:88:34
src/pages/RepoPage/hooks/useJSorTSPendoTracking.test.tsx > useJSorTSPendoTracking &gt; pendo is not present in the window &gt; does not call pendo.updateOptions
Stack Traces | 1.02s run time
AssertionError: expected &quot;spy&quot; to be called at least once

Ignored nodes: comments, script, style
&lt;html&gt;
  &lt;head /&gt;
  &lt;body&gt;
    &lt;div&gt;
      &lt;a
        href=&quot;/:provider/second-owner/second-repo&quot;
      &gt;
        ClickMe
      &lt;/a&gt;
    &lt;/div&gt;
  &lt;/body&gt;
&lt;/html&gt;
 ❯ .../RepoPage/hooks/useJSorTSPendoTracking.test.tsx:244:53
 ❯ runWithExpensiveErrorDiagnosticsDisabled node_modules/@.../dom/dist/config.js:47:12
 ❯ checkCallback node_modules/@.../dom/dist/wait-for.js:124:77
 ❯ Timeout.checkRealTimersCallback node_modules/@.../dom/dist/wait-for.js:118:16

To view more test analytics, go to the Test Analytics Dashboard
Got feedback? Let us know on Github

Copy link

codecov bot commented Nov 6, 2024

❌ 2 Tests Failed:

Tests completed Failed Passed Skipped
2999 2 2997 0
View the top 2 failed tests by shortest run time
src/pages/RepoPage/hooks/pathMatchersHooks.test.tsx > useMatchCoverageOnboardingPath &gt; when on a coverage onboarding path &gt; returns true when on /new/other-ci path
Stack Traces | 0.00618s run time
AssertionError: expected true not to be true // Object.is equality
 ❯ .../RepoPage/hooks/pathMatchersHooks.test.tsx:88:34
src/pages/RepoPage/hooks/useJSorTSPendoTracking.test.tsx > useJSorTSPendoTracking &gt; pendo is not present in the window &gt; does not call pendo.updateOptions
Stack Traces | 1.02s run time
AssertionError: expected &quot;spy&quot; to be called at least once

Ignored nodes: comments, script, style
&lt;html&gt;
  &lt;head /&gt;
  &lt;body&gt;
    &lt;div&gt;
      &lt;a
        href=&quot;/:provider/second-owner/second-repo&quot;
      &gt;
        ClickMe
      &lt;/a&gt;
    &lt;/div&gt;
  &lt;/body&gt;
&lt;/html&gt;
 ❯ .../RepoPage/hooks/useJSorTSPendoTracking.test.tsx:244:53
 ❯ runWithExpensiveErrorDiagnosticsDisabled node_modules/@.../dom/dist/config.js:47:12
 ❯ checkCallback node_modules/@.../dom/dist/wait-for.js:124:77
 ❯ Timeout.checkRealTimersCallback node_modules/@.../dom/dist/wait-for.js:118:16

To view more test analytics, go to the Test Analytics Dashboard
Got feedback? Let us know on Github

@codecov-releaser
Copy link
Contributor

✅ Deploy preview for gazebo ready!

Previews expire after 1 month automatically.

Storybook

Commit Created Cloud Enterprise
b32d518 Wed, 06 Nov 2024 09:56:53 GMT Cloud Enterprise

@RulaKhaled RulaKhaled closed this Nov 6, 2024
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