structurizr-inspection: Fixes model.deploymentnode.technology (it was checking the description property rather than technology).

Author: simonbrowndotje

changelog.md

@@ -15,6 +15,7 @@
 - structurizr-dsl: Adds support for a `jump` property on relationship styles.
 - structurizr-inspection: Adds a way to disable inspections via a workspace property named `structurizr.inspection` (`false` to disable).
 - structurizr-inspection: Default inspector adds a summary of error/warning/info/ignore counts as workspace properties.
+- structurizr-inspection: Fixes `model.deploymentnode.technology` (it was checking the description property rather than technology).
 
 ## v4.1.0 (28th May 2025)
 

structurizr-inspection/src/main/java/com/structurizr/inspection/model/DeploymentNodeTechnologyInspection.java

@@ -13,7 +13,7 @@ public DeploymentNodeTechnologyInspection(Inspector inspector) {
 
     @Override
     protected Violation inspect(DeploymentNode deploymentNode) {
-        if (StringUtils.isNullOrEmpty(deploymentNode.getDescription())) {
+        if (StringUtils.isNullOrEmpty(deploymentNode.getTechnology())) {
             return violation("The " + terminologyFor(deploymentNode).toLowerCase() + " \"" + nameOf(deploymentNode) + "\" is missing a technology.");
         }
 

structurizr-inspection/src/test/java/com/structurizr/inspection/model/DeploymentNodeTechnologyInspectionTests.java

@@ -17,7 +17,7 @@
     @Test
     public void run_WithoutDescription() {
         Workspace workspace = new Workspace("Name", "Description");
-        DeploymentNode deploymentNode = workspace.getModel().addDeploymentNode("Name");
+        DeploymentNode deploymentNode = workspace.getModel().addDeploymentNode("Name", "Description", "");
 
         Violation violation = new DeploymentNodeTechnologyInspection(new DefaultInspector(workspace)).run(deploymentNode);
         Assertions.assertEquals(Severity.ERROR, violation.getSeverity());