-
Notifications
You must be signed in to change notification settings - Fork 24
Use HTTP server for TUF conformance testing #1045
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
Conversation
52c7a19
to
e109a4e
Compare
30ed39b
to
981e60a
Compare
btw, is it possible to increase the verbosity of the client code somehow? Doesn't have to be a CLI flag or anything if that's complicated, a code change is fine, but would be useful to be able to see a little more of the client decisions... |
We don't really have much logging. But what are you looking for? |
was just interested to see what sort of decisions the client is making in general -- e.g. was interested in but if there's no logging like that then I know I'll have have to add some if I really want to know. |
there is some issue with the stream redirection:
What I imagine might happen (caveat: I barely read java):
This might not be important enough to fix considering what Appu said -- there's not much logging in the code. |
yeah I think we could run the server in a mode which tees the output to stdout/err as well as returning it to the conformance suite. So if someone happened to be testing locally running the server in a different terminal than the tests, then we'd be okay. |
7a6046b
to
93f902f
Compare
296b58e
to
0313c41
Compare
Signed-off-by: Aaron Lew <[email protected]>
0313c41
to
401e4d7
Compare
lets merge this for now, but I'd like to find a way to only keep one xfails file. So lets open an issue or update the build immediate after this. |
Use HTTP server for TUF conformance testing Signed-off-by: Aaron Lew <[email protected]>
Summary
This change speeds up TUF conformance testing by using an HTTP server.