Problem/Motivation

Search module is being deprecated. Usages is tests not in Search needs to removed and test coverage moved to the search module.

Steps to reproduce

Move jsonapi tests to the contact module.

Proposed resolution

Move tests from jsonapi and user to Search

  • core/modules/contact/tests/src/Functional/Jsonapi/ContactFormTest.php
  • core/modules/user/tests/src/Functional/UserSearchTest.php

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3565804

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’s picture

quietone commented
Title: Move JSON:API coverage for Search to the Search module » Move JSON:API and user search coverage for Search to the Search module
Issue summary: View changes

quietone opened merge request !14228

quietone’s picture

quietone commented
Status: Active » Needs review
smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +Needs Review Queue Initiative

Seems straight forward.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed 00c5ad6d2c3 to main and cb0ba704db6 to 11.x. Thanks!

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.

longwave closed merge request !14228

  • longwave committed cb0ba704 on 11.x 
    refactor: #3565804 Move JSON:API and user search coverage for Search to...

  • longwave committed 00c5ad6d on main 
    refactor: #3565804 Move JSON:API and user search coverage for Search to...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.