Problem/Motivation

There is a test view test_history which lives in views.module, but the test code itself lives in history.module.

In order to remove history.module we should move the view.

Steps to reproduce

Proposed resolution

Remaining tasks

Move the view. Update the existing test to use the new location.

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3568027

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

longwave created an issue. See original summary.

quietone made their first commit to this issue’s fork.

quietone opened merge request !14396

quietone’s picture

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

dcam commented
Status: Needs review » Reviewed & tested by the community

Looks OK to me. Tests are passing. I verified the class change by moving the view config myself. There isn't much to say about it. I grepped the views directory for "history" and didn't come up with any other references to the module.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Version: main » 11.x-dev
Status: Reviewed & tested by the community » Fixed

Committed and pushed 33cc9950e3f to main and 7bf33f22070 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 committed 7bf33f22 on 11.x 
    task: #3568027 Move views.view.test_history to history module

By:...

  • longwave committed 33cc9950 on main 
    task: #3568027 Move views.view.test_history to history module

By:...

longwave closed merge request !14396

Status: Fixed » Closed (fixed)

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