liquid-development/game-tracker
3
0
Fork 0
Code Issues 14 Pull Requests 6 Actions Releases 1 Activity

Timestamp zu allen Objekten hinzufügen #36

Merged
flixcoo merged 10 commits from feature/29-timestamp-zu-allen-objekten-hinzufügen into development 2025-11-19 19:23:32 +00:00
Conversation 5 Commits 10 Files Changed 14 +79 -92
3 changed files with 79 additions and 92 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit c76e193b4d - Show all commits

7
test/db_tests/game_test.dart
View File

@@ -51,7 +51,6 @@ void main() {
group('game tests', () { group('game tests', () {
test('game is added correctly', () async { test('game is added correctly', () async {
await withClock(fakeClock, () async {
await database.gameDao.addGame(game: testgame); await database.gameDao.addGame(game: testgame);
final result = await database.gameDao.getGameById(gameId: testgame.id); final result = await database.gameDao.getGameById(gameId: testgame.id);
@@ -77,16 +76,12 @@ void main() {
for (int i = 0; i < testgame.players!.length; i++) { for (int i = 0; i < testgame.players!.length; i++) {
expect(result.players![i].id, testgame.players![i].id); expect(result.players![i].id, testgame.players![i].id);
expect(result.players![i].name, testgame.players![i].name); expect(result.players![i].name, testgame.players![i].name);
expect( expect(result.players![i].createdAt, testgame.players![i].createdAt);
result.players![i].createdAt,
testgame.players![i].createdAt,
);
} }
} else { } else {
fail('Players is null'); fail('Players is null');
} }
}); });
});
test('game is deleted correctly', () async { test('game is deleted correctly', () async {
await database.gameDao.addGame(game: testgame); await database.gameDao.addGame(game: testgame);

19
test/db_tests/group_test.dart
View File

@@ -13,6 +13,7 @@ void main() {
late Player player3; late Player player3;
late Player player4; late Player player4;
late Group testgroup; late Group testgroup;
late Group testgroup2;
final fixedDate = DateTime(2025, 19, 11, 00, 11, 23); final fixedDate = DateTime(2025, 19, 11, 00, 11, 23);
final fakeClock = Clock(() => fixedDate); final fakeClock = Clock(() => fixedDate);
@@ -34,6 +35,11 @@ void main() {
name: 'Test Group', name: 'Test Group',
members: [player1, player2, player3], members: [player1, player2, player3],
); );
testgroup2 = Group(
id: 'gr2',
name: 'Second Group',
members: [player2, player3, player4],
);
}); });
}); });
tearDown(() async { tearDown(() async {
@@ -41,12 +47,6 @@ void main() {
}); });
group('group tests', () { group('group tests', () {
test('all groups get fetched correctly', () async { test('all groups get fetched correctly', () async {
await withClock(fakeClock, () async {
final testgroup2 = Group(
id: 'gr2',
name: 'Second Group',
members: [player2, player3, player4],
);
await database.groupDao.addGroup(group: testgroup); await database.groupDao.addGroup(group: testgroup);
await database.groupDao.addGroup(group: testgroup2); await database.groupDao.addGroup(group: testgroup2);
@@ -59,18 +59,14 @@ void main() {
expect(fetchedGroup1.members.elementAt(0).id, player1.id); expect(fetchedGroup1.members.elementAt(0).id, player1.id);
expect(fetchedGroup1.members.elementAt(0).createdAt, player1.createdAt); expect(fetchedGroup1.members.elementAt(0).createdAt, player1.createdAt);
final fetchedGroup2 = allGroups.firstWhere( final fetchedGroup2 = allGroups.firstWhere((g) => g.id == testgroup2.id);
(g) => g.id == testgroup2.id,
);
expect(fetchedGroup2.name, testgroup2.name); expect(fetchedGroup2.name, testgroup2.name);
expect(fetchedGroup2.members.length, testgroup2.members.length); expect(fetchedGroup2.members.length, testgroup2.members.length);
expect(fetchedGroup2.members.elementAt(0).id, player2.id); expect(fetchedGroup2.members.elementAt(0).id, player2.id);
expect(fetchedGroup2.members.elementAt(0).createdAt, player2.createdAt); expect(fetchedGroup2.members.elementAt(0).createdAt, player2.createdAt);
}); });
});
test('group and group members gets added correctly', () async { test('group and group members gets added correctly', () async {
await withClock(fakeClock, () async {
await database.groupDao.addGroup(group: testgroup); await database.groupDao.addGroup(group: testgroup);
final result = await database.groupDao.getGroupById( final result = await database.groupDao.getGroupById(
@@ -88,7 +84,6 @@ void main() {
expect(result.members[i].createdAt, testgroup.members[i].createdAt); expect(result.members[i].createdAt, testgroup.members[i].createdAt);
} }
}); });
});
test('group gets deleted correctly', () async { test('group gets deleted correctly', () async {
await database.groupDao.addGroup(group: testgroup); await database.groupDao.addGroup(group: testgroup);

7
test/db_tests/player_test.dart
View File

@@ -8,6 +8,7 @@ import 'package:game_tracker/data/dto/player.dart';
void main() { void main() {
late AppDatabase database; late AppDatabase database;
late Player testPlayer; late Player testPlayer;
late Player testPlayer2;
final fixedDate = DateTime(2025, 19, 11, 00, 11, 23); final fixedDate = DateTime(2025, 19, 11, 00, 11, 23);
final fakeClock = Clock(() => fixedDate); final fakeClock = Clock(() => fixedDate);
@@ -22,6 +23,7 @@ void main() {
withClock(fakeClock, () { withClock(fakeClock, () {
testPlayer = Player(name: 'Test Player'); testPlayer = Player(name: 'Test Player');
testPlayer2 = Player(name: 'Second Group');
}); });
}); });
tearDown(() async { tearDown(() async {
@@ -30,8 +32,6 @@ void main() {
group('player tests', () { group('player tests', () {
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 23:45:24 +00:00
Outdated
Review
Copy Link

Kannst du anstatt die fake clock verwenden nicht einfach überprüfen ob die timestamp des players = timestamp des db players ist?

Kannst du anstatt die fake clock verwenden nicht einfach überprüfen ob die timestamp des players = timestamp des db players ist?
sneeex commented 2025-11-18 23:46:34 +00:00
Outdated
Review
Copy Link

Nein ist ungenau, hab's probiert, funkt net

Nein ist ungenau, hab's probiert, funkt net
sneeex commented 2025-11-18 23:46:59 +00:00
Outdated
Review
Copy Link

Gibt dann abweichungen von Millisekunden, vermutlich weil die DB nicht so genau speichern kann wie das datetime object, weil die DB intern nämlich timestamps speichert und der ja nicht so genau ist.

["Drift supports two approaches of storing DateTime values in SQL:
As unix timestamp (the default): In this mode, drift stores date time values as an SQL INTEGER containing the unix timestamp (in seconds).]

Gibt dann abweichungen von Millisekunden, vermutlich weil die DB nicht so genau speichern kann wie das datetime object, weil die DB intern nämlich timestamps speichert und der ja nicht so genau ist. _["Drift supports two approaches of storing DateTime values in SQL: As unix timestamp (the default): In this mode, drift stores date time values as an SQL INTEGER containing the unix timestamp (in seconds).]_
flixcoo commented 2025-11-18 23:52:54 +00:00
Outdated
Review
Copy Link

Und wenn du ein Datum übergibst und danach checkst? Wir müssen ja nicht Testen das Datetime funktioniert sondern nur das der datentransfer geht

Und wenn du ein Datum übergibst und danach checkst? Wir müssen ja nicht Testen das Datetime funktioniert sondern nur das der datentransfer geht
sneeex commented 2025-11-18 23:55:25 +00:00
Outdated
Review
Copy Link

Ich finde das so schön und ist wohl auch Common practice, spricht doch nichts dagegen oder? Ich will ja eben auch wissen, ob der das Datum mit Zeit in der richtigen Genauigkeit speichert und nicht nur ob er es überhaupt tut. Sollte es mal durch ein Update oder so nicht mehr korrekt sein, ist es ja wichtig zu wissen, ob der datetime der in der DB gespeichert ist dem wahren entspricht. Außerdem will ich das ja so realitätsgetreu wie möglich halten und das datetime auch von der Klasse selbst erzeugen lassen, auch wenn ich es hijacke und nicht selber bei den Playern, groups und Games setzen.

Ich finde das so schön und ist wohl auch Common practice, spricht doch nichts dagegen oder? Ich will ja eben auch wissen, ob der das Datum mit Zeit in der richtigen Genauigkeit speichert und nicht nur ob er es überhaupt tut. Sollte es mal durch ein Update oder so nicht mehr korrekt sein, ist es ja wichtig zu wissen, ob der datetime der in der DB gespeichert ist dem wahren entspricht. Außerdem will ich das ja so realitätsgetreu wie möglich halten und das datetime auch von der Klasse selbst erzeugen lassen, auch wenn ich es hijacke und nicht selber bei den Playern, groups und Games setzen.
flixcoo commented 2025-11-19 08:20:46 +00:00
Outdated
Review
Copy Link

Also was mich halt bisschen genervt hat, das man diverse Sachen jetzt in await withClock(fakeClock, () async {}); wrappen muss. Kannst du es nicht sonst so machen, dass alle Objekte die genutzt werden in der setUp(() {}); Methode erstellt, damit man die fake clock nicht mehr in den einzelnen Tests brauch?

Also was mich halt bisschen genervt hat, das man diverse Sachen jetzt in `await withClock(fakeClock, () async {});` wrappen muss. Kannst du es nicht sonst so machen, dass alle Objekte die genutzt werden in der `setUp(() {});` Methode erstellt, damit man die fake clock nicht mehr in den einzelnen Tests brauch?
sneeex commented 2025-11-19 08:22:11 +00:00
Outdated
Review
Copy Link

Also was mich halt bisschen genervt hat, das man diverse Sachen jetzt in await withClock(fakeClock, () async {}); wrappen muss. Kannst du es nicht sonst so machen, dass alle Objekte die genutzt werden in der setUp(() {}); Methode erstellt, damit man die fake clock nicht mehr in den einzelnen Tests brauch?

Bestimmt muss mal schauen

> Also was mich halt bisschen genervt hat, das man diverse Sachen jetzt in `await withClock(fakeClock, () async {});` wrappen muss. Kannst du es nicht sonst so machen, dass alle Objekte die genutzt werden in der `setUp(() {});` Methode erstellt, damit man die fake clock nicht mehr in den einzelnen Tests brauch? Bestimmt muss mal schauen
test('all players get fetched correctly', () async { test('all players get fetched correctly', () async {
await withClock(fakeClock, () async {
final testPlayer2 = Player(name: 'Second Group');
await database.playerDao.addPlayer(player: testPlayer); await database.playerDao.addPlayer(player: testPlayer);
await database.playerDao.addPlayer(player: testPlayer2); await database.playerDao.addPlayer(player: testPlayer2);
@@ -50,10 +50,8 @@ void main() {
expect(fetchedPlayer2.name, testPlayer2.name); expect(fetchedPlayer2.name, testPlayer2.name);
expect(fetchedPlayer2.createdAt, testPlayer2.createdAt); expect(fetchedPlayer2.createdAt, testPlayer2.createdAt);
}); });
});
test('players get inserted correcly ', () async { test('players get inserted correcly ', () async {
await withClock(fakeClock, () async {
await database.playerDao.addPlayer(player: testPlayer); await database.playerDao.addPlayer(player: testPlayer);
final result = await database.playerDao.getPlayerById( final result = await database.playerDao.getPlayerById(
playerId: testPlayer.id, playerId: testPlayer.id,
@@ -63,7 +61,6 @@ void main() {
expect(result.name, testPlayer.name); expect(result.name, testPlayer.name);
expect(result.createdAt, testPlayer.createdAt); expect(result.createdAt, testPlayer.createdAt);
}); });
});
test('players get deleted correcly ', () async { test('players get deleted correcly ', () async {
await database.playerDao.addPlayer(player: testPlayer); await database.playerDao.addPlayer(player: testPlayer);