Skip to content

Commit 640d806

Browse files
committed
Test if setting event port makes it accessible in GitHub Action
Now with right requirement modified...
1 parent 4f62571 commit 640d806

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

src/test/org/firebirdsql/common/extension/RunEnvironmentExtension.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -65,9 +65,7 @@ public enum EnvironmentRequirement {
6565
@Override
6666
public boolean isMet() {
6767
if ("localhost".equals(DB_SERVER_URL) || "127.0.0.1".equals(DB_SERVER_URL)) {
68-
//return !DB_ON_DOCKER;
69-
// TODO Testing if port is exposed by default if configured
70-
return true;
68+
return !DB_ON_DOCKER;
7169
}
7270
return false;
7371
}
@@ -77,7 +75,9 @@ public boolean isMet() {
7775
public boolean isMet() {
7876
// NOTE: We're assuming that the AUX (event) port is (only) not available on Docker.
7977
// Otherwise, we need to check if we can actually establish an event connection.
80-
return !DB_ON_DOCKER;
78+
//return !DB_ON_DOCKER;
79+
// TODO Testing if port is exposed by default if configured
80+
return true;
8181
}
8282
},
8383
ALL_SRP_PLUGINS {

0 commit comments

Comments
 (0)