Remove Identifiers in favour of instance methods

3 files changed, 14 insertions, 21 deletions

diff --git a/src/test/java/org/zwobble/couscous/tests/frontends/java/ExpressionReadingTests.java b/src/test/java/org/zwobble/couscous/tests/frontends/java/ExpressionReadingTests.java
index 585508a..e8b391e 100644
--- a/src/test/java/org/zwobble/couscous/tests/frontends/java/ExpressionReadingTests.java
+++ b/src/test/java/org/zwobble/couscous/tests/frontends/java/ExpressionReadingTests.java
@@ -2,7 +2,7 @@ package org.zwobble.couscous.tests.frontends.java;
 
 import org.junit.Test;
 import org.zwobble.couscous.ast.*;
-import org.zwobble.couscous.ast.identifiers.Identifiers;
+import org.zwobble.couscous.ast.identifiers.Identifier;
 import org.zwobble.couscous.types.ScalarType;
 import org.zwobble.couscous.types.TypeParameter;
 import org.zwobble.couscous.types.Types;
@@ -233,7 +233,7 @@ public class ExpressionReadingTests {
 
     @Test
     public void canReadMethodCallsWithZeroExpressionsAsVarargs() {
-        TypeParameter typeParameter = typeParameter(Identifiers.method(Identifiers.forType("java.util.Arrays"), "asList"), "T");
+        TypeParameter typeParameter = typeParameter(Identifier.forType("java.util.Arrays").method("asList"), "T");
         assertEquals(
             methodCall(
                 staticReceiver("java.util.Arrays"),
@@ -251,7 +251,7 @@ public class ExpressionReadingTests {
 
     @Test
     public void canReadMethodCallsWithMultipleExpressionsAsVarargs() {
-        TypeParameter typeParameter = typeParameter(Identifiers.method(Identifiers.forType("java.util.Arrays"), "asList"), "T");
+        TypeParameter typeParameter = typeParameter(Identifier.forType("java.util.Arrays").method("asList"), "T");
         assertEquals(
             methodCall(
                 staticReceiver("java.util.Arrays"),
@@ -269,7 +269,7 @@ public class ExpressionReadingTests {
 
     @Test
     public void canReadMethodCallsWithArrayAsVarargs() {
-        TypeParameter typeParameter = typeParameter(Identifiers.method(Identifiers.forType("java.util.Arrays"), "asList"), "T");
+        TypeParameter typeParameter = typeParameter(Identifier.forType("java.util.Arrays").method("asList"), "T");
         assertEquals(
             methodCall(
                 staticReceiver("java.util.Arrays"),
@@ -287,7 +287,7 @@ public class ExpressionReadingTests {
 
     @Test
     public void canReadGenericStaticMethodCallsWithExplicitTypeParameters() {
-        TypeParameter typeParameter = typeParameter(Identifiers.method(Identifiers.forType("java.util.Collections"), "emptyList"), "T");
+        TypeParameter typeParameter = typeParameter(Identifier.forType("java.util.Collections").method("emptyList"), "T");
         assertEquals(
             methodCall(
                 staticReceiver("java.util.Collections"),
@@ -305,7 +305,7 @@ public class ExpressionReadingTests {
 
     @Test
     public void canReadGenericStaticMethodCallsWithImplicitTypeParameters() {
-        TypeParameter typeParameter = typeParameter(Identifiers.method(Identifiers.forType("java.util.Collections"), "emptyList"), "T");
+        TypeParameter typeParameter = typeParameter(Identifier.forType("java.util.Collections").method("emptyList"), "T");
         assertEquals(
             methodCall(
                 staticReceiver("java.util.Collections"),
diff --git a/src/test/java/org/zwobble/couscous/tests/frontends/java/JavaReaderTests.java b/src/test/java/org/zwobble/couscous/tests/frontends/java/JavaReaderTests.java
index acda60c..c6055e7 100644
--- a/src/test/java/org/zwobble/couscous/tests/frontends/java/JavaReaderTests.java
+++ b/src/test/java/org/zwobble/couscous/tests/frontends/java/JavaReaderTests.java
@@ -3,7 +3,6 @@ package org.zwobble.couscous.tests.frontends.java;
 import org.junit.Test;
 import org.zwobble.couscous.ast.*;
 import org.zwobble.couscous.ast.identifiers.Identifier;
-import org.zwobble.couscous.ast.identifiers.Identifiers;
 import org.zwobble.couscous.types.ScalarType;
 import org.zwobble.couscous.types.Types;
 
@@ -104,8 +103,8 @@ public class JavaReaderTests {
 
         MethodNode method = classNode.getMethods().get(0);
 
-        Identifier classIdentifier = Identifiers.forType("com.example.Example");
-        Identifier expectedMethodIdentifier = Identifiers.method(classIdentifier, "identity");
+        Identifier classIdentifier = Identifier.forType("com.example.Example");
+        Identifier expectedMethodIdentifier = classIdentifier.method("identity");
         assertThat(method.getTypeParameters(), contains(isFormalTypeParameter(expectedMethodIdentifier, "T")));
         assertThat(
             method.getArguments(),
@@ -215,13 +214,7 @@ public class JavaReaderTests {
     @Test
     public void canReadTryWithResourceWithoutOtherClauses() {
         ScalarType type = ScalarType.of("java.io.ByteArrayOutputStream");
-        Identifier identifier = Identifiers.variable(
-            Identifiers.method(
-                Identifiers.type(Identifiers.TOP, "com.example.Example"),
-                "main"
-            ),
-            "closeable"
-        );
+        Identifier identifier = Identifier.TOP.type("com.example.Example").method("main").variable("closeable");
         LocalVariableDeclarationNode expectedResource = localVariableDeclaration(identifier, "closeable", type, constructorCall(type, list()));
         assertThat(
             readStatements("int", "try(java.io.ByteArrayOutputStream closeable = new java.io.ByteArrayOutputStream()) { return 1; }"),
@@ -395,7 +388,7 @@ public class JavaReaderTests {
         assertThat(classes, hasSize(1));
         ClassNode classNode = (ClassNode) classes.get(0);
         assertEquals(ScalarType.of("com.example.Example"), classNode.getName());
-        assertEquals(list(formalTypeParameter(Identifiers.forType("com.example.Example"), "T")), classNode.getTypeParameters());
+        assertEquals(list(formalTypeParameter(Identifier.forType("com.example.Example"), "T")), classNode.getTypeParameters());
     }
 
     @Test
@@ -406,7 +399,7 @@ public class JavaReaderTests {
         assertThat(classes, hasSize(1));
         InterfaceNode classNode = (InterfaceNode) classes.get(0);
         assertEquals(ScalarType.of("com.example.Example"), classNode.getName());
-        assertEquals(list(formalTypeParameter(Identifiers.forType("com.example.Example"), "T")), classNode.getTypeParameters());
+        assertEquals(list(formalTypeParameter(Identifier.forType("com.example.Example"), "T")), classNode.getTypeParameters());
     }
 
     @Test
diff --git a/src/test/java/org/zwobble/couscous/tests/frontends/java/JavaTypesTests.java b/src/test/java/org/zwobble/couscous/tests/frontends/java/JavaTypesTests.java
index 75c3594..f903eeb 100644
--- a/src/test/java/org/zwobble/couscous/tests/frontends/java/JavaTypesTests.java
+++ b/src/test/java/org/zwobble/couscous/tests/frontends/java/JavaTypesTests.java
@@ -1,7 +1,7 @@
 package org.zwobble.couscous.tests.frontends.java;
 
 import org.junit.Test;
-import org.zwobble.couscous.ast.identifiers.Identifiers;
+import org.zwobble.couscous.ast.identifiers.Identifier;
 import org.zwobble.couscous.frontends.java.JavaTypes;
 import org.zwobble.couscous.types.ScalarType;
 import org.zwobble.couscous.types.TypeParameter;
@@ -24,7 +24,7 @@ public class JavaTypesTests {
 
     @Test
     public void bindingTypeToGenericTypeParameterReturnsBoundTypeParameter() {
-        TypeParameter typeParameter = typeParameter(Identifiers.forType("List"), "T");
+        TypeParameter typeParameter = typeParameter(Identifier.forType("List"), "T");
         assertEquals(
             boundTypeParameter(typeParameter, INT_TYPE),
             JavaTypes.bind(typeParameter, INT_TYPE));
@@ -33,7 +33,7 @@ public class JavaTypesTests {
     @Test
     public void bindingTypeToParameterizedTypeBindsParameters() {
         ScalarType rawList = ScalarType.of("List");
-        TypeParameter typeParameter = typeParameter(Identifiers.forType("List"), "T");
+        TypeParameter typeParameter = typeParameter(Identifier.forType("List"), "T");
         assertEquals(
             parameterizedType(rawList, list(boundTypeParameter(typeParameter, INT_TYPE))),
             JavaTypes.bind(