- Issue created by @bakop
- last update
over 1 year ago 23 pass, 24 fail - @bakop opened merge request.
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 2:33pm 10 August 2023 - last update
over 1 year ago 46 fail - last update
over 1 year ago 46 fail - last update
over 1 year ago 46 fail - Status changed to Active
over 1 year ago 3:06pm 11 August 2023 - 🇫🇷France bakop Lille
This visually solves the issue, but it breaks the tests. I'll look into modifying my stack to accommodate JavaScript functional tests and see how to address this more properly.
- last update
over 1 year ago 23 pass, 24 fail - Status changed to Needs review
over 1 year ago 1:41pm 16 August 2023 - 🇫🇷France bakop Lille
Finally, there seems to be an issue with loading test dependencies, such as the YouTube module for instance. This is not due to the fix, as I encounter the same problem without making any modifications. If anyone has an idea to fix this, please let me know.
- 🇮🇳India naveen.kumawat.metacube
Hi bakop, thank you for your quick solution, it resolved my blocker to move on!
- 🇫🇷France bakop Lille
Issue with test dependencies seems to be resolved in https://www.drupal.org/project/field_states_ui/issues/3416385#comment-15... 🐛 YoutubeFieldTest::testYoutubeField Undefined array key "youtube" Active
- Status changed to RTBC
8 months ago 5:04pm 19 April 2024 - 🇬🇧United Kingdom jofitz
I would suggest removing uuid from the default
$configuration
array because it is redundant AFAIK, but it could be argued that change is beyond the scope of this issue:$configuration += [ 'data' => [], 'uuid' => '', ];
Once 🐛 YoutubeFieldTest::testYoutubeField Undefined array key "youtube" Active has been committed then these test failures will disappear, therefore this will be ready to merge too.
Setting to RTBC to speed the process along, but if my suggested change is made then I will review again.