Skip to content

Commit f881eef

Browse files
committed
add comment to clarify afterMethod
1 parent 8cd74dc commit f881eef

File tree

2 files changed

+1
-1
lines changed

2 files changed

+1
-1
lines changed

src/main/java/io/managed/services/test/Environment.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,6 @@ public class Environment {
108108
* Setup constants from env variables or set default
109109
*/
110110
public static final String CLOUD_PROVIDER = getOrDefault(CLOUD_PROVIDER_ENV, "aws");
111-
112111
public static final String SUITE_ROOT = System.getProperty("user.dir");
113112
public static final Path LOG_DIR = getOrDefault(LOG_DIR_ENV, Paths::get, Paths.get(SUITE_ROOT, "target", "logs")).resolve("test-run-" + DATE_FORMAT.format(LocalDateTime.now()));
114113

src/test/java/io/managed/services/test/rhoas/KafkaRhoasBasicTests.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,7 @@ public void bootstrap() {
9595
@AfterMethod(alwaysRun = true)
9696
@SneakyThrows
9797
public void cleanMethod(Method method) {
98+
// Restore the owner to PRIMARY, after testing that changing owners works.
9899
if (method.getName().equals("testUpdateKafkaOwner")) {
99100
var adminOfflineToken = Environment.ADMIN_OFFLINE_TOKEN;
100101
var kafkaMgmtApi = KafkaMgmtApiUtils.kafkaMgmtApi(Environment.OPENSHIFT_API_URI, adminOfflineToken);

0 commit comments

Comments
 (0)