Skip to content

[POC] end-to-end Integration test #13

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

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

banadiga
Copy link
Member

@banadiga banadiga commented Jul 13, 2017

  • Module integration-test
  • with end-to-end test
  • Extracted Dockerfile
  • spring boot /info endpoint
  • Replace hard coded hosts and ports
  • Add public client dep on Client service #12
  • Rewrite test with public client

ADD api-gateway-service.jar api-gateway-service.jar
ADD wrapper.sh wrapper.sh
RUN bash -c 'chmod +x /wrapper.sh'
RUN bash -c 'touch /app.jar'
Copy link

Choose a reason for hiding this comment

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

opechatka

links:
- discovery-server
environment:
- WATE=true
Copy link

Choose a reason for hiding this comment

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

WTF? ;)

private static final String STORAGE_SERVICE = "storage-service_1";
private static final int STORAGE_SERVICE_PORT = 8091;

private static final String compose = EntToEndIntegrationTest.class.getClassLoader()
Copy link

Choose a reason for hiding this comment

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

uppercase?

@banadiga banadiga changed the title [POC] Integration test inited [POC] end-to-end Integration test Jul 18, 2017
@banadiga banadiga force-pushed the integration-tests branch from 21f0d8d to ddb6e3f Compare July 19, 2017 06:25
Copy link
Collaborator

@flylight flylight left a comment

Choose a reason for hiding this comment

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

Please, take a look on my comments.


if [ "$WAITING_FOR_DEPENDENCE" != "true" ]; then
echo "Starting config server immediately"
java -jar ./client-service.jar
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this file is autogenerated ? If yes then do we need it to be pushed ? If no then should we always maintain it and support configuration ?

Thread.sleep(SLEEP);
try {
httpResponse = client.execute(httpGet);
} catch (IOException ignored) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe throw it away instead of ignoring?

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.

3 participants