Skip to content
This repository has been archived by the owner on Apr 16, 2022. It is now read-only.

Remove duplication of server conn params tests #531

Closed
ggalmazor opened this issue Jun 14, 2018 · 1 comment
Closed

Remove duplication of server conn params tests #531

ggalmazor opened this issue Jun 14, 2018 · 1 comment

Comments

@ggalmazor
Copy link
Contributor

With the PR #474 we introduced a side-effect free way of testing server connection params. The new test uses an HTTP interaction layer with improved error handling and feedback.

There are still some places in Briefcase that use the old ServerConnectionTest, which could be replaced with the new tests.
@ggalmazor
Copy link
Contributor Author

Sentry issue: BRIEFCASE-7X

@ggalmazor ggalmazor mentioned this issue Jun 14, 2018
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ggalmazor