search

IBMStockTrader / trader

UI microservice for the Stock Trader app
Apache License 2.0
24 stars 63 forks source link

Integration tests and dockerfile fixes #12

Closed gasgithub closed 3 years ago

gasgithub commented 3 years ago

PR contains:

  1. Integration tests changes Changed pom.xml to correctly run tests. Changed existing test to test /health/live and /health/ready endpoint Added 'application endpoint' test to check /trader/login url
  2. Fixed Dockerfile In the current form docker image was not building due to incorrect paths. Also changed the target directory location to recommended /config instead of full /opt/ol/... path
  3. Changed the generated war name from trader-1.0-SNAPSHOT.war to just TraderUI.war which is used in the dockerfile anyway.