Skip to content

Commit

Permalink
Integration test fixes
Browse files Browse the repository at this point in the history
- Fix for the docker port output mismatch
  Integration tests were failing because of this.
  https://github.com/mendix/cf-mendix-buildpack/actions/runs/6119616281/job/16614934821

- Use Mx9.7 test app for logging tests
  • Loading branch information
pappachino authored and Yornik committed Sep 12, 2023
1 parent 0f680a9 commit 26b68f9
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion tests/integration/runner.py
Original file line number Diff line number Diff line change
Expand Up @@ -384,7 +384,7 @@ def _get_container_host_port(self, id_or_name, guest_port):
)
if not result[1]:
raise RuntimeError("Cannot get container port: {}".format(result[0]))
return int(result[0].split(":")[1].rstrip())
return int(result[0].split(":")[-1].rstrip())

def _get_container_ids(self, name):
return self._cmd(("docker", "ps", "-aqf", "name={}*".format(name)))[0].rsplit(
Expand Down
2 changes: 1 addition & 1 deletion tests/integration/test_logging.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ class TestCaseLogging(basetest.BaseTest):
def setUp(self):
super().setUp()
self.stage_container(
"Mendix8.1.1.58432_StarterApp.mda",
"BuildpackTestApp-mx9-7.mda",
env_vars={
"LOGGING_CONFIG": json.dumps({"Jetty": "TRACE"}),
"LOG_RATELIMIT": 500,
Expand Down

0 comments on commit 26b68f9

Please sign in to comment.