Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge 4.2.x into 5.0.x #130

Merged
merged 23 commits into from
Jan 31, 2025
Merged

Merge 4.2.x into 5.0.x #130

merged 23 commits into from
Jan 31, 2025

Conversation

matrei
Copy link
Contributor

@matrei matrei commented Jan 31, 2025

No description provided.

matrei added 23 commits January 22, 2025 07:31
This commit introduces explicit delegation to improve convenience and support for IDE autocompletion. The following delegates have been added:

- `browser` object delegate
- `page` object delegate
- `report(String)` method delegate
- `getPageSource()` method delegate
- Refactor `ContainerAwareDownloadSupport` to a more generic support trait and move `createFileInputSource` method to it.
- Move support classes to `support` package.
- This enables IDE autocompletion to display correct parameter names, improving developer experience.
Geb classes lack parameter metadata, which prevents proper IDE autocompletion. To address this, we manually delegate to enable accurate parameter names.

- Add delegate implementations for all Geb interfaces in `ContainerGebSpec`.
- Move container-related logic to the `ContainerSupport` class for better separation of concerns.
- Add `@CompileStatic` where possible.
The `@since` attribute in the `ContainerSupport` class was incorrectly set to `4.1`.
Updated it to the correct version, `4.2`.
Enhance Developer Experience with Explicit Delegation
- Upgrade `gradle-versions-plugin` to the latest release.
- See the [release notes](https://github.com/ben-manes/gradle-versions-plugin/releases/tag/v0.52.0) for details.
… to 3.26.3

- Upgrade `javaparser-core` to the latest patch release.
- See the [release notes](https://github.com/javaparser/javaparser/releases/tag/javaparser-parent-3.26.3) for details.
- Upgrade `selenium` to the latest release.
- See the [changelog](https://github.com/SeleniumHQ/selenium/blob/trunk/java/CHANGELOG) for details.
- Leverage `ContainerGebSpec.pageSource` delegate for improved readability.
- Use an instance of the page object for better type hints.
- Align test project with the main project's Gradle version.
- Migrate from `GRADLE_ENTERPRISE_ACCESS_KEY` to `DEVELOCITY_ACCESS_KEY`.
- Use `--continue` on build and test tasks to report all errors.
- Use `--no-build-cache` on publish to prevent caching issues.
- Remove `SECRING_FILE` environment variable from `publishToSonatype` as it's not needed.
- Update cog emoji in step name.
Miscellaneous cleanup and maintenance tasks
# Conflicts:
#	.github/workflows/gradle.yml
#	.github/workflows/release.yml
#	build.gradle
#	gradle.properties
#	spock-container-test-app/gradle/wrapper/gradle-wrapper.jar
#	spock-container-test-app/gradle/wrapper/gradle-wrapper.properties
#	spock-container-test-app/gradlew
#	spock-container-test-app/gradlew.bat
#	src/testFixtures/groovy/grails/plugin/geb/ContainerAwareDownloadSupport.groovy
#	src/testFixtures/groovy/grails/plugin/geb/ContainerGebSpec.groovy
@matrei matrei merged commit 52f93f2 into 5.0.x Jan 31, 2025
5 checks passed
@matrei matrei deleted the merge-4.2.x-into-5.0.x branch January 31, 2025 10:08
@matrei
Copy link
Contributor Author

matrei commented Feb 1, 2025

Needs grails/grails-gradle-plugin#396 for successful snapshot publishing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant