Skip to content

[DNM] Unit tests in single job failure #22263

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
Closed
wants to merge 3 commits into from
Closed

[DNM] Unit tests in single job failure #22263

wants to merge 3 commits into from

Conversation

wzieba
Copy link
Contributor

@wzieba wzieba commented Oct 7, 2025 •
edited
Loading

Testing #22259 & #22267

@wzieba wzieba added the Do Not Merge In PRs with this label, our automation will fail a require check, preventing accidental merging label Oct 7, 2025
@dangermattic
Copy link
Collaborator

1 Message
📖 This PR is still a Draft: some checks will be skipped.

Generated by 🚫 Danger

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Oct 7, 2025 •
edited
Loading

Jetpack📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack Jetpack
FlavorJalapeno
Build TypeDebug
Versionpr22263-9d0f35a
Commit9d0f35a
Direct Downloadjetpack-prototype-build-pr22263-9d0f35a.apk
 Note: Google Login is not supported on these builds.

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Oct 7, 2025 •
edited
Loading

WordPress📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress WordPress
FlavorJalapeno
Build TypeDebug
Versionpr22263-9d0f35a
Commit9d0f35a
Direct Downloadwordpress-prototype-build-pr22263-9d0f35a.apk
 Note: Google Login is not supported on these builds.

@wzieba wzieba mentioned this pull request Oct 7, 2025
Merged
@ParaskP7 ParaskP7 requested a review from Copilot October 8, 2025 09:40
Copilot
Copilot AI reviewed Oct 8, 2025
View reviewed changes
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR appears to be testing unit test failure behavior by intentionally removing a mock setup call in a test. The change removes the whenever mock configuration that was previously setting up the expected behavior for statsSqlUtils.select().

  • Removes mock setup for statsSqlUtils.select() call to test failure scenarios

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Base automatically changed from tests_in_single_job to trunk October 8, 2025 11:13
@wzieba wzieba force-pushed the tests_in_single_job_failure branch from 2c7298d to b15e19c Compare October 8, 2025 11:46
@wpmobilebot
Copy link
Contributor

wpmobilebot commented Oct 8, 2025 •
edited
Loading

🤖 Test Failure Analysis

Your tests failed. Claude has analyzed the failures - check the annotation for details.

@wzieba wzieba force-pushed the tests_in_single_job_failure branch from b15e19c to 9d0f35a Compare October 8, 2025 12:07
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Oct 8, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@wzieba wzieba mentioned this pull request Oct 8, 2025
Merged
@wzieba wzieba closed this Oct 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

Assignees

No one assigned

Labels

Do Not Merge In PRs with this label, our automation will fail a require check, preventing accidental merging

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@wzieba @dangermattic @wpmobilebot