refactor: extract validator-data-address-s3 module

[ndr-brt]

What this PR changes/adds

Extracts validator-data-address-s3 module

Why it does that

modularity

Further notes

• cleaned up end to end tests a little and ci workflows as well

Linked Issue(s)

Closes #352

Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

[github-actions]

Test Results

17 files  ±0  17 suites  ±0   26s ⏱️ +17s
62 tests  - 2  62 ✅  - 2  0 💤 ±0  0 ❌ ±0 
64 runs   - 2  64 ✅  - 2  0 💤 ±0  0 ❌ ±0 

Results for commit 5ebbd15. ± Comparison against base commit 484fb46.

This pull request removes 19 and adds 17 tests. Note that renamed tests count towards both.

org.eclipse.edc.aws.s3.S3DestinationDataAddressValidatorExtensionTest ‑ initialize(S3DataAddressValidatorExtension, ServiceExtensionContext)
org.eclipse.edc.aws.s3.validation.S3DestinationDataAddressValidatorTest ‑ should_fail_when_required_fields_are_missing()
org.eclipse.edc.aws.s3.validation.S3DestinationDataAddressValidatorTest ‑ should_pass_when_data_address_is_valid()
org.eclipse.edc.aws.s3.validation.S3SourceDataAddressValidatorTest ‑ should_fail_when_both_object_name_and_prefix_are_missing()
org.eclipse.edc.aws.s3.validation.S3SourceDataAddressValidatorTest ‑ should_fail_when_region_or_bucketName_is_missing()
org.eclipse.edc.aws.s3.validation.S3SourceDataAddressValidatorTest ‑ should_pass_when_data_address_is_valid_with_object_name()
org.eclipse.edc.aws.s3.validation.S3SourceDataAddressValidatorTest ‑ should_pass_when_data_address_is_valid_with_object_prefix()
org.eclipse.edc.connector.dataplane.aws.s3.S3DataSinkFactoryTest ‑ [1] validBucketName,  , validAccessKeyId, validSecretAccessKey
org.eclipse.edc.connector.dataplane.aws.s3.S3DataSinkFactoryTest ‑ [2]  , validRegion, validAccessKeyId, validSecretAccessKey
org.eclipse.edc.connector.dataplane.aws.s3.S3DataSinkFactoryTest ‑ validate_ShouldSucceedIfPropertiesAreValid()
…

org.eclipse.edc.aws.s3.validator.S3DataAddressValidatorExtensionTest ‑ initialize(S3DataAddressValidatorExtension, ServiceExtensionContext)
org.eclipse.edc.aws.s3.validator.S3DestinationDataAddressValidatorTest ‑ should_fail_when_required_fields_are_missing()
org.eclipse.edc.aws.s3.validator.S3DestinationDataAddressValidatorTest ‑ should_pass_when_data_address_is_valid()
org.eclipse.edc.aws.s3.validator.S3SourceDataAddressValidatorTest ‑ should_fail_when_both_object_name_and_prefix_are_missing()
org.eclipse.edc.aws.s3.validator.S3SourceDataAddressValidatorTest ‑ should_fail_when_region_or_bucketName_is_missing()
org.eclipse.edc.aws.s3.validator.S3SourceDataAddressValidatorTest ‑ should_pass_when_data_address_is_valid_with_object_name()
org.eclipse.edc.aws.s3.validator.S3SourceDataAddressValidatorTest ‑ should_pass_when_data_address_is_valid_with_object_prefix()
org.eclipse.edc.connector.dataplane.aws.s3.S3DataSinkFactoryTest ‑ validate_shouldFail_whenValidatorFails()
org.eclipse.edc.connector.dataplane.aws.s3.S3DataSinkFactoryTest ‑ validate_shouldSucceed_whenValidatorSucceeds()
org.eclipse.edc.connector.dataplane.aws.s3.S3DataSinkTest ‑ [1] [org.eclipse.edc.connector.dataplane.spi.pipeline.InputStreamDataSource@1a67bde5]
…

♻️ This comment has been updated with latest results.