I tried to add a test for the file resource at the REST module and I saw that the service fails when it is being enabled. The error is the following:

Drupal\Component\Plugin\Exception\PluginException: The plugin (entity:file) did not specify an instance class. in Drupal\Component\Plugin\Factory\DefaultFactory::getPluginClass() (line 62 of /home/juampy/projects/drupal8/core/lib/Drupal/Component/Plugin/Factory/DefaultFactory.php).

A patch with the test will be attached in the following comment.

CommentFileSizeAuthor
#4 drupal-file-resource-test-2128783-1.patch1.73 KBjuampynr
#1 drupal-file-resource-test-2128783-1.patch1.73 KBjuampynr
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

juampynr’s picture

juampynr CreditAttribution: juampynr commented
FileSize
drupal-file-resource-test-2128783-1.patch1.73 KB
juampynr’s picture

juampynr CreditAttribution: juampynr commented
juampynr’s picture

juampynr CreditAttribution: juampynr commented
Status: Active » Needs review

Here is the patch with the test.

juampynr’s picture

juampynr CreditAttribution: juampynr commented
FileSize
drupal-file-resource-test-2128783-1.patch1.73 KB

Attaching again to see if it triggers the testbot.

juampynr’s picture

juampynr CreditAttribution: juampynr commented
Related issues: +#2128791: File resource needs an access controller

Status: Needs review » Needs work

The last submitted patch, 1: drupal-file-resource-test-2128783-1.patch, failed testing.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented

@juampy
Are you sure you upload the right file?

moshe weitzman’s picture

moshe weitzman CreditAttribution: moshe weitzman commented

Any chance we can get this going again?

juampynr’s picture

juampynr CreditAttribution: juampynr commented

Sure! I will find some time today.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented

You're not enabling the file module, so that doesn't seem very surprising to me?

juampynr’s picture

juampynr CreditAttribution: juampynr commented
Status: Needs work » Closed (duplicate)

This test fits perfectly to prove #2128791: File resource needs an access controller so I am closing this one as a duplicate since I have added this test part of the patch of that issue.