From 4b7ceea3c6741c7e9be74e40126003ba8ba39f82 Mon Sep 17 00:00:00 2001 From: anchouls Date: Tue, 19 Dec 2023 14:29:22 +0100 Subject: [PATCH 1/3] Added java tests --- .../academy/test/system/core/MethodUtils.kt | 4 +- .../system/core/models/method/TestMethod.kt | 24 ++- .../core/models/variable/TestVariable.kt | 31 +++- .../test/system/core/TestJavaClass.java | 154 ++++++++++++++++++ .../core/testData/java/CustomJavaClass.java | 4 + .../system/core/testData/java/JavaClass.java | 46 ++++++ .../test/system/core/JavaClassTests.kt | 14 ++ .../core/testData/java/TestJavaClass.kt | 129 +++++++++++++++ .../test/system/core/testData/sam/Sam.kt | 2 +- 9 files changed, 395 insertions(+), 13 deletions(-) create mode 100644 core/src/test/java/org/jetbrains/academy/test/system/core/TestJavaClass.java create mode 100644 core/src/test/java/org/jetbrains/academy/test/system/core/testData/java/CustomJavaClass.java create mode 100644 core/src/test/java/org/jetbrains/academy/test/system/core/testData/java/JavaClass.java create mode 100644 core/src/test/kotlin/org/jetbrains/academy/test/system/core/JavaClassTests.kt create mode 100644 core/src/test/kotlin/org/jetbrains/academy/test/system/core/testData/java/TestJavaClass.kt diff --git a/core/src/main/kotlin/org/jetbrains/academy/test/system/core/MethodUtils.kt b/core/src/main/kotlin/org/jetbrains/academy/test/system/core/MethodUtils.kt index bc7b532..d411a5b 100644 --- a/core/src/main/kotlin/org/jetbrains/academy/test/system/core/MethodUtils.kt +++ b/core/src/main/kotlin/org/jetbrains/academy/test/system/core/MethodUtils.kt @@ -39,9 +39,9 @@ fun Array.findMethod(method: TestMethod, customErrorMessage: String? = n it.name == method.name } } - val returnTypeJava = (method.returnTypeJava?.let { listOf(it) } ?: listOfNotNull(method.returnType.type, *method.returnType.possibleBounds.toTypedArray())).map { it.lowercase() } + val returnTypeJava = (listOf(method.returnTypeJava)).map { it.lowercase() } val filteredByType = - filteredByName.filterByCondition(customErrorMessage ?: "The method ${method.name} should have the return type ${method.returnType.getTypePrettyString()}") { + filteredByName.filterByCondition(customErrorMessage ?: "The method ${method.name} should have the return type ${method.returnType?.getTypePrettyString()}") { it.returnType.name.getShortName().lowercase() in returnTypeJava } val filteredByArgumentsCount = diff --git a/core/src/main/kotlin/org/jetbrains/academy/test/system/core/models/method/TestMethod.kt b/core/src/main/kotlin/org/jetbrains/academy/test/system/core/models/method/TestMethod.kt index 6d4d474..45c49ce 100644 --- a/core/src/main/kotlin/org/jetbrains/academy/test/system/core/models/method/TestMethod.kt +++ b/core/src/main/kotlin/org/jetbrains/academy/test/system/core/models/method/TestMethod.kt @@ -20,12 +20,28 @@ import kotlin.reflect.jvm.kotlinFunction */ data class TestMethod( val name: String, - val returnType: TestKotlinType, + val returnTypeJava: String, + val returnType: TestKotlinType? = null, val arguments: List = emptyList(), - val returnTypeJava: String? = null, val visibility: Visibility = Visibility.PUBLIC, val hasGeneratedPartInName: Boolean = false, ) { + constructor( + name: String, + returnTypeJava: String, + arguments: List, + visibility: Visibility + ) : this( + name = name, + returnTypeJava = returnTypeJava, + returnType = null, + arguments = arguments, + visibility = visibility, + hasGeneratedPartInName = false + ) + + private fun getTypePrettyString() = returnType?.getTypePrettyString() ?: returnTypeJava + fun prettyString(withToDo: Boolean = true): String { val args = arguments.joinToString(", ") { it.paramPrettyString() } val body = if (withToDo) { @@ -33,7 +49,7 @@ data class TestMethod( } else { "// Some code" } - return "${visibility.key} fun $name($args): ${returnType.getTypePrettyString()} = $body" + return "${visibility.key} fun $name($args): ${getTypePrettyString()} = $body" } private fun TestVariable.paramPrettyString() = "$name: $javaType" @@ -48,6 +64,6 @@ data class TestMethod( this.visibility.key, "\"The visibility of the method $name must be ${this.visibility.key}\"" ) - kotlinFunction.returnType.checkType(returnType, returnTypeJava ?: returnType.type, "the function $name") + kotlinFunction.returnType.checkType(returnType, returnTypeJava, "the function $name") } } diff --git a/core/src/main/kotlin/org/jetbrains/academy/test/system/core/models/variable/TestVariable.kt b/core/src/main/kotlin/org/jetbrains/academy/test/system/core/models/variable/TestVariable.kt index 867da47..8192d5a 100644 --- a/core/src/main/kotlin/org/jetbrains/academy/test/system/core/models/variable/TestVariable.kt +++ b/core/src/main/kotlin/org/jetbrains/academy/test/system/core/models/variable/TestVariable.kt @@ -39,6 +39,29 @@ data class TestVariable( val isConst: Boolean = false, // TODO: add nullability? ) { + constructor( + name: String, + javaType: String, + value: String?, + visibility: Visibility?, + isFinal: Boolean, + isInPrimaryConstructor: Boolean, + isStatic: Boolean, + isConst: Boolean + ) : this( + name = name, + javaType = javaType, + value = value, + kotlinType = null, + visibility = visibility, + mutability = if (isFinal) VariableMutability.VAL else VariableMutability.VAR, + isInPrimaryConstructor = isInPrimaryConstructor, + isStatic = isStatic, + isConst = isConst + ) + + constructor(name: String, javaType: String) : this(name, javaType, null, null, null, null, false, false, false) + private fun getTypePrettyString() = kotlinType?.getTypePrettyString() ?: javaType fun prettyString(): String { @@ -69,10 +92,7 @@ data class TestVariable( } } field.kotlinProperty?.returnType?.checkType( - kotlinType, - javaType, - "the field $name", - false + kotlinType, javaType, "the field $name", false ) } } @@ -86,8 +106,7 @@ fun TestVariable.isVariableExist(fileContent: String): Boolean { val defWithType = variableDefTemplateWithType() if (!(baseDef in fileContent || defWithType in fileContent)) { error( - "The code should contains a definition of the ${this.name} variable! " + - "Please, add <$baseDef> or <$defWithType> code in your solution." + "The code should contains a definition of the ${this.name} variable! " + "Please, add <$baseDef> or <$defWithType> code in your solution." ) } return true diff --git a/core/src/test/java/org/jetbrains/academy/test/system/core/TestJavaClass.java b/core/src/test/java/org/jetbrains/academy/test/system/core/TestJavaClass.java new file mode 100644 index 0000000..f2a303b --- /dev/null +++ b/core/src/test/java/org/jetbrains/academy/test/system/core/TestJavaClass.java @@ -0,0 +1,154 @@ +package org.jetbrains.academy.test.system.core; + +import org.jetbrains.academy.test.system.core.models.Visibility; +import org.jetbrains.academy.test.system.core.models.classes.ClassType; +import org.jetbrains.academy.test.system.core.models.classes.TestClass; +import org.jetbrains.academy.test.system.core.models.method.TestMethod; +import org.jetbrains.academy.test.system.core.models.variable.TestVariable; +import org.junit.jupiter.api.BeforeAll; +import org.junit.jupiter.api.Test; + +import java.util.List; + +import static java.util.Collections.emptyList; + +public class TestJavaClass { + private static TestClass javaClassTestClass; + + @BeforeAll + static void beforeAll() { + javaClassTestClass = new TestClass( + "JavaClass", + "org.jetbrains.academy.test.system.core.testData.java", + Visibility.PUBLIC, + ClassType.CLASS, + List.of( + new TestVariable( + "PUBLIC_CONSTANT", + "int", + "1", + Visibility.PUBLIC, + true, + false, + false, + true + ), + new TestVariable( + "PRIVATE_CONSTANT", + "int", + "2", + Visibility.PRIVATE, + true, + false, + false, + true + ), + new TestVariable( + "publicStaticVar", + "int", + "3", + Visibility.PUBLIC, + false, + false, + true, + false + ), + new TestVariable( + "privateStaticVar", + "int", + "4", + Visibility.PRIVATE, + false, + false, + true, + false + ), + new TestVariable( + "publicVar", + "String", + "publicVar", + Visibility.PUBLIC, + false, + false, + false, + false + ), + new TestVariable( + "privateVar", + "String", + "privateVar", + Visibility.PRIVATE, + false, + false, + false, + false + ), + new TestVariable( + "customClass", + "CustomJavaClass", + null, + Visibility.PUBLIC, + false, + false, + false, + false + ), + new TestVariable( + "primaryConstructorVar", + "String", + null, + Visibility.PUBLIC, + false, + true, + false, + false + ) + ), + List.of( + new TestMethod( + "publicMethod", + "void", + emptyList(), + Visibility.PUBLIC + ), + new TestMethod( + "privateMethod", + "void", + emptyList(), + Visibility.PRIVATE + ), + new TestMethod( + "calculateSum", + "double", + List.of( + new TestVariable("a", "double"), + new TestVariable("b", "double") + ), + Visibility.PUBLIC + ), + new TestMethod( + "getString", + "String", + List.of(new TestVariable("string", "String")), + Visibility.PRIVATE + ), + new TestMethod( + "processList", + "List", + List.of(new TestVariable("list", "List")), + Visibility.PUBLIC + ) + ), + false, + emptyList(), + emptyList() + ); + } + + @Test + public void javaClassTest() { + Class clazz = javaClassTestClass.checkBaseDefinition(); + javaClassTestClass.checkFieldsDefinition(clazz, true); + javaClassTestClass.checkDeclaredMethods(clazz); + } +} diff --git a/core/src/test/java/org/jetbrains/academy/test/system/core/testData/java/CustomJavaClass.java b/core/src/test/java/org/jetbrains/academy/test/system/core/testData/java/CustomJavaClass.java new file mode 100644 index 0000000..d5a843f --- /dev/null +++ b/core/src/test/java/org/jetbrains/academy/test/system/core/testData/java/CustomJavaClass.java @@ -0,0 +1,4 @@ +package org.jetbrains.academy.test.system.core.testData.java; + +public class CustomJavaClass { +} diff --git a/core/src/test/java/org/jetbrains/academy/test/system/core/testData/java/JavaClass.java b/core/src/test/java/org/jetbrains/academy/test/system/core/testData/java/JavaClass.java new file mode 100644 index 0000000..3767c75 --- /dev/null +++ b/core/src/test/java/org/jetbrains/academy/test/system/core/testData/java/JavaClass.java @@ -0,0 +1,46 @@ +package org.jetbrains.academy.test.system.core.testData.java; + +import java.util.List; + +public class JavaClass { + + public static final int PUBLIC_CONSTANT = 1; + private static final int PRIVATE_CONSTANT = 2; + + public static int publicStaticVar = 3; + private static int privateStaticVar = 4; + + public String publicVar = "publicVar"; + private String privateVar = "privateVar"; + + public CustomJavaClass customClass = null; + + public String primaryConstructorVar; + + public JavaClass(String primaryVar) { + this.primaryConstructorVar = primaryVar; + } + + public void publicMethod() { + System.out.println("Public instance method called!"); + } + + private void privateMethod() { + System.out.println("Private instance method called!"); + } + + public double calculateSum(double a, double b) { + return a + b; + } + + private String getString(String string) { + return string; + } + + public List processList(List list) { + for (String item : list) { + System.out.println("Processing item: " + item); + } + return List.of("item1", "item2", "item3"); + } +} diff --git a/core/src/test/kotlin/org/jetbrains/academy/test/system/core/JavaClassTests.kt b/core/src/test/kotlin/org/jetbrains/academy/test/system/core/JavaClassTests.kt new file mode 100644 index 0000000..ace1a44 --- /dev/null +++ b/core/src/test/kotlin/org/jetbrains/academy/test/system/core/JavaClassTests.kt @@ -0,0 +1,14 @@ +package org.jetbrains.academy.test.system.core + +import org.jetbrains.academy.test.system.core.testData.java.javaClassTestClass +import org.junit.jupiter.api.Test + +class JavaClassTests { + + @Test + fun javaClassTest() { + val clazz = javaClassTestClass.checkBaseDefinition() + javaClassTestClass.checkFieldsDefinition(clazz) + javaClassTestClass.checkDeclaredMethods(clazz) + } +} diff --git a/core/src/test/kotlin/org/jetbrains/academy/test/system/core/testData/java/TestJavaClass.kt b/core/src/test/kotlin/org/jetbrains/academy/test/system/core/testData/java/TestJavaClass.kt new file mode 100644 index 0000000..5e458d1 --- /dev/null +++ b/core/src/test/kotlin/org/jetbrains/academy/test/system/core/testData/java/TestJavaClass.kt @@ -0,0 +1,129 @@ +package org.jetbrains.academy.test.system.core.testData.java + +import org.jetbrains.academy.test.system.core.models.Visibility +import org.jetbrains.academy.test.system.core.models.classes.TestClass +import org.jetbrains.academy.test.system.core.models.method.TestMethod +import org.jetbrains.academy.test.system.core.models.variable.TestVariable +import org.jetbrains.academy.test.system.core.models.variable.VariableMutability + +val javaClassTestClass = TestClass( + "JavaClass", + "org.jetbrains.academy.test.system.core.testData.java", + declaredFields = listOf( + TestVariable( + name = "PUBLIC_CONSTANT", + javaType = "int", + value = "1", + visibility = Visibility.PUBLIC, + mutability = VariableMutability.VAL, + isInPrimaryConstructor = false, + isConst = true + ), + TestVariable( + name = "PRIVATE_CONSTANT", + javaType = "int", + value = "2", + visibility = Visibility.PRIVATE, + mutability = VariableMutability.VAL, + isInPrimaryConstructor = false, + isConst = true + ), + TestVariable( + name = "publicStaticVar", + javaType = "int", + value = "3", + visibility = Visibility.PUBLIC, + mutability = VariableMutability.VAR, + isInPrimaryConstructor = false, + isStatic = true + ), + TestVariable( + name = "privateStaticVar", + javaType = "int", + value = "4", + visibility = Visibility.PRIVATE, + mutability = VariableMutability.VAR, + isInPrimaryConstructor = false, + isStatic = true + ), + TestVariable( + name = "publicVar", + javaType = "String", + value = "publicVar", + visibility = Visibility.PUBLIC, + mutability = VariableMutability.VAR, + isInPrimaryConstructor = false, + ), + TestVariable( + name = "privateVar", + javaType = "String", + value = "privateVar", + visibility = Visibility.PRIVATE, + mutability = VariableMutability.VAR, + isInPrimaryConstructor = false, + ), + TestVariable( + name = "customClass", + javaType = "CustomJavaClass", + visibility = Visibility.PUBLIC, + mutability = VariableMutability.VAR, + isInPrimaryConstructor = false, + ), + TestVariable( + name = "primaryConstructorVar", + javaType = "String", + visibility = Visibility.PUBLIC, + mutability = VariableMutability.VAR, + isInPrimaryConstructor = true, + ), + ), + customMethods = listOf( + TestMethod( + name = "publicMethod", + returnTypeJava = "void", + visibility = Visibility.PUBLIC + ), + TestMethod( + name = "privateMethod", + returnTypeJava = "void", + visibility = Visibility.PRIVATE + ), + TestMethod( + name = "calculateSum", + returnTypeJava = "double", + arguments = listOf( + TestVariable( + name = "a", + javaType = "double", + ), + TestVariable( + name = "b", + javaType = "double", + ), + ), + visibility = Visibility.PUBLIC + ), + TestMethod( + name = "getString", + returnTypeJava = "String", + arguments = listOf( + TestVariable( + name = "string", + javaType = "String", + ), + ), + visibility = Visibility.PRIVATE + ), + TestMethod( + name = "processList", + returnTypeJava = "List", + arguments = listOf( + TestVariable( + name = "list", + javaType = "List" + ), + ), + visibility = Visibility.PUBLIC + ), + ) +) diff --git a/core/src/test/kotlin/org/jetbrains/academy/test/system/core/testData/sam/Sam.kt b/core/src/test/kotlin/org/jetbrains/academy/test/system/core/testData/sam/Sam.kt index cc8d164..d8c9b21 100644 --- a/core/src/test/kotlin/org/jetbrains/academy/test/system/core/testData/sam/Sam.kt +++ b/core/src/test/kotlin/org/jetbrains/academy/test/system/core/testData/sam/Sam.kt @@ -16,7 +16,7 @@ val mySamInterfaceTestClass = TestClass( customMethods = listOf( TestMethod( "samMethod", - TestKotlinType( + returnType = TestKotlinType( "List", params = listOf("java.lang.string") ), From 2e3e3dd65104e0d8cc2d2c8da67984091c9881e4 Mon Sep 17 00:00:00 2001 From: anchouls Date: Wed, 20 Dec 2023 13:37:41 +0100 Subject: [PATCH 2/3] Fixed TestMethod --- build.gradle.kts | 2 +- .../academy/test/system/core/MethodUtils.kt | 4 ++-- .../test/system/core/models/method/TestMethod.kt | 14 ++++++-------- .../system/core/testData/java/TestJavaClass.kt | 2 ++ 4 files changed, 11 insertions(+), 11 deletions(-) diff --git a/build.gradle.kts b/build.gradle.kts index 51e935e..03ca216 100644 --- a/build.gradle.kts +++ b/build.gradle.kts @@ -8,7 +8,7 @@ plugins { } group = "org.jetbrains.academy.test.system" -version = "2.1.1" +version = "2.1.2" allprojects { apply { diff --git a/core/src/main/kotlin/org/jetbrains/academy/test/system/core/MethodUtils.kt b/core/src/main/kotlin/org/jetbrains/academy/test/system/core/MethodUtils.kt index d411a5b..bc7b532 100644 --- a/core/src/main/kotlin/org/jetbrains/academy/test/system/core/MethodUtils.kt +++ b/core/src/main/kotlin/org/jetbrains/academy/test/system/core/MethodUtils.kt @@ -39,9 +39,9 @@ fun Array.findMethod(method: TestMethod, customErrorMessage: String? = n it.name == method.name } } - val returnTypeJava = (listOf(method.returnTypeJava)).map { it.lowercase() } + val returnTypeJava = (method.returnTypeJava?.let { listOf(it) } ?: listOfNotNull(method.returnType.type, *method.returnType.possibleBounds.toTypedArray())).map { it.lowercase() } val filteredByType = - filteredByName.filterByCondition(customErrorMessage ?: "The method ${method.name} should have the return type ${method.returnType?.getTypePrettyString()}") { + filteredByName.filterByCondition(customErrorMessage ?: "The method ${method.name} should have the return type ${method.returnType.getTypePrettyString()}") { it.returnType.name.getShortName().lowercase() in returnTypeJava } val filteredByArgumentsCount = diff --git a/core/src/main/kotlin/org/jetbrains/academy/test/system/core/models/method/TestMethod.kt b/core/src/main/kotlin/org/jetbrains/academy/test/system/core/models/method/TestMethod.kt index 45c49ce..34a7f2b 100644 --- a/core/src/main/kotlin/org/jetbrains/academy/test/system/core/models/method/TestMethod.kt +++ b/core/src/main/kotlin/org/jetbrains/academy/test/system/core/models/method/TestMethod.kt @@ -20,9 +20,9 @@ import kotlin.reflect.jvm.kotlinFunction */ data class TestMethod( val name: String, - val returnTypeJava: String, - val returnType: TestKotlinType? = null, + val returnType: TestKotlinType, val arguments: List = emptyList(), + val returnTypeJava: String? = null, val visibility: Visibility = Visibility.PUBLIC, val hasGeneratedPartInName: Boolean = false, ) { @@ -33,15 +33,13 @@ data class TestMethod( visibility: Visibility ) : this( name = name, - returnTypeJava = returnTypeJava, - returnType = null, + returnType = TestKotlinType(returnTypeJava), arguments = arguments, + returnTypeJava = returnTypeJava, visibility = visibility, hasGeneratedPartInName = false ) - private fun getTypePrettyString() = returnType?.getTypePrettyString() ?: returnTypeJava - fun prettyString(withToDo: Boolean = true): String { val args = arguments.joinToString(", ") { it.paramPrettyString() } val body = if (withToDo) { @@ -49,7 +47,7 @@ data class TestMethod( } else { "// Some code" } - return "${visibility.key} fun $name($args): ${getTypePrettyString()} = $body" + return "${visibility.key} fun $name($args): ${returnType.getTypePrettyString()} = $body" } private fun TestVariable.paramPrettyString() = "$name: $javaType" @@ -64,6 +62,6 @@ data class TestMethod( this.visibility.key, "\"The visibility of the method $name must be ${this.visibility.key}\"" ) - kotlinFunction.returnType.checkType(returnType, returnTypeJava, "the function $name") + kotlinFunction.returnType.checkType(returnType, returnTypeJava ?: returnType.type, "the function $name") } } diff --git a/core/src/test/kotlin/org/jetbrains/academy/test/system/core/testData/java/TestJavaClass.kt b/core/src/test/kotlin/org/jetbrains/academy/test/system/core/testData/java/TestJavaClass.kt index 5e458d1..0d4fc7d 100644 --- a/core/src/test/kotlin/org/jetbrains/academy/test/system/core/testData/java/TestJavaClass.kt +++ b/core/src/test/kotlin/org/jetbrains/academy/test/system/core/testData/java/TestJavaClass.kt @@ -81,11 +81,13 @@ val javaClassTestClass = TestClass( TestMethod( name = "publicMethod", returnTypeJava = "void", + arguments = emptyList(), visibility = Visibility.PUBLIC ), TestMethod( name = "privateMethod", returnTypeJava = "void", + arguments = emptyList(), visibility = Visibility.PRIVATE ), TestMethod( From 7bd4c848c1129632e0e4f59c29826654ab4a9fe6 Mon Sep 17 00:00:00 2001 From: anchouls Date: Wed, 20 Dec 2023 14:51:23 +0100 Subject: [PATCH 3/3] Updated isConst argument --- .../core/models/variable/TestVariable.kt | 5 ++-- .../test/system/core/TestJavaClass.java | 12 ++------- .../core/testData/java/TestJavaClass.kt | 26 ++++++++++++------- 3 files changed, 20 insertions(+), 23 deletions(-) diff --git a/core/src/main/kotlin/org/jetbrains/academy/test/system/core/models/variable/TestVariable.kt b/core/src/main/kotlin/org/jetbrains/academy/test/system/core/models/variable/TestVariable.kt index 8192d5a..c3aaaa7 100644 --- a/core/src/main/kotlin/org/jetbrains/academy/test/system/core/models/variable/TestVariable.kt +++ b/core/src/main/kotlin/org/jetbrains/academy/test/system/core/models/variable/TestVariable.kt @@ -46,8 +46,7 @@ data class TestVariable( visibility: Visibility?, isFinal: Boolean, isInPrimaryConstructor: Boolean, - isStatic: Boolean, - isConst: Boolean + isStatic: Boolean ) : this( name = name, javaType = javaType, @@ -57,7 +56,7 @@ data class TestVariable( mutability = if (isFinal) VariableMutability.VAL else VariableMutability.VAR, isInPrimaryConstructor = isInPrimaryConstructor, isStatic = isStatic, - isConst = isConst + isConst = isStatic && isFinal ) constructor(name: String, javaType: String) : this(name, javaType, null, null, null, null, false, false, false) diff --git a/core/src/test/java/org/jetbrains/academy/test/system/core/TestJavaClass.java b/core/src/test/java/org/jetbrains/academy/test/system/core/TestJavaClass.java index f2a303b..2bc7e60 100644 --- a/core/src/test/java/org/jetbrains/academy/test/system/core/TestJavaClass.java +++ b/core/src/test/java/org/jetbrains/academy/test/system/core/TestJavaClass.java @@ -30,7 +30,6 @@ static void beforeAll() { Visibility.PUBLIC, true, false, - false, true ), new TestVariable( @@ -40,7 +39,6 @@ static void beforeAll() { Visibility.PRIVATE, true, false, - false, true ), new TestVariable( @@ -50,8 +48,7 @@ static void beforeAll() { Visibility.PUBLIC, false, false, - true, - false + true ), new TestVariable( "privateStaticVar", @@ -60,8 +57,7 @@ static void beforeAll() { Visibility.PRIVATE, false, false, - true, - false + true ), new TestVariable( "publicVar", @@ -70,7 +66,6 @@ static void beforeAll() { Visibility.PUBLIC, false, false, - false, false ), new TestVariable( @@ -80,7 +75,6 @@ static void beforeAll() { Visibility.PRIVATE, false, false, - false, false ), new TestVariable( @@ -90,7 +84,6 @@ static void beforeAll() { Visibility.PUBLIC, false, false, - false, false ), new TestVariable( @@ -100,7 +93,6 @@ static void beforeAll() { Visibility.PUBLIC, false, true, - false, false ) ), diff --git a/core/src/test/kotlin/org/jetbrains/academy/test/system/core/testData/java/TestJavaClass.kt b/core/src/test/kotlin/org/jetbrains/academy/test/system/core/testData/java/TestJavaClass.kt index 0d4fc7d..38cb1d2 100644 --- a/core/src/test/kotlin/org/jetbrains/academy/test/system/core/testData/java/TestJavaClass.kt +++ b/core/src/test/kotlin/org/jetbrains/academy/test/system/core/testData/java/TestJavaClass.kt @@ -15,25 +15,25 @@ val javaClassTestClass = TestClass( javaType = "int", value = "1", visibility = Visibility.PUBLIC, - mutability = VariableMutability.VAL, + isFinal = true, isInPrimaryConstructor = false, - isConst = true + isStatic = true ), TestVariable( name = "PRIVATE_CONSTANT", javaType = "int", value = "2", visibility = Visibility.PRIVATE, - mutability = VariableMutability.VAL, + isFinal = true, isInPrimaryConstructor = false, - isConst = true + isStatic = true ), TestVariable( name = "publicStaticVar", javaType = "int", value = "3", visibility = Visibility.PUBLIC, - mutability = VariableMutability.VAR, + isFinal = false, isInPrimaryConstructor = false, isStatic = true ), @@ -42,7 +42,7 @@ val javaClassTestClass = TestClass( javaType = "int", value = "4", visibility = Visibility.PRIVATE, - mutability = VariableMutability.VAR, + isFinal = false, isInPrimaryConstructor = false, isStatic = true ), @@ -51,30 +51,36 @@ val javaClassTestClass = TestClass( javaType = "String", value = "publicVar", visibility = Visibility.PUBLIC, - mutability = VariableMutability.VAR, + isFinal = false, isInPrimaryConstructor = false, + isStatic = false ), TestVariable( name = "privateVar", javaType = "String", value = "privateVar", visibility = Visibility.PRIVATE, - mutability = VariableMutability.VAR, + isFinal = false, isInPrimaryConstructor = false, + isStatic = false ), TestVariable( name = "customClass", javaType = "CustomJavaClass", + value = null, visibility = Visibility.PUBLIC, - mutability = VariableMutability.VAR, + isFinal = false, isInPrimaryConstructor = false, + isStatic = false ), TestVariable( name = "primaryConstructorVar", javaType = "String", + value = null, visibility = Visibility.PUBLIC, - mutability = VariableMutability.VAR, + isFinal = false, isInPrimaryConstructor = true, + isStatic = false ), ), customMethods = listOf(