Move non-migration tests to Forum in preparation for deprecation

Removed all the migration changes so they can be done in a separate test.

The failing test was InstallUninstallTest. This tests that all modules can be installed and uninstalled. However, deprecated modules are excluded from the list of all modules so I think that the forum specific code should be removed when Forum is marked deprecated. For, now I have added @todo statements.

DependencyTest - This is testing core functionality of install and uninstall so I don't think that a forum specific version is needed. However, I have left the forum version in the MR in case that is wrong.

Left a review, thanks for working on this

Updated the MR and replied, so setting back to NR.

Looks good to me, I agree, the existing coverage of path processing for /user is sufficient.

Thank you for working on this 💙

The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Rebased and tests passing, restoring RTBC

The MR doesn't apply to 10.1.x, following 📌 Remove dead class PathProcessor in url_alter_test module Fixed where core/modules/system/tests/modules/url_alter_test/src/PathProcessor.php was removed.

Once again, GitLab has decided I can't push, after just pushing. Therefore, here is a patch.

Applied patch #29 and did a search for 'forum'.

All the instances I found (outside Forum module) were in migrate code. Assuming that is fine.

Removing credit for @ravi.shankar as this is 10.1.x only so a 10.0.x patch isn't needed.

Committed to 10.1.x and pushed, thanks!

quietone → closed merge request !1815

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