Small improvements for serialization

Author: stefan-ka

src/main/java/org/contextmapper/discovery/ContextMapSerializer.java

@@ -19,7 +19,6 @@
 import org.contextmapper.discovery.cml.ContextMapToCMLConverter;
 import org.contextmapper.discovery.model.ContextMap;
 import org.contextmapper.dsl.ContextMappingDSLStandaloneSetup;
-import org.contextmapper.dsl.contextMappingDSL.ContextMappingDSLFactory;
 import org.contextmapper.dsl.contextMappingDSL.ContextMappingModel;
 import org.eclipse.emf.common.util.URI;
 import org.eclipse.emf.ecore.resource.Resource;
@@ -39,6 +38,8 @@ public class ContextMapSerializer {
     public void serializeContextMap(ContextMap contextMap, File cmlFile) throws IOException {
         if (!FilenameUtils.getExtension(cmlFile.toString()).equals("cml"))
             throw new IllegalArgumentException("The CML file must end with the file extension '*.cml'!");
+        if (contextMap.getBoundedContexts().size() <= 0)
+            throw new IllegalArgumentException("The Context Map must at least contain one Bounded Context to be serialized!");
 
         ContextMappingDSLStandaloneSetup.doSetup();
         Resource resource = new ResourceSetImpl().createResource(URI.createURI(cmlFile.toURI().toString()));

src/main/java/org/contextmapper/discovery/cml/ContextMapToCMLConverter.java

@@ -36,7 +36,9 @@ public ContextMappingModel convert(org.contextmapper.discovery.model.ContextMap
         model.setMap(contextMap);
 
         for (org.contextmapper.discovery.model.BoundedContext boundedContext : inputMap.getBoundedContexts()) {
-            model.getBoundedContexts().add(convert(boundedContext));
+            BoundedContext bc = convert(boundedContext);
+            model.getBoundedContexts().add(bc);
+            contextMap.getBoundedContexts().add(bc);
         }
 
         for (Relationship relationship : inputMap.getRelationships()) {

src/test/java/org/contextmapper/discovery/ContextMapSerializerTest.java

@@ -19,6 +19,7 @@
 import org.contextmapper.discovery.strategies.boundedcontexts.SpringBootBoundedContextDiscoveryStrategy;
 import org.contextmapper.discovery.strategies.names.SeparatorToCamelCaseBoundedContextNameMappingStrategy;
 import org.contextmapper.discovery.strategies.relationships.DockerComposeRelationshipDiscoveryStrategy;
+import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
@@ -68,4 +69,28 @@ public void canSaveDiscoveredModelAsCMLFile() throws IOException {
         assertTrue(new File(TEST_CML_FILE).exists());
     }
 
+    @Test
+    public void cannotSerializeOtherThanCMLFile() {
+        // given
+        ContextMapDiscoverer discoverer = new ContextMapDiscoverer()
+                .usingBoundedContextDiscoveryStrategies(
+                        new SpringBootBoundedContextDiscoveryStrategy("test.microservice.spring.boot"));
+
+        // when, then
+        Assertions.assertThrows(IllegalArgumentException.class, () -> {
+            new ContextMapSerializer().serializeContextMap(discoverer.discoverContextMap(), new File("test.ext"));
+        });
+    }
+
+    @Test
+    public void cannotSerializeEmptyContextMap() {
+        // given
+        ContextMap contextMap = new ContextMap();
+
+        // when, then
+        Assertions.assertThrows(IllegalArgumentException.class, () -> {
+            new ContextMapSerializer().serializeContextMap(contextMap, new File("test.cml"));
+        });
+    }
+
 }

src/test/java/org/contextmapper/discovery/strategies/boundedcontexts/SpringBootBoundedContextDiscoveryStrategyTest.java

@@ -17,7 +17,6 @@
 
 import org.contextmapper.discovery.ContextMapDiscoverer;
 import org.contextmapper.discovery.model.BoundedContext;
-import org.contextmapper.discovery.strategies.boundedcontexts.SpringBootBoundedContextDiscoveryStrategy;
 import org.junit.jupiter.api.Test;
 
 import java.util.Set;