-
Notifications
You must be signed in to change notification settings - Fork 24
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
issue-183: Use tags instead of suites to separate tests #184
Conversation
54d5429
to
c6abda9
Compare
Reported bugs were not introduced with this PR but simply reported here because tests are now executing. |
b7a00d2
to
8b098ed
Compare
d84ef26
to
d35f100
Compare
SonarCloud Quality Gate failed. |
The failing quality gate is related to decreased code coverage which seems to be because SonarQube is (for some unknown reason) ignoring the integration test reports. We should probably look into it as a follow-up. Other than that - this PR is ready. |
...a/org/carlspring/cloud/storage/s3fs/fileSystemProvider/NewAsynchronousFileChannelTestIT.java
Show resolved
Hide resolved
src/test/java/org/carlspring/cloud/storage/s3fs/fileSystemProvider/NewByteChannelIT.java
Show resolved
Hide resolved
Could you please create a follow-up task for this? Thanks! |
Follow up here: |
d35f100
to
3b73308
Compare
src/test/java/org/carlspring/cloud/storage/s3fs/util/EnvironmentBuilder.java
Outdated
Show resolved
Hide resolved
src/test/java/org/carlspring/cloud/storage/s3fs/util/EnvironmentBuilder.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! :)
This allows to reduce build time for running integration tests in CIs (i.e. when the unit tests have already ran in a previous build job and can safely be skipped in the current one)
…minio and finally it-s3 This should reduce S3 costs while running tests because MinIO has S3 compatible API. If MinIO tests fail, it's highly likely the S3 ones will fail or succeed as well.
9f8ac79
to
b314cf1
Compare
b314cf1
to
8320d7f
Compare
Pull Request Description
This pull request closes #183.
It is important to note this pull request has introduced two new profiles -
it-s3
andit-minio
which are used to trigger tests tagged with@S3IntegrationTest
and@MinioIntegrationTest
.Acceptance Test
mvn clean install -Punit-tests,it-minio,it-s3
still works.Questions
Does this pull request break backward compatibility?
Does this pull request require other pull requests to be merged first?
Does this require an update of the documentation?