Increased test coverage for Acquia env settings.

[AlexSkrypnyk]

Summary by CodeRabbit

• Tests
  • Enhanced test coverage with additional validation for configuration path fallback behavior
  • Improved test infrastructure to support flexible configuration initialization

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

---

Walkthrough

Added a new test method to verify config directory fallback behavior when DRUPAL_CONFIG_PATH is absent, and extended the test utility method requireSettingsFile to accept optional pre-configured settings for test initialization.

Changes

Cohort / File(s)	Summary
Test Method Addition tests/phpunit/Drupal/EnvironmentSettingsTest.php	Added testEnvironmentAcquiaConfigVcsDirectoryFallback() test method to verify that config_sync_directory falls back to config_vcs_directory when AH_SITE_ENVIRONMENT is set and DRUPAL_CONFIG_PATH is absent.
Test Utility Enhancement tests/phpunit/Drupal/SettingsTestCase.php	Updated requireSettingsFile() method signature to accept optional array $pre_settings parameter, enabling pre-seeded settings initialization. Added RuntimeException check for missing application root directory.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• Fixed config_sync_directory not having a default value. #1948: Both PRs relate to config_sync_directory fallback behavior—this PR adds test coverage for the fallback to config_vcs_directory when DRUPAL_CONFIG_PATH is absent.
• [#2261] Fixed Acquia settings to allow DRUPAL_CONFIG_PATH override independently of config_vcs_directory. #2263: Both PRs touch Acquia config handling and DRUPAL_CONFIG_PATH vs config_vcs_directory fallback logic and test coverage.

Poem

🐰 A hop through the config, we test every way,
When paths fall back gracefully, all systems obey,
With settings pre-seeded, the tests shine so bright,
Vortex spins onward—your Drupal done right! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically summarizes the main change: adding a new test method to increase test coverage for Acquia environment settings fallback behavior.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feature/increase-settings-coverage

---

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

[AlexSkrypnyk]

Code Coverage Report:
  2026-02-17 07:33:01

 Summary:
  Classes:  0.00% (0/1)
  Methods:  0.00% (0/2)
  Lines:   94.65% (177/187)

[github-actions]

Code Coverage Report:
  2026-02-17 07:33:40

 Summary:
  Classes:  0.00% (0/1)
  Methods:  0.00% (0/2)
  Lines:   94.65% (177/187)

[AlexSkrypnyk]

Code Coverage Report:
  2026-02-17 07:36:50

 Summary:
  Classes:  0.00% (0/1)
  Methods:  0.00% (0/2)
  Lines:   94.65% (177/187)

[AlexSkrypnyk]

Code Coverage Report:
  2026-02-17 07:36:55

 Summary:
  Classes:  0.00% (0/1)
  Methods:  0.00% (0/2)
  Lines:   94.65% (177/187)

[codecov]

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 77.12%. Comparing base (f1d3e92) to head (0bb7bf2).
⚠️ Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
...baseline/tests/phpunit/Drupal/SettingsTestCase.php	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2298      +/-   ##
==========================================
- Coverage   77.69%   77.12%   -0.58%     
==========================================
  Files         117      110       -7     
  Lines        6174     6015     -159     
  Branches       44        0      -44     
==========================================
- Hits         4797     4639     -158     
+ Misses       1377     1376       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.