From 5588eb372687118c09c38a4e01ea2e25073b7587 Mon Sep 17 00:00:00 2001 From: albertache1998 Date: Mon, 8 Jan 2024 13:24:11 +0100 Subject: [PATCH 1/4] add unit tests --- .../kotlin/listOfLists/ListOfListsUnitTest.kt | 53 +++++++++++++++++++ 1 file changed, 53 insertions(+) create mode 100644 core-kotlin-modules/core-kotlin-collections-list-2/src/test/kotlin/listOfLists/ListOfListsUnitTest.kt diff --git a/core-kotlin-modules/core-kotlin-collections-list-2/src/test/kotlin/listOfLists/ListOfListsUnitTest.kt b/core-kotlin-modules/core-kotlin-collections-list-2/src/test/kotlin/listOfLists/ListOfListsUnitTest.kt new file mode 100644 index 000000000..b57dd0d0d --- /dev/null +++ b/core-kotlin-modules/core-kotlin-collections-list-2/src/test/kotlin/listOfLists/ListOfListsUnitTest.kt @@ -0,0 +1,53 @@ +package listOfLists + +import org.junit.jupiter.api.Assertions.assertEquals +import org.junit.jupiter.api.Assertions.assertTrue +import org.junit.jupiter.api.Test + +class ListOfListsUnitTest { + + @Test + fun `test creating a list of lists using listOf()`() { + val listOfLists = listOf(listOf(1, 2, 3), listOf(4, 5, 6), listOf(7, 8, 9)) + assertEquals(3, listOfLists.size) + assertEquals(listOf(1, 2, 3), listOfLists[0]) + assertEquals(listOf(4, 5, 6), listOfLists[1]) + assertEquals(listOf(7, 8, 9), listOfLists[2]) + } + + @Test + fun `test creating a list of lists using List constructor`() { + val listOfLists = List(3) { MutableList(3) {0} } + + assertEquals(3, listOfLists.size) + assertEquals(listOf(0, 0, 0), listOfLists[0]) + assertEquals(listOf(0, 0, 0), listOfLists[1]) + assertEquals(listOf(0, 0, 0), listOfLists[2]) + } + + @Test + fun `test creating a list of lists using map method`() { +// val listOfMutableLists = (0..2).map { mutableListOf() } + val listOfLists = (0..2).map { _ -> (0..2).map { 0 } } + + assertEquals(3, listOfLists.size) + assertEquals(listOf(0, 0, 0), listOfLists[0]) + assertEquals(listOf(0, 0, 0), listOfLists[1]) + assertEquals(listOf(0, 0, 0), listOfLists[2]) + } + + @Test + fun `test creating a list of mutablelists using map method`() { + val listOfMutableLists = (0..2).map { mutableListOf() } + + assertEquals(3, listOfMutableLists.size) + assertTrue(listOfMutableLists.all { it.isEmpty() }) + } + + @Test + fun `test creating a list of lists using Array() and fill()`() { + val listOfLists = Array(3) { Array(5) { 0 } } + assertEquals(3, listOfLists.size) + assertTrue(listOfLists.all { it.all { it == 0 } }) + } +} \ No newline at end of file From 234d1b6a9db3e88f6e89534b71da3e738b976ed3 Mon Sep 17 00:00:00 2001 From: albertache1998 Date: Mon, 8 Jan 2024 13:34:04 +0100 Subject: [PATCH 2/4] added unit tests --- .../src/test/kotlin/listOfLists/ListOfListsUnitTest.kt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/core-kotlin-modules/core-kotlin-collections-list-2/src/test/kotlin/listOfLists/ListOfListsUnitTest.kt b/core-kotlin-modules/core-kotlin-collections-list-2/src/test/kotlin/listOfLists/ListOfListsUnitTest.kt index b57dd0d0d..a4dd9947e 100644 --- a/core-kotlin-modules/core-kotlin-collections-list-2/src/test/kotlin/listOfLists/ListOfListsUnitTest.kt +++ b/core-kotlin-modules/core-kotlin-collections-list-2/src/test/kotlin/listOfLists/ListOfListsUnitTest.kt @@ -1,14 +1,15 @@ package listOfLists import org.junit.jupiter.api.Assertions.assertEquals -import org.junit.jupiter.api.Assertions.assertTrue import org.junit.jupiter.api.Test +import kotlin.test.assertTrue class ListOfListsUnitTest { @Test fun `test creating a list of lists using listOf()`() { val listOfLists = listOf(listOf(1, 2, 3), listOf(4, 5, 6), listOf(7, 8, 9)) + assertEquals(3, listOfLists.size) assertEquals(listOf(1, 2, 3), listOfLists[0]) assertEquals(listOf(4, 5, 6), listOfLists[1]) @@ -27,7 +28,6 @@ class ListOfListsUnitTest { @Test fun `test creating a list of lists using map method`() { -// val listOfMutableLists = (0..2).map { mutableListOf() } val listOfLists = (0..2).map { _ -> (0..2).map { 0 } } assertEquals(3, listOfLists.size) From 694e9dbbe8b2e0f5cfae7da04a439bcbf7d7fbee Mon Sep 17 00:00:00 2001 From: albertache1998 Date: Thu, 11 Jan 2024 00:02:09 +0100 Subject: [PATCH 3/4] code cleanup --- .../kotlin/listOfLists/ListOfListsUnitTest.kt | 15 ++++----------- 1 file changed, 4 insertions(+), 11 deletions(-) diff --git a/core-kotlin-modules/core-kotlin-collections-list-2/src/test/kotlin/listOfLists/ListOfListsUnitTest.kt b/core-kotlin-modules/core-kotlin-collections-list-2/src/test/kotlin/listOfLists/ListOfListsUnitTest.kt index a4dd9947e..84e835e88 100644 --- a/core-kotlin-modules/core-kotlin-collections-list-2/src/test/kotlin/listOfLists/ListOfListsUnitTest.kt +++ b/core-kotlin-modules/core-kotlin-collections-list-2/src/test/kotlin/listOfLists/ListOfListsUnitTest.kt @@ -7,7 +7,7 @@ import kotlin.test.assertTrue class ListOfListsUnitTest { @Test - fun `test creating a list of lists using listOf()`() { + fun `Creates an immutable list of immutable lists using listOf()`() { val listOfLists = listOf(listOf(1, 2, 3), listOf(4, 5, 6), listOf(7, 8, 9)) assertEquals(3, listOfLists.size) @@ -17,7 +17,7 @@ class ListOfListsUnitTest { } @Test - fun `test creating a list of lists using List constructor`() { + fun `Creates an immutable list of mutable lists using List constructor`() { val listOfLists = List(3) { MutableList(3) {0} } assertEquals(3, listOfLists.size) @@ -27,7 +27,7 @@ class ListOfListsUnitTest { } @Test - fun `test creating a list of lists using map method`() { + fun `Creates an immutable list of immutable lists using map method`() { val listOfLists = (0..2).map { _ -> (0..2).map { 0 } } assertEquals(3, listOfLists.size) @@ -37,17 +37,10 @@ class ListOfListsUnitTest { } @Test - fun `test creating a list of mutablelists using map method`() { + fun `Creates an immutable list of mutable lists using map method`() { val listOfMutableLists = (0..2).map { mutableListOf() } assertEquals(3, listOfMutableLists.size) assertTrue(listOfMutableLists.all { it.isEmpty() }) } - - @Test - fun `test creating a list of lists using Array() and fill()`() { - val listOfLists = Array(3) { Array(5) { 0 } } - assertEquals(3, listOfLists.size) - assertTrue(listOfLists.all { it.all { it == 0 } }) - } } \ No newline at end of file From a2e7059cf7c7cb0d771d211fd823e7b831bb902c Mon Sep 17 00:00:00 2001 From: albertache1998 Date: Fri, 12 Jan 2024 02:51:44 +0100 Subject: [PATCH 4/4] code cleanup --- .../kotlin/listOfLists/ListOfListsUnitTest.kt | 24 +++++++++---------- 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/core-kotlin-modules/core-kotlin-collections-list-2/src/test/kotlin/listOfLists/ListOfListsUnitTest.kt b/core-kotlin-modules/core-kotlin-collections-list-2/src/test/kotlin/listOfLists/ListOfListsUnitTest.kt index 84e835e88..3a816bad9 100644 --- a/core-kotlin-modules/core-kotlin-collections-list-2/src/test/kotlin/listOfLists/ListOfListsUnitTest.kt +++ b/core-kotlin-modules/core-kotlin-collections-list-2/src/test/kotlin/listOfLists/ListOfListsUnitTest.kt @@ -8,32 +8,32 @@ class ListOfListsUnitTest { @Test fun `Creates an immutable list of immutable lists using listOf()`() { - val listOfLists = listOf(listOf(1, 2, 3), listOf(4, 5, 6), listOf(7, 8, 9)) + val listOfLists = listOf(listOf(1, 2), listOf(3, 4), listOf(5, 6)) assertEquals(3, listOfLists.size) - assertEquals(listOf(1, 2, 3), listOfLists[0]) - assertEquals(listOf(4, 5, 6), listOfLists[1]) - assertEquals(listOf(7, 8, 9), listOfLists[2]) + assertEquals(listOf(1, 2), listOfLists[0]) + assertEquals(listOf(3, 4), listOfLists[1]) + assertEquals(listOf(5, 6), listOfLists[2]) } @Test fun `Creates an immutable list of mutable lists using List constructor`() { - val listOfLists = List(3) { MutableList(3) {0} } + val listOfLists = List(3) { MutableList(2) {0} } assertEquals(3, listOfLists.size) - assertEquals(listOf(0, 0, 0), listOfLists[0]) - assertEquals(listOf(0, 0, 0), listOfLists[1]) - assertEquals(listOf(0, 0, 0), listOfLists[2]) + assertEquals(listOf(0, 0), listOfLists[0]) + assertEquals(listOf(0, 0), listOfLists[1]) + assertEquals(listOf(0, 0), listOfLists[2]) } @Test fun `Creates an immutable list of immutable lists using map method`() { - val listOfLists = (0..2).map { _ -> (0..2).map { 0 } } + val listOfLists = (0..2).map { _ -> (0..1).map { 0 } } assertEquals(3, listOfLists.size) - assertEquals(listOf(0, 0, 0), listOfLists[0]) - assertEquals(listOf(0, 0, 0), listOfLists[1]) - assertEquals(listOf(0, 0, 0), listOfLists[2]) + assertEquals(listOf(0, 0), listOfLists[0]) + assertEquals(listOf(0, 0), listOfLists[1]) + assertEquals(listOf(0, 0), listOfLists[2]) } @Test