Problem/Motivation
When i'm running PHPUNIT Functional Test case that time facing error. The error message is:
Drupal\Core\Config\Schema\SchemaIncompleteException: Schema errors for views.view.forward_logs with the following errors: views.view.forward_logs:display.default.display_options.filters.status.value variable type is boolean but applied schema class is Drupal\Core\TypedData\Plugin\DataType\StringData
Steps to reproduce
Run any PHPUnit Functional test with the 'views' module enabled, you can got above error message.
Proposed resolution
Go to forward/config/optional : here views.view.forward_logs.yml & views.view.forward_statistics.yml files. Need to update
filters:
status:
value: true
to
filters:
status:
value: '1'
Also uploaded patch fix-forward-log-error.patch file
Remaining tasks
N/A
User interface changes
N/A
API changes
N/A
Data model changes
N/A
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | forward-schema-errors-3291470-7.patch | 936 bytes | suchitnikamshiv |
Issue fork forward-3291470
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
- 3291470-SchemaIncompleteException
changes, plain diff MR !10
Comments
Comment #2
suchitnikamshiv commentedComment #3
sastha commentedComment #4
suchitnikamshiv commentedComment #5
suchitnikamshiv commentedComment #6
sastha commentedCan you please try value: '1'
Comment #7
suchitnikamshiv commentedOk Sastha, I have tried and it's working fine. Also patch file updated. Please check once.
Comment #8
gobinathmDoes this issue Exist in 4.0.1 ?
Comment #9
suchitnikamshiv commentedI have tested on 4.0.1 version, it is showing same error message.
Comment #10
suchitnikamshiv commentedComment #11
dwwConfirmed this is still a problem in the 4.0.x branch, which is the only one still being supported.
We're not currently seeing it with the existing tests, since none of them are enabling views. Once I do that, the test starts failing.
So we need a merge request here, to modify
tests/src/Functional/PermissionsTest.phpto enable views, and perhaps to visit these views and do a little mild access verification on those while we're at it.I'll be working on this later today, assuming no one beats me to it. 😅
Comment #12
dwwBack from lunch. Working on this now.
Comment #13
dwwComment #15
dwwConverted to an MR and added test coverage of the default views. Slick, the test-only job failed as expected:
https://git.drupalcode.org/project/forward/-/jobs/8917489
Comment #18
dwwFixed and merged. This will go out soon in 4.0.3.
Thanks, everyone!