From 8d7089ceb03c782ea15ee4d680399b5813a9a06b Mon Sep 17 00:00:00 2001 From: Mohammad Saddam Hossain Date: Wed, 21 Oct 2026 08:16:14 +0530 Subject: [PATCH] fix(cucumber): quarantine check no longer silently skipped before context init checkCucumberQuarantine called SeleniumBootContext.getConfig(), which throws when the framework is not initialised; the blanket catch swallowed it, so quarantined scenarios ran. Guard with isInitialized(). Fixes the order-dependent CucumberQuarantineTest failures that blocked #184/#185 CI. Co-Authored-By: Claude Sonnet 5.5 --- src/main/java/com/seleniumboot/cucumber/CucumberHooks.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/java/com/seleniumboot/cucumber/CucumberHooks.java b/src/main/java/com/seleniumboot/cucumber/CucumberHooks.java index bd4f00a..0b488b6 100644 --- a/src/main/java/com/seleniumboot/cucumber/CucumberHooks.java +++ b/src/main/java/com/seleniumboot/cucumber/CucumberHooks.java @@ -219,7 +219,7 @@ private static void applyRetryTag(Scenario scenario) { static void checkCucumberQuarantine(Scenario scenario) { try { com.seleniumboot.config.SeleniumBootConfig.Quarantine cfg = - SeleniumBootContext.getConfig().getQuarantine(); + SeleniumBootContext.isInitialized() ? SeleniumBootContext.getConfig().getQuarantine() : null; if (cfg != null && !cfg.isEnabled()) return; // ── 1. In-file tag check (user adds @quarantine to the scenario) ──