Skip to content

Commit c217db5

Browse files
committed
Fix test compilation errors by adding frequency field
All test files needed the new `frequency: None,` field added to their Args struct initializations after the field was added to support the --frequency command-line option. Fixed files: - tests/bidirected_inversion_test.rs - tests/bidirected_tests.rs - tests/integration_tests.rs - tests/mathematical_bidirected_tests.rs - tests/test_complex_structural_variations.rs - tests/test_edge_traversal.rs - tests/test_inversion_detection.rs - tests/test_programmatic_variations.rs - tests/test_real_inversion.rs - tests/test_topological_sort.rs All tests now compile and run successfully.
1 parent d3a3559 commit c217db5

10 files changed

+49
-0
lines changed

tests/bidirected_inversion_test.rs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@ fn test_simple_inversion() {
5555
sgd_sort: false,
5656
groom: false,
5757
aligner: "allwave".to_string(),
58+
frequency: None,
5859
};
5960

6061
run_seqrush_bidirected_simple(args).unwrap();
@@ -142,6 +143,7 @@ fn test_complex_rearrangement() {
142143
sgd_sort: false,
143144
groom: false,
144145
aligner: "allwave".to_string(),
146+
frequency: None,
145147
};
146148

147149
run_seqrush_bidirected_simple(args).unwrap();
@@ -237,6 +239,7 @@ fn test_self_inverse_palindrome() {
237239
sgd_sort: false,
238240
groom: false,
239241
aligner: "allwave".to_string(),
242+
frequency: None,
240243
};
241244

242245
run_seqrush_bidirected_simple(args).unwrap();

tests/bidirected_tests.rs

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -128,6 +128,7 @@ fn test_reverse_complement_alignment() {
128128
sgd_sort: false,
129129
groom: false,
130130
aligner: "allwave".to_string(),
131+
frequency: None,
131132
};
132133

133134
run_seqrush_bidirected_simple(args).unwrap();
@@ -186,6 +187,7 @@ fn test_palindromic_sequence() {
186187
sgd_sort: false,
187188
groom: false,
188189
aligner: "allwave".to_string(),
190+
frequency: None,
189191
};
190192

191193
run_seqrush_bidirected_simple(args).unwrap();
@@ -238,6 +240,7 @@ fn test_inversion_between_sequences() {
238240
sgd_sort: false,
239241
groom: false,
240242
aligner: "allwave".to_string(),
243+
frequency: None,
241244
};
242245

243246
run_seqrush_bidirected_simple(args).unwrap();
@@ -300,6 +303,7 @@ fn test_complex_rearrangement() {
300303
sgd_sort: false,
301304
groom: false,
302305
aligner: "allwave".to_string(),
306+
frequency: None,
303307
};
304308

305309
run_seqrush_bidirected_simple(args).unwrap();
@@ -350,6 +354,7 @@ fn test_self_alignment_with_repeats() {
350354
sgd_sort: false,
351355
groom: false,
352356
aligner: "allwave".to_string(),
357+
frequency: None,
353358
};
354359

355360
run_seqrush_bidirected_simple(args).unwrap();
@@ -437,6 +442,7 @@ fn test_multiple_sequence_alignment() {
437442
sgd_sort: false,
438443
groom: false,
439444
aligner: "allwave".to_string(),
445+
frequency: None,
440446
};
441447

442448
run_seqrush_bidirected_simple(args).unwrap();

tests/integration_tests.rs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,6 +66,7 @@ fn test_performance_scaling() {
6666
sgd_sort: false,
6767
groom: false,
6868
aligner: "allwave".to_string(),
69+
frequency: None,
6970
};
7071

7172
let start = Instant::now();
@@ -166,6 +167,7 @@ fn test_real_world_example() {
166167
sgd_sort: false,
167168
groom: false,
168169
aligner: "allwave".to_string(),
170+
frequency: None,
169171
};
170172

171173
run_seqrush(args).unwrap();
@@ -219,6 +221,7 @@ fn test_command_line_interface() {
219221
sgd_sort: false,
220222
groom: false,
221223
aligner: "allwave".to_string(),
224+
frequency: None,
222225
};
223226

224227
run_seqrush(args).unwrap();
@@ -284,6 +287,7 @@ fn test_parallel_consistency() {
284287
sgd_sort: false,
285288
groom: false,
286289
aligner: "allwave".to_string(),
290+
frequency: None,
287291
};
288292

289293
run_seqrush(args).unwrap();

tests/mathematical_bidirected_tests.rs

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -105,6 +105,7 @@ fn test_base_case_empty() {
105105
sgd_sort: false,
106106
groom: false,
107107
aligner: "allwave".to_string(),
108+
frequency: None,
108109
};
109110

110111
run_seqrush_bidirected_simple(args).unwrap();
@@ -168,6 +169,7 @@ fn test_base_case_single_base() {
168169
sgd_sort: false,
169170
groom: false,
170171
aligner: "allwave".to_string(),
172+
frequency: None,
171173
};
172174

