moditect#28 wip

integration-test/src/main/resources/META-INF/deptective.json

@@ -1,20 +1,24 @@
 {
-    "packages" : [
+    "components" : [
         {
-            "name" : "com.example.ui",
+            "name" : "ui",
+            "contains" : [ "com.example.ui" ],
             "reads" : [
-                "com.example.persistence",
-                "com.example.service"
+                "persistence",
+                "service"
             ]
         },
         {
-            "name" : "com.example.persistence"
+            "name" : "persistence",
+            "contains" : [ "com.example.persistence" ]
         },
         {
-            "name" : "com.example.service"
+            "name" : "service",
+            "contains" : [ "com.example.service" ]
         },
         {
-            "name" : "com.example.rest"
+            "name" : "rest",
+            "contains" : [ "com.example.rest" ]
         }
     ],
     "whitelisted" : [

javac-plugin/src/main/java/org/moditect/deptective/internal/export/JsonSerializer.java

@@ -35,21 +35,21 @@ public class JsonSerializer implements ModelSerializer {
     private final ObjectMapper mapper;
 
     private final ObjectNode root;
-    private final ArrayNode packages;
+    private final ArrayNode components;
     private final ArrayNode whitelisted;
 
     public JsonSerializer() {
         mapper = new ObjectMapper();
         mapper.enable(SerializationFeature.INDENT_OUTPUT);
 
         root = mapper.createObjectNode();
-        packages = root.putArray("packages");
+        components = root.putArray("components");
         whitelisted = root.putArray("whitelisted");
     }
 
     @Override
     public void addComponent(Component component) {
-        packages.add(toJsonNode(component, mapper));
+        components.add(toJsonNode(component, mapper));
     }
 
     @Override
@@ -72,6 +72,14 @@ private JsonNode toJsonNode(Component component, ObjectMapper mapper) {
 
         node.put("name", component.getName());
 
+        if (!component.getContained().isEmpty()) {
+            ArrayNode reads = node.putArray("contains");
+            component.getContained()
+                    .stream()
+                    .sorted()
+                    .forEach(r -> reads.add(r.toString()));
+        }
+
         if (!component.getReads().isEmpty()) {
             ArrayNode reads = node.putArray("reads");
             component.getReads()

javac-plugin/src/main/java/org/moditect/deptective/internal/handler/PackageReferenceCollector.java

@@ -83,7 +83,7 @@ public boolean onEnteringCompilationUnit(CompilationUnitTree tree) {
 
         currentPackageName = packageNameTree.toString();
         packagesOfCurrentCompilation.add(currentPackageName);
-
+        builder.addContains(currentPackageName, PackagePattern.getPattern(currentPackageName));
         return true;
     }
 

javac-plugin/src/main/java/org/moditect/deptective/internal/model/Component.java

@@ -16,6 +16,7 @@
 package org.moditect.deptective.internal.model;
 
 import java.util.ArrayList;
+import java.util.Collection;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
@@ -57,11 +58,16 @@ public Builder addRead(String read, ReadKind readKind) {
             return this;
         }
 
-        public Builder addContains(List<PackagePattern> contains) {
+        public Builder addContains(Collection<PackagePattern> contains) {
             contained.addAll(contains);
             return this;
         }
 
+        public Builder addContains(PackagePattern contains) {
+            contained.add(contains);
+            return this;
+        }
+
         public Component build() {
             return new Component(name, contained, reads);
         }

javac-plugin/src/main/java/org/moditect/deptective/internal/model/ConfigParser.java

@@ -18,7 +18,6 @@
 import java.io.IOException;
 import java.io.InputStream;
 import java.util.ArrayList;
-import java.util.Arrays;
 import java.util.Collections;
 import java.util.Iterator;
 import java.util.List;
@@ -71,23 +70,23 @@ public PackageDependencies getPackageDependencies() {
 
     private PackageDependencies parseConfig(InputStream config) throws Exception {
         ObjectMapper objectMapper = new ObjectMapper();
-        return parsePackages(objectMapper.readTree(config));
+        return parseComponents(objectMapper.readTree(config));
     }
 
     private PackageDependencies parseConfig(String config) throws Exception {
         ObjectMapper objectMapper = new ObjectMapper();
-        return parsePackages(objectMapper.readTree(config));
+        return parseComponents(objectMapper.readTree(config));
     }
 
-    private PackageDependencies parsePackages(JsonNode config) throws IOException {
+    private PackageDependencies parseComponents(JsonNode config) throws IOException {
         PackageDependencies.Builder builder = PackageDependencies.builder();
 
-        ArrayNode packages = (ArrayNode) config.get("packages");
+        ArrayNode components = (ArrayNode) config.get("components");
 
-        if (packages != null) {
-            Iterator<JsonNode> it = packages.iterator();
+        if (components != null) {
+            Iterator<JsonNode> it = components.iterator();
             while (it.hasNext()) {
-                parsePackage(it.next(), builder);
+                parseComponent(it.next(), builder);
             }
         }
 
@@ -104,11 +103,12 @@ private PackageDependencies parsePackages(JsonNode config) throws IOException {
         return builder.build();
     }
 
-    private void parsePackage(JsonNode pakkage, Builder builder) {
-        String name = pakkage.get("name").asText();
-        List<String> reads = parseReads((ArrayNode) (pakkage.get("reads")));
+    private void parseComponent(JsonNode component, Builder builder) {
+        String name = component.get("name").asText();
+        List<PackagePattern> contains = parseContains((ArrayNode) (component.get("contains")));
+        List<String> reads = parseReads((ArrayNode) (component.get("reads")));
 
-        builder.addComponent(name, Arrays.asList(PackagePattern.getPattern(name)), reads);
+        builder.addComponent(name, contains, reads);
     }
 
     private List<String> parseReads(ArrayNode arrayNode) {
@@ -125,4 +125,19 @@ private List<String> parseReads(ArrayNode arrayNode) {
 
         return packages;
     }
+
+    private List<PackagePattern> parseContains(ArrayNode arrayNode) {
+        if (arrayNode == null) {
+            return Collections.emptyList();
+        }
+
+        Iterator<JsonNode> it = arrayNode.iterator();
+        List<PackagePattern> components = new ArrayList<>();
+
+        while (it.hasNext()) {
+            components.add(PackagePattern.getPattern(it.next().asText()));
+        }
+
+        return components;
+    }
 }

javac-plugin/src/main/java/org/moditect/deptective/internal/model/PackageDependencies.java

@@ -53,6 +53,11 @@ public void addComponent(String name, List<PackagePattern> contains, List<String
             );
         }
 
+        public void addContains(String componentName, PackagePattern contained) {
+            Component.Builder builder = componentsByName.computeIfAbsent(componentName, n -> Component.builder(n));
+            builder.addContains(contained);
+        }
+
         public void addRead(String name, String readComponent, ReadKind readKind) {
             Component.Builder builder = componentsByName.computeIfAbsent(name, n -> Component.builder(n));
             builder.addRead(readComponent, readKind);

javac-plugin/src/test/java/org/moditect/deptective/config/ConfigParsingTest.java

@@ -30,19 +30,25 @@ public void shouldLoadConfig() throws Exception {
         PackageDependencies dependencies = new ConfigParser(
                 lines(
                         "{",
-                        "    \"packages\" : [",
+                        "    \"components\" : [",
                         "        {",
-                        "            \"name\" : \"com.example.ui\",",
+                        "            \"name\" : \"ui\",",
+                        "            \"contains\" : [ \"com.example.ui\" ],",
                         "            \"reads\" : [",
-                        "                \"com.example.service\",",
-                        "                \"com.example.persistence\"",
+                        "                \"service\",",
+                        "                \"persistence\"",
                         "            ]",
                         "        },",
                         "        {",
-                        "            \"name\" : \"com.example.service\",",
+                        "            \"name\" : \"service\",",
+                        "            \"contains\" : [ \"com.example.service\" ],",
                         "            \"reads\" : [",
-                        "                \"com.example.persistence\"",
+                        "                \"persistence\"",
                         "            ]",
+                        "        },",
+                        "        {",
+                        "            \"name\" : \"persistence\",",
+                        "            \"contains\" : [ \"com.example.persistence\" ]",
                         "        }",
                         "    ],",
                         "    \"whitelisted\" : [",
@@ -56,14 +62,17 @@ public void shouldLoadConfig() throws Exception {
 
         Component ui = dependencies.getComponentByPackage("com.example.ui");
         assertThat(ui).isNotNull();
-        assertThat(ui.allowedToRead(component("com.example.service"))).isTrue();
-        assertThat(ui.allowedToRead(component("com.example.persistence"))).isTrue();
+        assertThat(ui.getName()).isEqualTo("ui");
+        assertThat(ui.containsPackage("com.example.ui")).isTrue();
+        assertThat(ui.allowedToRead(component("service"))).isTrue();
+        assertThat(ui.allowedToRead(component("persistence"))).isTrue();
 
         Component service = dependencies.getComponentByPackage("com.example.service");
         assertThat(service).isNotNull();
-        assertThat(service).isNotNull();
-        assertThat(service.allowedToRead(component("com.example.ui"))).isFalse();
-        assertThat(service.allowedToRead(component("com.example.persistence"))).isTrue();
+        assertThat(service.getName()).isEqualTo("service");
+        assertThat(service.containsPackage("com.example.service")).isTrue();
+        assertThat(service.allowedToRead(component("ui"))).isFalse();
+        assertThat(service.allowedToRead(component("persistence"))).isTrue();
 
         assertThat(dependencies.isWhitelisted("java.awt")).isTrue();
         assertThat(dependencies.isWhitelisted("java.awt.color")).isTrue();

javac-plugin/src/test/java/org/moditect/deptective/plugintest/analyze/AnalyzeTest.java

@@ -61,17 +61,21 @@ public void shouldGenerateConfig() throws Exception {
 
         String expectedConfig = lines(
                 "{",
-                "    \"packages\" : [ {",
+                "    \"components\" : [ {",
                 "      \"name\" : \"org.moditect.deptective.plugintest.analyze.bar\",",
+                "      \"contains\" : [ \"org.moditect.deptective.plugintest.analyze.bar\" ],",
                 "      \"reads\" : [ \"org.moditect.deptective.plugintest.analyze.qux\" ]",
                 "    }, {",
                 "      \"name\" : \"org.moditect.deptective.plugintest.analyze.foo\",",
+                "      \"contains\" : [ \"org.moditect.deptective.plugintest.analyze.foo\" ],",
                 "      \"reads\" : [",
                 "          \"org.moditect.deptective.plugintest.analyze.qux\",",
-                "          \"org.moditect.deptective.plugintest.analyze.bar\" ]\n",
-                "    } ],",
-                "    \"whitelisted\" : [ \"java.math\" ]",
-                "  }]"
+                "          \"org.moditect.deptective.plugintest.analyze.bar\" ]",
+                "    }, {",
+                "      \"name\" : \"org.moditect.deptective.plugintest.analyze.qux\",",
+                "      \"contains\" : [ \"org.moditect.deptective.plugintest.analyze.qux\" ]",
+                "    } ]",
+                "}"
         );
 
         Optional<JavaFileObject> deptectiveFile = compilation

javac-plugin/src/test/java/org/moditect/deptective/plugintest/analyzewhitelistallexternal/AnalyzeWhitelistAllExternalTest.java

@@ -59,10 +59,12 @@ public void shouldGenerateConfig() throws Exception {
 
         String expectedConfig = lines(
                 "{",
-                "    \"packages\" : [ {",
-                "      \"name\" : \"org.moditect.deptective.plugintest.analyzewhitelistallexternal.bar\"",
+                "    \"components\" : [ {",
+                "      \"name\" : \"org.moditect.deptective.plugintest.analyzewhitelistallexternal.bar\",",
+                "      \"contains\" : [ \"org.moditect.deptective.plugintest.analyzewhitelistallexternal.bar\" ]",
                 "    }, {",
                 "      \"name\" : \"org.moditect.deptective.plugintest.analyzewhitelistallexternal.foo\",",
+                "      \"contains\" : [ \"org.moditect.deptective.plugintest.analyzewhitelistallexternal.foo\" ],",
                 "      \"reads\" : [ \"org.moditect.deptective.plugintest.analyzewhitelistallexternal.bar\" ]",
                 "    } ],",
                 "    \"whitelisted\" : [ \"java.io\", \"java.math\", \"java.net\" ]",

javac-plugin/src/test/resources/org/moditect/deptective/plugintest/basic/deptective.json

@@ -1,28 +1,36 @@
 {
-    "packages" : [
+    "components" : [
         {
-            "name" : "org.moditect.deptective.plugintest.basic.barctorcall"
+            "name" : "org.moditect.deptective.plugintest.basic.barctorcall",
+            "contains" : [ "org.moditect.deptective.plugintest.basic.barctorcall" ]
         },
         {
-            "name" : "org.moditect.deptective.plugintest.basic.barfield"
+            "name" : "org.moditect.deptective.plugintest.basic.barfield",
+            "contains" : [ "org.moditect.deptective.plugintest.basic.barfield" ]
         },
         {
-            "name" : "org.moditect.deptective.plugintest.basic.barlocalvar"
+            "name" : "org.moditect.deptective.plugintest.basic.barlocalvar",
+            "contains" : [ "org.moditect.deptective.plugintest.basic.barlocalvar" ]
         },
         {
-            "name" : "org.moditect.deptective.plugintest.basic.barloopvar"
+            "name" : "org.moditect.deptective.plugintest.basic.barloopvar",
+            "contains" : [ "org.moditect.deptective.plugintest.basic.barloopvar" ]
         },
         {
-            "name" : "org.moditect.deptective.plugintest.basic.barparameter"
+            "name" : "org.moditect.deptective.plugintest.basic.barparameter",
+            "contains" : [ "org.moditect.deptective.plugintest.basic.barparameter" ]
         },
         {
-            "name" : "org.moditect.deptective.plugintest.basic.barretval"
+            "name" : "org.moditect.deptective.plugintest.basic.barretval",
+            "contains" : [ "org.moditect.deptective.plugintest.basic.barretval" ]
         },
         {
-            "name" : "org.moditect.deptective.plugintest.basic.bartypearg"
+            "name" : "org.moditect.deptective.plugintest.basic.bartypearg",
+            "contains" : [ "org.moditect.deptective.plugintest.basic.bartypearg" ]
         },
         {
             "name" : "org.moditect.deptective.plugintest.basic.foo",
+            "contains" : [ "org.moditect.deptective.plugintest.basic.foo" ],
             "reads" : [
                 "java.util"
             ]

javac-plugin/src/test/resources/org/moditect/deptective/plugintest/javac/deptective.json

@@ -1,5 +1,5 @@
 {
-    "packages" : [
+    "components" : [
         {
             "name" : "com.example.foo"
         }

javac-plugin/src/test/resources/org/moditect/deptective/plugintest/unconfiguredpackage/deptective.json

@@ -1,4 +1,4 @@
 {
-    "packages" : [
+    "components" : [
     ]
 }

javac-plugin/src/test/resources/org/moditect/deptective/plugintest/visualize/deptective.json

@@ -1,13 +1,15 @@
 {
-    "packages" : [
+    "components" : [
         {
             "name" : "org.moditect.deptective.plugintest.visualize.bar",
+            "contains" : [ "org.moditect.deptective.plugintest.visualize.bar" ],
             "reads" : [
                 "org.moditect.deptective.plugintest.visualize.qux"
             ]
         },
         {
             "name" : "org.moditect.deptective.plugintest.visualize.foo",
+            "contains" : [ "org.moditect.deptective.plugintest.visualize.foo" ],
             "reads" : [
                 "org.moditect.deptective.plugintest.visualize.qux"
             ]

javac-plugin/src/test/resources/org/moditect/deptective/plugintest/whitelist/deptective.json

@@ -1,10 +1,12 @@
 {
-    "packages" : [
+    "components" : [
         {
-            "name" : "org.moditect.deptective.plugintest.whitelist.foo"
+            "name" : "org.moditect.deptective.plugintest.whitelist.foo",
+            "contains" : [ "org.moditect.deptective.plugintest.whitelist.foo" ]
         },
         {
-            "name" : "org.moditect.deptective.plugintest.whitelist.bar"
+            "name" : "org.moditect.deptective.plugintest.whitelist.bar",
+            "contains" : [ "org.moditect.deptective.plugintest.whitelist.bar" ]
         }
     ],
     "whitelisted" : [