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

Artipie http3 test #13

Merged
merged 6 commits into from
Nov 30, 2023

Conversation

olenagerasimova
Copy link
Member

@olenagerasimova olenagerasimova commented Nov 30, 2023

Added integration test for plugin and artipie, also:

  • get rid of artipie ppom usage (I don't think it's appropriate here, will check release after merge)
  • removed unused deps and plugins from pom, used modern junit version
  • changed jetty http api usage to get response body as InputStream (via InputStreamResponseListener) instead of byte array after getting buffering capacity exceeded error for some heavy jar

MatcherAssert.assertThat("Maven install is not successful", exec.getExitCode() == 0);
MatcherAssert.assertThat(
res,
Matchers.stringContainsInOrder(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How can it prove that the packages were gotten by HTTP 3?

Copy link
Member Author

@olenagerasimova olenagerasimova Nov 30, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a) https://artipie:8091/my-maven-proxy - is a path to artipie http3 repository
b) is such configuration maven client can perform request over http3 only (it fails trying to get dependency from central over http 1)

So there are only 2 possibilities in current configuration: either build is success and deps were obtained via http3 or it fails

@olenagerasimova
Copy link
Member Author

@dgarus I've corrected logs and assertion

Copy link
Contributor

@dgarus dgarus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@olenagerasimova olenagerasimova merged commit 57d6d4f into artipie:master Nov 30, 2023
3 checks passed
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.

2 participants