fix: team json tests
Some checks failed
Pull Request Pipeline / test (pull_request) Failing after 43s
Pull Request Pipeline / lint (pull_request) Successful in 52s

This commit is contained in:
2026-05-21 21:22:12 +02:00
parent 46134a4f5c
commit 14eb77e241
2 changed files with 24 additions and 9 deletions

View File

@@ -200,15 +200,9 @@ class DataTransferService {
.map((id) => playerById[id]) .map((id) => playerById[id])
.whereType<Player>() .whereType<Player>()
.toList(); .toList();
final team = Team.fromJson(map);
return Team( return team.copyWith(members: members);
id: map['id'] as String,
name: map['name'] as String,
createdAt: DateTime.parse(map['createdAt'] as String),
color: AppColor.values.byName(map['color'] ?? AppColor.blue),
members: members,
score: map['score'] as int?,
);
}).toList(); }).toList();
} }

View File

@@ -55,7 +55,12 @@ void main() {
members: [testPlayer1, testPlayer2], members: [testPlayer1, testPlayer2],
); );
testTeam = Team(name: 'Test Team', members: [testPlayer1, testPlayer2]); testTeam = Team(
name: 'Test Team',
color: AppColor.yellow,
score: 5,
members: [testPlayer1, testPlayer2],
);
testMatch = Match( testMatch = Match(
name: 'Test Match', name: 'Test Match',
@@ -666,6 +671,8 @@ void main() {
'name': testTeam.name, 'name': testTeam.name,
'memberIds': [testPlayer1.id], 'memberIds': [testPlayer1.id],
'createdAt': testTeam.createdAt.toIso8601String(), 'createdAt': testTeam.createdAt.toIso8601String(),
'color': testTeam.color.toString(),
'score': testTeam.score,
}, },
]; ];
@@ -679,6 +686,8 @@ void main() {
expect(teams[0].name, testTeam.name); expect(teams[0].name, testTeam.name);
expect(teams[0].members.length, 1); expect(teams[0].members.length, 1);
expect(teams[0].members[0].id, testPlayer1.id); expect(teams[0].members[0].id, testPlayer1.id);
expect(teams[0].color, testTeam.color);
expect(teams[0].score, testTeam.score);
}); });
test('parseTeamsFromJson() empty list', () { test('parseTeamsFromJson() empty list', () {
@@ -715,6 +724,9 @@ void main() {
'gameId': testGame.id, 'gameId': testGame.id,
'groupId': testGroup.id, 'groupId': testGroup.id,
'playerIds': [testPlayer1.id, testPlayer2.id], 'playerIds': [testPlayer1.id, testPlayer2.id],
'isTeamMatch': false,
'teams': null,
'scores': null,
'notes': testMatch.notes, 'notes': testMatch.notes,
'createdAt': testMatch.createdAt.toIso8601String(), 'createdAt': testMatch.createdAt.toIso8601String(),
}, },
@@ -770,6 +782,9 @@ void main() {
'name': testMatch.name, 'name': testMatch.name,
'gameId': 'non-existent-game-id', 'gameId': 'non-existent-game-id',
'playerIds': [testPlayer1.id], 'playerIds': [testPlayer1.id],
'isTeamMatch': false,
'teams': null,
'scores': null,
'notes': '', 'notes': '',
'createdAt': testMatch.createdAt.toIso8601String(), 'createdAt': testMatch.createdAt.toIso8601String(),
}, },
@@ -801,6 +816,9 @@ void main() {
'gameId': testGame.id, 'gameId': testGame.id,
'groupId': null, 'groupId': null,
'playerIds': [testPlayer1.id], 'playerIds': [testPlayer1.id],
'isTeamMatch': false,
'teams': null,
'scores': null,
'notes': '', 'notes': '',
'createdAt': testMatch.createdAt.toIso8601String(), 'createdAt': testMatch.createdAt.toIso8601String(),
}, },
@@ -831,6 +849,9 @@ void main() {
'name': testMatch.name, 'name': testMatch.name,
'gameId': testGame.id, 'gameId': testGame.id,
'playerIds': [testPlayer1.id], 'playerIds': [testPlayer1.id],
'isTeamMatch': false,
'teams': null,
'scores': null,
'notes': '', 'notes': '',
'createdAt': testMatch.createdAt.toIso8601String(), 'createdAt': testMatch.createdAt.toIso8601String(),
'endedAt': endedDate.toIso8601String(), 'endedAt': endedDate.toIso8601String(),