Skip to content

Conversation

@portante
Copy link
Member

@portante portante commented Feb 26, 2022

The functional tests were only for the agent. There is no need to run them for the server side.

This partially fixes issue #2647 by always having an agent environment for the functional tests.

This is foundational work for PR #2551.

@portante
Copy link
Member Author

This PR is layered on top of PRs #2646 and #2644. We'll pull this out of Draft mode when those PRs land.

@lgtm-com

This comment was marked as resolved.

@portante portante linked an issue Feb 26, 2022 that may be closed by this pull request
@portante portante linked an issue Feb 27, 2022 that may be closed by this pull request
Copy link
Member

@dbutenhof dbutenhof left a comment

Choose a reason for hiding this comment

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

Pretty simple when stripped of the common layers reviewed elsewhere. Looks fine, but there's no point in approving until those two go in.

webbnh
webbnh previously approved these changes Feb 28, 2022
Copy link
Member

@webbnh webbnh left a comment

Choose a reason for hiding this comment

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

What Dave said...except for the part about approving. 😁

@portante portante force-pushed the rework-func-tests branch 4 times, most recently from 0107fc1 to 0153260 Compare March 1, 2022 16:29
The functional tests were only for the agent.  There is no need to run
them for the server side.

This partially fixes distributed-system-analysis#2647 by always having an agent environment for
the functional tests.
@portante portante force-pushed the rework-func-tests branch from 0153260 to d35f6de Compare March 1, 2022 17:29
@portante portante marked this pull request as ready for review March 1, 2022 17:29
Copy link
Member Author

@portante portante left a comment

Choose a reason for hiding this comment

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

Ready for final review.

@portante portante requested review from dbutenhof and webbnh March 1, 2022 17:30
Copy link
Member

@webbnh webbnh left a comment

Choose a reason for hiding this comment

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

👍

@dbutenhof dbutenhof merged commit 9831f77 into distributed-system-analysis:main Mar 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Run agent/server pytest-based functional tests as a separate test

3 participants