fix(test): running a test without Arrange section outputs informative…

… fail reason Closes: #53
Decision-Driven-Development · Jul 31, 2024 · 5615348 · 5615348
1 parent 807d016
commit 5615348
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 14 deletions.
diff --git a/src/main/java/ru/ewc/checklogic/StateFactory.java b/src/main/java/ru/ewc/checklogic/StateFactory.java
@@ -95,6 +95,11 @@ private Map<String, Locator> locatorsFromFile() {
         if (this.src != null) {
             final Map<String, Map<String, Object>> raw =
                 (Map<String, Map<String, Object>>) new Yaml().loadAll(this.src).iterator().next();
+            if (raw == null) {
+                throw new IllegalStateException(
+                    "There is no Arrange section in the test file, you should add one"
+                );
+            }
             raw.forEach((name, data) -> locators.put(name, new InMemoryStorage(data)));
         }
         return locators;

diff --git a/src/main/java/ru/ewc/checklogic/server/WebPages.java b/src/main/java/ru/ewc/checklogic/server/WebPages.java
@@ -101,24 +101,30 @@ private String templateNamed(final String template, final Map<String, String> va
     @SneakyThrows
     private TestResult performTest(final TestData test) {
         final SoftAssertions softly = new SoftAssertions();
-        final FullServerContext target = new ServerContextFactory(this.root)
-            .fromStateFile(Files.newInputStream(new File(test.file()).toPath()));
         TestResult result;
+        final FullServerContext target;
         try {
-            if (!test.command().isEmpty()) {
-                target.perform(test.command());
+            target = new ServerContextFactory(this.root)
+                .fromStateFile(Files.newInputStream(new File(test.file()).toPath()));
+            try {
+                if (!test.command().isEmpty()) {
+                    target.perform(test.command());
+                }
+                for (final String locator : test.expectations().keySet()) {
+                    softly
+                        .assertThat(target.stateFor(locator, test.expectations().get(locator)))
+                        .describedAs(String.format("State for entity '%s'", locator))
+                        .containsExactlyInAnyOrderEntriesOf(test.expectations().get(locator));
+                }
+                softly.assertAll();
+                result = new TestResult(test.toString(), true, "");
+            } catch (final AssertionError error) {
+                result = new TestResult(test.toString(), false, error.getMessage());
             }
-            for (final String locator : test.expectations().keySet()) {
-                softly
-                    .assertThat(target.stateFor(locator, test.expectations().get(locator)))
-                    .describedAs(String.format("State for entity '%s'", locator))
-                    .containsExactlyInAnyOrderEntriesOf(test.expectations().get(locator));
-            }
-            softly.assertAll();
-            result = new TestResult(test.toString(), true, "");
-        } catch (final AssertionError error) {
-            result = new TestResult(test.toString(), false, error.getMessage());
+        } catch (IllegalStateException exception) {
+            result = new TestResult(test.toString(), false, exception.getMessage());
         }
+
         return result;
     }