Skip to content

Fix PHP coverage not working correctly in IDE due to output file not being accessible#12

Merged
Molkobain merged 1 commit intomasterfrom
feature/add-volume-for-php-coverage
Dec 22, 2025
Merged

Fix PHP coverage not working correctly in IDE due to output file not being accessible#12
Molkobain merged 1 commit intomasterfrom
feature/add-volume-for-php-coverage

Conversation

@Molkobain
Copy link
Copy Markdown
Contributor

@Molkobain Molkobain commented Dec 18, 2025

Base information

Question Answer
Related to a SourceForge thread / Another PR / Combodo ticket? No
Type of change? Bug fix

Symptom (bug)

When executing PHP coverage in PHPStorm, it can't access the result file as it is output in the PHP container.

Reproduction procedure (bug)

  1. On iTop 3.2.2
  2. With PHPStorm as the IDE
  3. Launch unit tests with coverage enabled
  4. See that the process ends well but it is not displayed

Cause (bug)

With PHPStorm, the XML file is output in /opt/phpstorm-coverage in the PHP container.
PHPStorm running locally can't access the result file.

Proposed solution (bug and enhancement)

Add a new volume to the Docker compose recipe to mount the PHP coverage output folder locally.
Thanks to @Lenaick for pointing this out to me!

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have tested all changes I made on an iTop instance
  • Would a unit test be relevant and have I added it?
  • Is the PR clear and detailed enough so anyone can understand without digging in the code?

Checklist of things to do before PR is ready to merge

  • Discuss with the team if the env. var. is well named and placed

@Molkobain Molkobain self-assigned this Dec 18, 2025
@Molkobain Molkobain added the bug Something isn't working label Dec 18, 2025
@Molkobain Molkobain changed the title 🐛 Fix PHP coverage not working correctly in IDE due to output file not being accessible Fix PHP coverage not working correctly in IDE due to output file not being accessible Dec 18, 2025
@Molkobain Molkobain merged commit af53bce into master Dec 22, 2025
@Molkobain Molkobain deleted the feature/add-volume-for-php-coverage branch December 22, 2025 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants