8000 Fixed: GITHUB-297: The "connectTimeout" parameter is not configurable by shaburov · Pull Request #298 · timols/java-gitlab-api · GitHub
[go: up one dir, main page]

Skip to content

Fixed: GITHUB-297: The "connectTimeout" parameter is not configurable#298

Merged
timols merged 3 commits intotimols:masterfrom
shaburov:fix.issue.297.parameter.connectTimeout.is.not.configurable
May 3, 2018
Merged

Fixed: GITHUB-297: The "connectTimeout" parameter is not configurable#298
timols merged 3 commits intotimols:masterfrom
shaburov:fix.issue.297.parameter.connectTimeout.is.not.configurable

Conversation

@shaburov
Copy link
Contributor

Refactoring: add getter for host field GitlabAPI &&
remove unused IOException from the method signature

Refactoring: add getter for host field GitlabAPI &&
remove unused IOException from the method signature
@shaburov
Copy link
Contributor Author

@timols ^

@timols
Copy link
Owner
timols commented Apr 24, 2018

@shaburov this is great. Any way to add a test for this?

@shaburov
Copy link
Contributor Author

@timols
I wanted to write tests, but I did not think of how to simulate a case with the error "connection timeout".
With the development of tests for the error "request timeout", problems would not exactly arise. But with this I do not know what to do.

@shaburov
Copy link
Contributor Author

@timols or did you mean unit tests and not integration tests?

@timols
Copy link
Owner
timols commented Apr 26, 2018

Unit tests would be a great start!

@shaburov
Copy link
Contributor Author
shaburov commented May 3, 2018

@timols ^ review please

@timols timols merged commit 58b0179 into timols:master May 3, 2018
@timols
Copy link
Owner
timols commented May 3, 2018

@shaburov this is great. Thanks for this contribution!

@shaburov shaburov deleted the fix.issue.297.parameter.connectTimeout.is.not.configurable branch May 3, 2018 22:35
@shaburov
Copy link
Contributor Author
shaburov commented May 3, 2018

@timols I am glad to help

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

Comments

0