Skip to content

Use testcontainers#741

Merged
hierynomus merged 2 commits intohierynomus:masterfrom
vladimirlagunov:use-testcontainers
Nov 10, 2021
Merged

Use testcontainers#741
hierynomus merged 2 commits intohierynomus:masterfrom
vladimirlagunov:use-testcontainers

Conversation

@vladimirlagunov
Copy link
Copy Markdown
Contributor

It allows running different SSH servers with different configurations in tests, giving ability to cover more bugs, like mentioned in #733.

It allows running different SSH servers with different configurations in tests, giving ability to cover more bugs, like mentioned in hierynomus#733.
@hierynomus
Copy link
Copy Markdown
Owner

🤩 Thanks @vladimirlagunov

@hierynomus hierynomus merged commit d5805a6 into hierynomus:master Nov 10, 2021
vladimirlagunov added a commit to vladimirlagunov/sshj that referenced this pull request Jan 21, 2022
* Replace abstract class IntegrationBaseSpec with composition through IntegrationTestUtil

* Switch to testcontainers in integration tests

It allows running different SSH servers with different configurations in tests, giving ability to cover more bugs, like mentioned in hierynomus#733.

(cherry picked from commit d5805a6)
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