Skip to content

Commit 3e71c40

Browse files
committed
Fix Database Tests
1 parent cc16875 commit 3e71c40

File tree

1 file changed

+14
-21
lines changed

1 file changed

+14
-21
lines changed

src/test/kotlin/io/codemc/api/database/TestDatabase.kt

Lines changed: 14 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ class TestDatabase {
2929

3030
val u1 = getUser(n1)
3131
assertNotNull(u1)
32-
assertEquals(u1?.discord, 123456789L)
32+
assertEquals(123456789L, u1?.discord)
3333

3434
removeUser(n1)
3535

@@ -38,7 +38,7 @@ class TestDatabase {
3838

3939
val u2 = getUser(n2)
4040
assertNotNull(u2)
41-
assertEquals(u2?.discord, 987654321L)
41+
assertEquals(987654321L, u2?.discord)
4242

4343
removeUser(n2)
4444
}
@@ -50,35 +50,31 @@ class TestDatabase {
5050

5151
val u1 = getUser(name)
5252
assertNotNull(u1)
53-
assertEquals(u1?.discord, 0L)
53+
assertEquals(0L, u1?.discord)
5454

5555
updateUser(name, 123456789L)
5656

5757
val u2 = getUser(name)
5858
assertNotNull(u2)
59-
assertEquals(u2?.discord, 123456789L)
59+
assertEquals(123456789L, u2?.discord)
6060

6161
removeUser(name)
6262
}
6363

6464
@Test
6565
fun testMultipleUsers() {
6666
val users = listOf(
67-
"TestUser",
68-
"TestUser2",
67+
"TestMultiUser",
68+
"TestMultiUser2",
6969
)
7070

7171
users.forEach { s -> addUser(s, 4567L) }
7272

73-
val retrieved = getAllUsers()
74-
assertEquals(users.size, retrieved.size)
75-
for (i in retrieved.indices) {
76-
assertEquals(retrieved[i].username, users[i])
77-
assertEquals(retrieved[i].discord, 4567L)
78-
}
73+
assertEquals(4567L, getUser(users[0])?.discord)
74+
assertEquals(4567L, getUser(users[1])?.discord)
7975

80-
removeAllUsers()
81-
assertTrue(getAllUsers().isEmpty())
76+
removeUser(users[0])
77+
removeUser(users[1])
8278
}
8379

8480
@Test
@@ -102,15 +98,12 @@ class TestDatabase {
10298

10399
users.forEachIndexed { index, s -> addUser(s, index.toLong()) }
104100

105-
val retrieved = getAllUsers()
106-
assertEquals(users.size, retrieved.size)
107-
for (i in retrieved.indices) {
108-
assertEquals(retrieved[i].username, users[i])
109-
assertEquals(retrieved[i].discord, i.toLong())
101+
for ((i, user) in users.withIndex()) {
102+
assertNotNull(getUser(user))
103+
assertEquals(i.toLong(), getUser(user)?.discord)
110104
}
111105

112-
removeAllUsers()
113-
assertTrue(getAllUsers().isEmpty())
106+
users.forEach { removeUser(it) }
114107
}
115108

116109
}

0 commit comments

Comments
 (0)