Problem/Motivation

Preparation for deprecating the book module.

Move non-migration book-related tests to the book module, so it can be easily reincarnated as a contrib module.

Steps to reproduce

Proposed resolution

Find the tests and move them

Remaining tasks

Patch
Review
Commit

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3376271

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

Spokje created an issue. See original summary.

Spokje opened merge request !4451

spokje’s picture

spokje
Primary language Dutch
commented
Assigned: spokje » Unassigned
Status: Active » Needs review
smustgrave’s picture

smustgrave commented
Status: Needs review » Needs work

Think there are some additional tests in

HandlerAllTest
OliveroTest
UpdatePathTestBaseFilledTest

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

quietone’s picture

quietone commented
Status: Needs work » Needs review

UpdatePathTestBaseFilledTest involves the database dump and should be handled in a separate issue.

smustgrave’s picture

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

Could a follow up be opened for UpdatePathTestBaseFilledTest

  • catch committed 5b37bdb1 on 10.2.x 
    Issue #3376271 by quietone, Spokje, smustgrave: Move non-migration book-...

  • catch committed 30512d2e on 11.x 
    Issue #3376271 by quietone, Spokje, smustgrave: Move non-migration book-...
catch’s picture

catch
he/him
Primary language English
commented
Version: 11.x-dev » 10.2.x-dev
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 11.x and cherry-picked to 10.2.x, thanks!

quietone closed merge request !4451

Status: Fixed » Closed (fixed)

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