Problem/Motivation

Because of #3083275: [meta] Update tests that rely on Classy to not rely on it anymore and Classy being deprecated in Drupal 9 + removed in Drupal 10,: Tests that aren't specifically testing Classy yet declare $defaultTheme = 'classy'; should be refactored to use Stark as the default theme instead.

Proposed resolution

Change all tests in this module to use Stark as the default theme, and refactor the tests where needed so they continue to function properly.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#2 3269266-2.patch2.34 KBdanflanagan8

Comments

danflanagan8 created an issue. See original summary.

danflanagan8’s picture

danflanagan8
Primary language English
Location St. Louis, US
commented
Assigned: danflanagan8 » Unassigned
Status: Active » Needs review
StatusFileSize
new 3269266-2.patch2.34 KB

This is a teeny tiny one. Just one test using classy.

solideogloria’s picture

solideogloria commented

Removed, since I posted in the wrong issue.

danflanagan8’s picture

danflanagan8
Primary language English
Location St. Louis, US
commented

@solideogloria, this issue is a task that has a very limited scope that is unrelated to your feature request. You may want to put a feature request into the issue queue. But posting on this issue is not going to get you very far. Cheers!

solideogloria’s picture

solideogloria commented

Sorry, posted in the wrong issue. I was looking at a different one.

longwave’s picture

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

As stated this is a nice easy one.

  • lauriii committed c6aab40 on 10.0.x 
    Issue #3269266 by danflanagan8, longwave: Contextual Links tests should...

  • lauriii committed dd3c026 on 9.4.x 
    Issue #3269266 by danflanagan8, longwave: Contextual Links tests should...
lauriii’s picture

lauriii
he/him
Primary language Finnish
Location Finland
commented
Status: Reviewed & tested by the community » Fixed

Committed c6aab40 and pushed to 10.0.x. Also cherry-picked to 9.4.x. Thanks!

Status: Fixed » Closed (fixed)

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