Skip to content

Commit 7c91a2f

Browse files
committed
fix(test): Ensure different generated IDs
1 parent c3a48d6 commit 7c91a2f

2 files changed

Lines changed: 30 additions & 5 deletions

File tree

crates/game_api/tests/overview_event.rs

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,12 @@ async fn event_overview_original_maps_diff() -> anyhow::Result<()> {
4141
};
4242

4343
let map_id = rand::random_range(1..=100);
44-
let original_map_id = rand::random_range(1..=100);
44+
let original_map_id = loop {
45+
let val = rand::random_range(1..=100);
46+
if val != map_id {
47+
break val;
48+
}
49+
};
4550

4651
let maps = [map_id, original_map_id].map(|map_id| maps::ActiveModel {
4752
id: Set(map_id),

crates/game_api/tests/player_finished_event.rs

Lines changed: 24 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,12 @@ async fn event_finish_transitive_save() -> anyhow::Result<()> {
167167
};
168168

169169
let original_map_id = rand::random_range(1..=100);
170-
let map_id = rand::random_range(1..=100);
170+
let map_id = loop {
171+
let val = rand::random_range(1..=100);
172+
if val != original_map_id {
173+
break val;
174+
}
175+
};
171176

172177
let maps = [original_map_id, map_id].map(|map_id| maps::ActiveModel {
173178
id: Set(map_id),
@@ -309,7 +314,12 @@ async fn event_finish_save_to_original() -> anyhow::Result<()> {
309314
};
310315

311316
let original_map_id = rand::random_range(1..=100);
312-
let map_id = rand::random_range(1..=100);
317+
let map_id = loop {
318+
let val = rand::random_range(1..=100);
319+
if val != original_map_id {
320+
break val;
321+
}
322+
};
313323

314324
let maps = [original_map_id, map_id].map(|map_id| maps::ActiveModel {
315325
id: Set(map_id),
@@ -450,7 +460,12 @@ async fn event_finish_non_transitive_save() -> anyhow::Result<()> {
450460
};
451461

452462
let original_map_id = rand::random_range(1..=100);
453-
let map_id = rand::random_range(1..=100);
463+
let map_id = loop {
464+
let val = rand::random_range(1..=100);
465+
if val != original_map_id {
466+
break val;
467+
}
468+
};
454469

455470
let maps = [original_map_id, map_id].map(|map_id| maps::ActiveModel {
456471
id: Set(map_id),
@@ -562,7 +577,12 @@ async fn event_finish_save_non_event_record() -> anyhow::Result<()> {
562577
};
563578

564579
let original_map_id = rand::random_range(1..=100);
565-
let map_id = rand::random_range(1..=100);
580+
let map_id = loop {
581+
let val = rand::random_range(1..=100);
582+
if val != original_map_id {
583+
break val;
584+
}
585+
};
566586

567587
let maps = [original_map_id, map_id].map(|map_id| maps::ActiveModel {
568588
id: Set(map_id),

0 commit comments

Comments
 (0)