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

HTML report fails to load dependencies (JS, CSS, Fonts) #654

Closed
merauluka opened this issue Nov 19, 2018 · 0 comments
Closed

HTML report fails to load dependencies (JS, CSS, Fonts) #654

merauluka opened this issue Nov 19, 2018 · 0 comments

Comments

@merauluka
Copy link

Q A
php-code-coverage version 5.3.2
PHP version 7.1.18
Driver Xdebug
Xdebug version (if used) 2.5.0
Installation Method Composer
Usage Method PHPUnit
PHPUnit version (if used) 6.5.13

I am using PHP Unit alongside Drupal to test custom classes we are creating. I really like using the HTML report since it shows a detailed breakdown of the lines and methods covered by our tests. However, after a recent update, our virtual machines are no longer loading dependencies from the generated reports. I see this is due to a change that came through that makes the dependency folders for css, fonts, and js hidden folders.

The original post that the issue was marked as having solved requested that the folders begin with an underscore to ensure that the dependency folders begin with an underscore to prevent unintended overriding of a project's theme folders. (see #451 and its commit, 57aa564)

Since this was committed over a year ago, I'm assuming that this hasn't been a problem for many others. But we are in need of a path forward for this feature. I welcome any ideas that could help us resolve this. Currently I've been having to remove the leading period from the directories and in the code referencing the directories and it instantly begins working again.

merauluka pushed a commit to merauluka/php-code-coverage that referenced this issue Nov 20, 2018
merauluka pushed a commit to merauluka/php-code-coverage that referenced this issue Nov 20, 2018
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

No branches or pull requests

2 participants