173175
run_seqrush_bidirected_simple(args).unwrap();
@@ -278,6 +280,7 @@ fn test_identical_sequences_share_nodes() {
278280
sgd_sort: false,
279281
groom: false,
280282
aligner: "allwave".to_string(),
283+
frequency: None,
281284
};
282285

283286
run_seqrush_bidirected_simple(args).unwrap();
@@ -342,6 +345,7 @@ fn test_inductive_sequence_extension() {
342345
sgd_sort: false,
343346
groom: false,
344347
aligner: "allwave".to_string(),
348+
frequency: None,
345349
};
346350

347351
run_seqrush_bidirected_simple(args).unwrap();
@@ -450,6 +454,7 @@ fn test_path_reconstruction_correctness() {
450454
sgd_sort: false,
451455
groom: false,
452456
aligner: "allwave".to_string(),
457+
frequency: None,
453458
};
454459

455460
run_seqrush_bidirected_simple(args).unwrap();
@@ -519,6 +524,7 @@ fn test_edge_connectivity() {
519524
sgd_sort: false,
520525
groom: false,
521526
aligner: "allwave".to_string(),
527+
frequency: None,
522528
};
523529

524530
run_seqrush_bidirected_simple(args).unwrap();
@@ -587,6 +593,7 @@ fn test_palindromic_sequences() {
587593
sgd_sort: false,
588594
groom: false,
589595
aligner: "allwave".to_string(),
596+
frequency: None,
590597
};
591598

592599
run_seqrush_bidirected_simple(args).unwrap();
@@ -642,6 +649,7 @@ fn test_union_find_with_matches() {
642649
sgd_sort: false,
643650
groom: false,
644651
aligner: "allwave".to_string(),
652+
frequency: None,
645653
};
646654

647655
run_seqrush_bidirected_simple(args).unwrap();
@@ -709,6 +717,7 @@ fn test_large_sequences() {
709717
sgd_sort: false,
710718
groom: false,
711719
aligner: "allwave".to_string(),
720+
frequency: None,
712721
};
713722

714723
run_seqrush_bidirected_simple(args).unwrap();
@@ -765,6 +774,7 @@ fn test_transitivity_of_matches() {
765774
sgd_sort: false,
766775
groom: false,
767776
aligner: "allwave".to_string(),
777+
frequency: None,
768778
};
769779

770780
run_seqrush_bidirected_simple(args).unwrap();

tests/test_complex_structural_variations.rs

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -181,6 +181,7 @@ fn test_complex_cnv_repeats() {
181181
sgd_sort: false,
182182
groom: false,
183183
aligner: "allwave".to_string(),
184+
frequency: None,
184185
};
185186

186187
run_inversion_aware_seqrush(args).unwrap();
@@ -247,6 +248,7 @@ fn test_y_shaped_breakpoint() {
247248
sgd_sort: false,
248249
groom: false,
249250
aligner: "allwave".to_string(),
251+
frequency: None,
250252
};
251253

252254
run_inversion_aware_seqrush(args).unwrap();
@@ -307,6 +309,7 @@ fn test_nested_inversions() {
307309
sgd_sort: false,
308310
groom: false,
309311
aligner: "allwave".to_string(),
312+
frequency: None,
310313
};
311314

312315
run_inversion_aware_seqrush(args).unwrap();
@@ -378,6 +381,7 @@ fn test_random_rearrangements() {
378381
sgd_sort: false,
379382
groom: false,
380383
aligner: "allwave".to_string(),
384+
frequency: None,
381385
};
382386

383387
run_inversion_aware_seqrush(args).unwrap();
@@ -450,6 +454,7 @@ fn test_tandem_repeat_polymorphism() {
450454
sgd_sort: false,
451455
groom: false,
452456
aligner: "allwave".to_string(),
457+
frequency: None,
453458
};
454459

455460
run_inversion_aware_seqrush(args).unwrap();
@@ -517,6 +522,7 @@ fn test_complex_nested_variations() {
517522
sgd_sort: false,
518523
groom: false,
519524
aligner: "allwave".to_string(),
525+
frequency: None,
520526
};
521527

522528
run_inversion_aware_seqrush(args).unwrap();
@@ -588,6 +594,7 @@ fn test_interspersed_repeats_with_inversions() {
588594
sgd_sort: false,
589595
groom: false,
590596
aligner: "allwave".to_string(),
597+
frequency: None,
591598
};
592599

593600
run_inversion_aware_seqrush(args).unwrap();
@@ -666,6 +673,7 @@ fn test_breakpoint_resolution() {
666673
sgd_sort: false,
667674
groom: false,
668675
aligner: "allwave".to_string(),
676+
frequency: None,
669677
};
670678

671679
run_inversion_aware_seqrush(args).unwrap();

tests/test_edge_traversal.rs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -140,6 +140,7 @@ fn test_self_loops_in_gfa() {
140140
sgd_sort: false,
141141
groom: false,
142142
aligner: "allwave".to_string(),
143+
frequency: None,
143144
};
144145

