Skip to content
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

Add integration test for endpoint #3

Closed
binkley opened this issue Dec 8, 2016 · 5 comments
Closed

Add integration test for endpoint #3

binkley opened this issue Dec 8, 2016 · 5 comments
Assignees
Projects

Comments

@binkley
Copy link
Owner

binkley commented Dec 8, 2016

No description provided.

@binkley
Copy link
Owner Author

binkley commented Dec 10, 2016

Interestingly, much easier to do a full integration test than just a simple controller test with mocking!

For example, TestRestTemplate is only present for injection with Boot 1.4 when allocating an actual port for the web environment (e.g., RANDOM_PORT), not when mocking (MOCK).

@binkley
Copy link
Owner Author

binkley commented Dec 10, 2016

The "simplest thing that could possibly work" works easily in IntelliJ, but fails in mvn verify.

@binkley
Copy link
Owner Author

binkley commented Dec 10, 2016

See spring-projects/spring-boot#6254.

So, telling Spring Boot Plugin to give the executable jar a classifier of "exec" (i.e., sproingk-0-SNAPSHOT-exec.jar).

binkley added a commit that referenced this issue Dec 10, 2016
More work to go, such as not being so darn noisy!
@binkley binkley self-assigned this Dec 11, 2016
@binkley
Copy link
Owner Author

binkley commented Dec 11, 2016

Before moving to test, quiet build.

binkley pushed a commit that referenced this issue Dec 12, 2016
binkley pushed a commit that referenced this issue Dec 12, 2016
@binkley
Copy link
Owner Author

binkley commented Dec 15, 2016

Investigate if the maven goals are set up wrong:

--- maven-failsafe-plugin:2.19.1:integration-test (run-integration-tests) @ sproink ---
Skipping execution of surefire because it has already been run for this configuration

@binkley binkley reopened this Dec 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

1 participant