Problem/Motivation

PHPUnit tests fail when testing install configuration due to the path not existing.

Steps to reproduce

Write phpunit test to assert configuration was installed

Proposed resolution

Change xmlsitemap.install to use fromUri(base) instead as this does not need the front page route.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

adam-delaney created an issue. See original summary.

adam-delaney’s picture

Uploading patch

adam-delaney’s picture

Status: Active » Needs review
adam-delaney’s picture

Status: Needs review » Needs work

The last submitted patch, 2: 3232394-2.patch, failed testing. View results

Dave Reid’s picture

Status: Needs work » Needs review
FileSize
687 bytes

Fixing the extra argument in Url::fromUri() that might fix the tests.

  • adam-delaney authored 994b84a on 8.x-1.x
    Issue #3232394 by adam-delaney, Dave Reid: Fixed error if front page...

  • adam-delaney authored 994b84a on 2.x
    Issue #3232394 by adam-delaney, Dave Reid: Fixed error if front page...
Dave Reid’s picture

Status: Needs review » Fixed

Committed #6 to 8.x-1.x! Thanks!

Status: Fixed » Closed (fixed)

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