Add Gradle and Maven version parameters to Java

[etienne-chrp]

closes #260

[etienne-chrp]

@microsoft-github-policy-service agree

[joshspicer]

Thank you for taking the time to add this functionality (and adding the test). I ran the CI and the version that the scenario is trying to pass doesn't seem to exist.

------
 > [dev_containers_target_stage 4/4] RUN cd /tmp/build-features/java_1 && chmod +x ./devcontainer-features-install.sh && ./devcontainer-features-install.sh:
#17 13.51 Enjoy!!!
#17 13.53 Updating /etc/bash.bashrc and /etc/zsh/zshrc...
#17 14.68 Version 18 not found. Available versions:
#17 14.68 19.0.1-open
#17 14.68 11.0.12-open
#17 14.68 11.0.2-open
#17 14.68 8.0.302-open
#17 14.68 8.0.282-open
#17 14.68 8.0.265-open
#17 14.69 ERROR: Feature "./java" (Unknown) failed to install! Look at the documentation at https://github.com/devcontainers/features/tree/main/src/java for help troubleshooting this error.

[Samruddhi Khandale (samruddhikhandale)]

Thanks for opening the PR. Left some comments.

[etienne-chrp]

I created a new commit that is fixing the tests that are failing in the pipeline, as mentioned here: #263 (review)

[etienne-chrp]

I created a new commit that is fixing the tests that are failing in the pipeline, as mentioned here: #263 (review)

Included in the PR is a refactoring from a pipe syntax with grep to a process substitution syntax. The check helper under certain conditions is not accepting the pipe syntax and the grep was executed on the output of the check function, by using the process substitution syntax all the tests are displayed properly.

[joshspicer]

Thank you, this looks great!

[joshspicer]

I created a new commit that is fixing the tests that are failing in the pipeline, as mentioned here: #263 (review)

Included in the PR is a refactoring from a pipe syntax with grep to a process substitution syntax. The check helper under certain conditions is not accepting the pipe syntax and the grep was executed on the output of the check function, by using the process substitution syntax all the tests are displayed properly.

I agree, that check command has room for improvement. A lot of the test scripts do bash -c '<cmd> | grep <word>, but this looks more elegant :)