From 5ecfef0aa0a237d6010aa8e73294dd0a792a52bb Mon Sep 17 00:00:00 2001 From: Remco Smits Date: Sun, 19 Jan 2025 14:36:02 +0100 Subject: [PATCH] Fix failing test because of race condition By sorting the breakpoints based on the line number, the order does not matter. --- crates/collab/src/tests/debug_panel_tests.rs | 20 +++++++++++--------- 1 file changed, 11 insertions(+), 9 deletions(-) diff --git a/crates/collab/src/tests/debug_panel_tests.rs b/crates/collab/src/tests/debug_panel_tests.rs index f47cf9b546..04f4a38d3e 100644 --- a/crates/collab/src/tests/debug_panel_tests.rs +++ b/crates/collab/src/tests/debug_panel_tests.rs @@ -1040,16 +1040,10 @@ async fn test_updated_breakpoints_send_to_dap( let called_set_breakpoints = called_set_breakpoints.clone(); move |_, args| { assert_eq!("/a/test.txt", args.source.path.unwrap()); + let mut breakpoints = args.breakpoints.unwrap(); + breakpoints.sort_by_key(|b| b.line); assert_eq!( vec![ - SourceBreakpoint { - line: 3, - column: None, - condition: None, - hit_condition: None, - log_message: None, - mode: None - }, SourceBreakpoint { line: 2, column: None, @@ -1057,9 +1051,17 @@ async fn test_updated_breakpoints_send_to_dap( hit_condition: None, log_message: None, mode: None + }, + SourceBreakpoint { + line: 3, + column: None, + condition: None, + hit_condition: None, + log_message: None, + mode: None } ], - args.breakpoints.unwrap() + breakpoints ); assert!(!args.source_modified.unwrap());