Problem/Motivation

The Search module is used in olivero_test

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3586616

Command icon 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:

About issue forks

Comments

quietone created an issue. See original summary.

quietone opened merge request !15552

quietone’s picture

quietone commented
Status: Active » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

quietone’s picture

quietone commented
Status: Fixed » Needs review
dcam’s picture

dcam commented
Status: Needs review » Needs work

I can't verify that the tests are actually running. Please check the test output. I expected to see two instances of a test named oliveroRemoveBlockConsoleTest.js run, but there's only one. It appears to be the original one in core/tests/Drupal/Nightwatch. I also expected to see oliveroSearchFormTest.js in the output, but it's missing.

I don't know anything about Nightwatch testing, but it looks like there may be an expectation that the JS files should be in a Tests subdirectory, e.g. core/modules/search/tests/src/Nightwatch/Tests/. The three other core modules with Nightwatch tests (ckeditor, navigation, and toolbar) follow this convention. Maybe they need to be moved.

Otherwise the code splits and moves look OK to me. I didn't really find anything to comment about.