The following line of code is in the tests:

$path['options']['base_url'] = $GLOBALS['base_url'] . base_path();

Concatenating $GLOBALS['base_url'] and base_path() shouldn't ever be done as the subdirectory will appear twice:

Example:

$GLOBALS['base_url'] == 'http://example.com/drupal';
base_bath() == '/drupal/';
$GLOBALS['base_url'] . base_bath() == 'http://example.com/drupal/drupal/';
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

stefan.r’s picture

Status: Active » Needs review
FileSize
949 bytes
stefan.r’s picture

There is a further issue in the tests if they are run from a URL with a language prefix that does not exist in the sandbox (same as #2284727).

Updated patch attached.

DamienMcKenna’s picture

Status: Needs review » Closed (duplicate)
stefan.r’s picture

@DamienMcKenna, I actually don't think this is a duplicate. This issue refers to two bugs in the Global Redirect automated test itself, neither of which is caused by the (similar) issue in #304025: not working when drupal is installed in a subdirectory

DamienMcKenna’s picture

Status: Closed (duplicate) » Needs review

Fair enough.

Chris Matthews’s picture

The 5 year old patch in #2 to globalredirect.test applied cleanly to the latest 7.x-1.x-dev, but still needs community review & testing.