Alias case sensitivity after the removal of the preload cache

Problem/Motivation

Since #3496369: Multiple load path aliases without the preload cache the Redirect module tests have been failing. Specifically \Drupal\Tests\redirect\Functional\GlobalRedirectTest::testRedirects when it asserts that 'Test-node' will be redirected to 'test-node'

Steps to reproduce

Run the test

Proposed resolution

The problem was caused by the removal of the AliasManager cache and how the static cache works on \Drupal\path_alias\AliasManager::getPathByAlias(). In head this does:

    // Look for path in storage.
    if ($path_alias = $this->pathAliasRepository->lookupByAlias($alias, $langcode)) {
      $this->lookupMap[$langcode][$path_alias['path']] = $alias;
      return $path_alias['path'];
    }

which results in the node/1 path having the alias Test-node in the redirect test. It should be

    // Look for path in storage.
    if ($path_alias = $this->pathAliasRepository->lookupByAlias($alias, $langcode)) {
      $this->lookupMap[$langcode][$path_alias['path']] = $path_alias['alias'];
      return $path_alias['path'];
    }

which would result in node/1 path having the correct alias test-node.

Remaining tasks

Fix alias manager to store the correct alias and add missing test coverage to core.

User interface changes

None

Introduced terminology

N/a

API changes

None

Data model changes

None

Release notes snippet

N/a