Based on the tests that I have seen in this module my gut says that the following modules are missing from info.yml files:
- field
- entity_reference

It would be also good to know why `system` module must to be enabled in tests.

The easiest way to find missing dependencies is installing Drupal with minimal profile and enabling this and PBE submodule.

composer.json should contains a dependency to drupal/core.

Comments

mxr576 created an issue. See original summary.

  • Peter Majmesku committed 2fe9f1e on 8.x-1.x 
    Issue #2965008: Missing dependencies in info.yml and composer.json
jepster_’s picture

jepster_
Location 🇩🇪Düsseldorf
commented
Status: Active » Fixed

The system module is essential, since it is a basic core module. It's description says: "Handles general site configuration for administrators.". I have been needing it for Kernel Base Tests.

Thanks for the hint regarding the dependencies. I have added them into the dev branch.

jepster_’s picture

jepster_
Location 🇩🇪Düsseldorf
commented
Status: Fixed » Closed (fixed)
mxr576’s picture

mxr576
Primary language Hungarian
Location Hungary
commented

Why did you removed taxonomy as a dependency in your commit? I did not say it is unnecessary.
Also, I'd use "^8.0" in the composer as a dependency instead of `~8`.

jepster_’s picture

jepster_
Location 🇩🇪Düsseldorf
commented

Thanks for the hint. Sorry, I missed that one. I have updated the code in the dev branch.

jepster_’s picture

jepster_
Location 🇩🇪Düsseldorf
commented

Has been released in version 8.x-1.49. See https://www.drupal.org/project/permissions_by_term/releases/8.x-1.49.