145146
run_seqrush(args).unwrap();
@@ -213,6 +214,7 @@ fn test_complex_graph_produces_valid_gfa() {
213214
sgd_sort: false,
214215
groom: false,
215216
aligner: "allwave".to_string(),
217+
frequency: None,
216218
};
217219

218220
run_seqrush(args).unwrap();

tests/test_inversion_detection.rs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@ fn test_inversion_detection_simple() {
5555
sgd_sort: false,
5656
groom: false,
5757
aligner: "allwave".to_string(),
58+
frequency: None,
5859
};
5960

6061
println!("Running inversion-aware alignment...");
@@ -132,6 +133,7 @@ fn test_inversion_detection_with_context() {
132133
sgd_sort: false,
133134
groom: false,
134135
aligner: "allwave".to_string(),
136+
frequency: None,
135137
};
136138

137139
println!("\nRunning inversion detection with context...");
@@ -189,6 +191,7 @@ fn test_no_inversion() {
189191
sgd_sort: false,
190192
groom: false,
191193
aligner: "allwave".to_string(),
194+
frequency: None,
192195
};
193196

194197
run_inversion_aware_seqrush(args).unwrap();
@@ -244,6 +247,7 @@ fn test_multiple_inversions() {
244247
sgd_sort: false,
245248
groom: false,
246249
aligner: "allwave".to_string(),
250+
frequency: None,
247251
};
248252

249253
println!("\nRunning multiple inversion detection...");

tests/test_programmatic_variations.rs

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -202,6 +202,7 @@ fn test_programmatic_inversions_scale() {
202202
sgd_sort: false,
203203
groom: false,
204204
aligner: "allwave".to_string(),
205+
frequency: None,
205206
};
206207

207208
run_inversion_aware_seqrush(args).unwrap();
@@ -279,6 +280,7 @@ fn test_gc_content_variation() {
279280
sgd_sort: false,
280281
groom: false,
281282
aligner: "allwave".to_string(),
283+
frequency: None,
282284
};
283285

284286
run_seqrush_bidirected_simple(args).unwrap();
@@ -372,6 +374,7 @@ fn test_motif_based_variations() {
372374
sgd_sort: false,
373375
groom: false,
374376
aligner: "allwave".to_string(),
377+
frequency: None,
375378
};
376379

377380
run_inversion_aware_seqrush(args).unwrap();
@@ -456,6 +459,7 @@ fn test_microsatellite_instability() {
456459
sgd_sort: false,
457460
groom: false,
458461
aligner: "allwave".to_string(),
462+
frequency: None,
459463
};
460464

461465
run_inversion_aware_seqrush(args).unwrap();
@@ -542,6 +546,7 @@ fn test_complex_structural_variation_patterns() {
542546
sgd_sort: false,
543547
groom: false,
544548
aligner: "allwave".to_string(),
549+
frequency: None,
545550
};
546551

547552
run_inversion_aware_seqrush(args).unwrap();
@@ -639,6 +644,7 @@ fn test_nested_tandem_inversions() {
639644
sgd_sort: false,
640645
groom: false,
641646
aligner: "allwave".to_string(),
647+
frequency: None,
642648
};
643649

644650
run_inversion_aware_seqrush(args).unwrap();

tests/test_real_inversion.rs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,7 @@ fn test_real_inversion_case() {
5656
sgd_sort: false,
5757
groom: false,
5858
aligner: "allwave".to_string(),
59+
frequency: None,
5960
};
6061

6162
println!("Testing real inversion case...");
@@ -113,6 +114,7 @@ fn test_larger_inversion_with_unique_content() {
113114
sgd_sort: false,
114115
groom: false,
115116
aligner: "allwave".to_string(),
117+
frequency: None,
116118
};
117119

118120
println!("\nTesting larger inversion with unique content...");
@@ -169,6 +171,7 @@ fn test_inversion_with_snps() {
169171
sgd_sort: false,
170172
groom: false,
171173
aligner: "allwave".to_string(),
174+
frequency: None,
172175
};
173176

174177
println!("\nTesting inversion detection with SNPs...");

tests/test_topological_sort.rs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,7 @@ fn test_sorting_creates_sequential_node_ids() {
8484
sgd_sort: false,
8585
groom: false,
8686
aligner: "allwave".to_string(),
87+
frequency: None,
8788
};
8889

8990
run_seqrush(args_unsorted).unwrap();
@@ -197,6 +198,7 @@ fn test_sorting_preserves_graph_structure() {
197198
sgd_sort: false,
198199
groom: false,
199200
aligner: "allwave".to_string(),
201+
frequency: None,
200202
};
201203

202204
run_seqrush(args_sorted).unwrap();
@@ -233,6 +235,7 @@ fn test_sorting_preserves_graph_structure() {
233235
sgd_sort: false,
234236
groom: false,
235237
aligner: "allwave".to_string(),
238+
frequency: None,
236239
};
237240

238241
run_seqrush(args_unsorted).unwrap();

0 commit comments

Comments
 (0)