Merge grails-spring-security, grails-mail, and grails-redis

[jdaugherty]

Per vote https://lists.apache.org/thread/72rsnnnpcqdn642478xwd8go42rlpt6y , this merges grails-mail, grails-spring-security, and grails-redis.

Some highlights of this PR:

1. Add a new module: grails-testing-support-redis to facilitate running the redis based tests like the mongodb tests.
2. code formatting of the imported code bases (since our code style validations apply)
3. removal of spring dependency management plugin from spring security (transitive dependencies were resolving & it's incompatible with the composite build)
4. Selenium was unpinned in this merge from '4.25.0', this was supposed to be a temporary workaround because of https://issues.chromium.org/issues/42323769 The real cause was not having a wait { }

After merging this to 7.0.x, I will merge all the way up to 8.0.x and then pull in the 8.x specific changes.

[jdaugherty]

@jamesfredley since this is a contribution of code from an external source, you must review it as an ASF member.

[jamesfredley]

As an ASF member I approve the contribution of this apache 2.0 licensed code being included in grails-core

[jamesfredley]

Reviewing it now

[jamesfredley]

This leaves the multi-connection Redis coverage disabled. Since the PR is adding Redis CI coverage, the multi-pool wiring should either be exercised via testcontainers/CI-provisioned Redis instances or this disabled test should not be counted as covered behavior.

[jdaugherty]

This is out of scope of this change. I'm happy to make an issue for this, but we're just trying to merge libraries and I've already done enough on fixing the existing problems.

[jamesfredley]

Finding 4 - new Spring Security and Redis CI jobs are not publish gates.

.github/workflows/gradle.yml has publish depend on publishGradle, �uild, unctional, hibernate5Functional, and mongodbFunctional, but not the new security or
edis jobs. As written, snapshot publication can proceed even if those new lanes fail. Please add them to
eeds and gate the if: expression on their results.

[jdaugherty]

@jamesfredley good catch on the path renames. I've found a few other issues as a result and expanded our verification scripts to catch these cases in the future.

[jdaugherty]

Finding 4 - new Spring Security and Redis CI jobs are not publish gates.

.github/workflows/gradle.yml has publish depend on publishGradle, �uild, unctional, hibernate5Functional, and mongodbFunctional, but not the new security or edis jobs. As written, snapshot publication can proceed even if those new lanes fail. Please add them to eeds and gate the if: expression on their results.

This was a good catch. I've updated the PR for this.

[jdaugherty]

@jamesfredley I believe all of your feedback has been implemented. On the slack topic, I don't see anything different about the imported SVGs. It seems like a concern separate from this PR as they're the same SVGs that are in grails-core already / part of the application generation.

[testlens-app]

✅ All tests passed ✅

🏷️ Commit: b30b80a
▶️ Tests: 32772 executed
⚪️ Checks: 51/51 completed

---

Learn more about TestLens at testlens.app.