Scheduler's automated testing on Drupal.org is now failing at 8.2 and 8.3 due to a views schema mis-match:
[Uncaught exception] Line 93 of core/lib/Drupal/Core/Config/Testing/ConfigSchemaChecker.php:
Drupal\Core\Config\Schema\SchemaIncompleteException: Schema errors for views.view.scheduler_scheduled_content with the following errors:
views.view.scheduler_scheduled_content:display.default.display_options.filters.status.value variable type is boolean but applied schema class is Drupal\Core\TypedData\Plugin\DataType\StringData
in Drupal\Core\Config\Testing\ConfigSchemaChecker->onConfigSave()
There must have been a change in core between 25th August and 12th September which altered the schema checking. The test run due to the commit on 25th Aug was fully green and passed.
The tests still pass with D8.1 https://www.drupal.org/node/3292/qa
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | update to D8.2.0-rc1.png | 198.05 KB | jonathan1055 |
| #2 | 2799095-1.scheduled_content_view.status_is_string.patch | 508 bytes | jonathan1055 |
Comments
Comment #2
jonathan1055 commentedIn 8.3 the view works OK when browsing to /admin/content/scheduled
However, I editted the view, did not change anything but saved, then used Configuration Update to export the view.
Apart from the uuid and _core properties the only other difference was
instead of
for display_options.filters.status on line 433.
I did replicate the test failures at 8.3 locally. This patch should fix the problem.
Comment #4
jonathan1055 commentedThe committed codebase is now passing all tests again, on Drupal 8.1, 8.2 and 8.3
Comment #5
jonathan1055 commentedIt seems that the tests at 8.2.x and 8.3.x pass, but the change to the schema causes failures at 8.1.x. We get the reverse of the problem above:
So, 8.3 and 8.2 want the value as string and 8.1 wants it as boolean. How do we solve that?
I noticed when I upgraded my test 8.2 site to 8.2.0-rc1 there was a database update for views to change all boolean filter values to strings - see attached.
Comment #6
jonathan1055 commentedSimplenews Scheduler #2802407: Add migration support has the same problem.
Comment #7
jonathan1055 commentedNow that Drupal 8.2.0 has been released, we no longer need to support automated testing at 8.1.x, hence this issue can be closed.
Comment #9
jonathan1055 commentedUnassigning.