test(workflows): assert self.data preserves the raw non-mapping workflow value

Address review: the previous assertion only proved the key stayed present; it
would pass even if construction replaced the malformed value with {}. Assert
definition.data["workflow"] equals the original parsed value and is still a
non-mapping, proving the guard normalizes only the local variable.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: jawwad-ali

tests/test_workflows.py

@@ -3626,8 +3626,15 @@ def test_non_mapping_workflow_block_parses_then_validates(self, block):
         assert definition.id == ""
         errors = validate_workflow(definition)
         assert any("workflow.id" in e for e in errors)
-        # The raw value is preserved on .data for validation/inspection.
-        assert "workflow" in definition.data
+        # The RAW malformed value is preserved on .data (the guard only
+        # normalizes the local var, not self.data). Assert it was NOT replaced
+        # with {} by comparing against the original parse and confirming it is
+        # still a non-mapping.
+        import yaml
+
+        raw_workflow = yaml.safe_load(block).get("workflow")
+        assert definition.data["workflow"] == raw_workflow
+        assert not isinstance(definition.data["workflow"], dict)
 
     def test_from_string_invalid(self):
         from specify_cli.workflows.engine import WorkflowDefinition