Skip to content

test: fix site exporter promoter test namespace#1498

Open
superdav42 wants to merge 3 commits into
mainfrom
feature/auto-20260621-213438-gh1495
Open

test: fix site exporter promoter test namespace#1498
superdav42 wants to merge 3 commits into
mainfrom
feature/auto-20260621-213438-gh1495

Conversation

@superdav42

Copy link
Copy Markdown
Collaborator

Summary

  • Fixes the Main_Site_Promoter_Test namespace to mirror the source namespace, WP_Ultimo\Site_Exporter.
  • Aligns the test file package annotations with the source namespace and marks them as tests.
  • Removes the redundant same-namespace Main_Site_Promoter import.

Resolves #1495

Testing

  • php -l tests/WP_Ultimo/Site_Exporter/Main_Site_Promoter_Test.php
  • vendor/bin/phpcs tests/WP_Ultimo/Site_Exporter/Main_Site_Promoter_Test.php
  • vendor/bin/phpunit --filter Main_Site_Promoter_Test

@coderabbitai

coderabbitai Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@superdav42, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 19 minutes. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more credits in the billing tab to continue.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits.

🚦 How do rate limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan refill rate.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, the refill rate gradually slows as usage increases. The highest same-day bursts are limited more strictly.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: fced507c-da71-4542-ab4f-6ce92fe7df43

📥 Commits

Reviewing files that changed from the base of the PR and between 6026996 and 0ef97af.

📒 Files selected for processing (1)
  • tests/WP_Ultimo/Site_Exporter/Main_Site_Promoter_Test.php
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/auto-20260621-213438-gh1495

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@superdav42

Copy link
Copy Markdown
Collaborator Author

MERGE_SUMMARY

Implemented issue #1495 by changing tests/WP_Ultimo/Site_Exporter/Main_Site_Promoter_Test.php to the source-mirroring WP_Ultimo\Site_Exporter namespace, aligning package annotations, and removing the redundant same-namespace import.

Verification:

  • php -l tests/WP_Ultimo/Site_Exporter/Main_Site_Promoter_Test.php
  • vendor/bin/phpcs tests/WP_Ultimo/Site_Exporter/Main_Site_Promoter_Test.php
  • vendor/bin/phpunit --filter Main_Site_Promoter_Test

@github-actions

Copy link
Copy Markdown

🔨 Build Complete - Ready for Testing!

📦 Download Build Artifact (Recommended)

Download the zip build, upload to WordPress and test:

🌐 Test in WordPress Playground (Very Experimental)

Click the link below to instantly test this PR in your browser - no installation needed!
Playground support for multisite is very limitied, hopefully it will get better in the future.

🚀 Launch in Playground

Login credentials: admin / password

@github-actions

Copy link
Copy Markdown

🔨 Build Complete - Ready for Testing!

📦 Download Build Artifact (Recommended)

Download the zip build, upload to WordPress and test:

🌐 Test in WordPress Playground (Very Experimental)

Click the link below to instantly test this PR in your browser - no installation needed!
Playground support for multisite is very limitied, hopefully it will get better in the future.

🚀 Launch in Playground

Login credentials: admin / password

@superdav42

Copy link
Copy Markdown
Collaborator Author

BLOCKED: merge cannot complete yet.

Evidence:

  • PR: test: fix site exporter promoter test namespace #1498 (feature/auto-20260621-213438-gh1495) remains open.
  • Merge helper passed review-bot gate but branch protection blocked merge while checks/review requirements were active.
  • After merging latest origin/main into the branch and pushing, refreshed CI still has failing required matrix jobs: PHP 8.2, PHP 8.4, PHP 8.5 in Unit & Integration Tests run 27928399809.
  • Retrieved PHP 8.4 job log shows broad unrelated full-suite failures ending with Tests: 10009, Assertions: 25882, Errors: 466, Failures: 102, Skipped: 56, Risky: 1, including tests/unit/Checkout_Editor_Assets_Test.php:22, not the edited site-exporter namespace test.

Local verification for the implemented change passed:

  • php -l tests/WP_Ultimo/Site_Exporter/Main_Site_Promoter_Test.php
  • vendor/bin/phpcs tests/WP_Ultimo/Site_Exporter/Main_Site_Promoter_Test.php
  • vendor/bin/phpunit --filter Main_Site_Promoter_Test

@superdav42 superdav42 added the status:in-review PR open, awaiting review/merge label Jun 22, 2026
@github-actions

Copy link
Copy Markdown

🔨 Build Complete - Ready for Testing!

📦 Download Build Artifact (Recommended)

Download the zip build, upload to WordPress and test:

🌐 Test in WordPress Playground (Very Experimental)

Click the link below to instantly test this PR in your browser - no installation needed!
Playground support for multisite is very limitied, hopefully it will get better in the future.

🚀 Launch in Playground

Login credentials: admin / password

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status:in-review PR open, awaiting review/merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

quality-debt: tests/WP_Ultimo/Site_Exporter/Main_Site_Promoter_Test.php — PR #1470 review feedback (high)

1 participant