Skip to content

Commit

Permalink
display min required merges in options
Browse files Browse the repository at this point in the history
  • Loading branch information
Kr0nox committed Feb 18, 2025
1 parent 966e227 commit 22efb29
Show file tree
Hide file tree
Showing 5 changed files with 10 additions and 3 deletions.
1 change: 1 addition & 0 deletions report-viewer/src/model/CliOptions.ts
Original file line number Diff line number Diff line change
Expand Up @@ -37,4 +37,5 @@ export interface CliMergingOptions {
enabled: boolean
minNeighborLength: number
maxGapSize: number
minimumRequiredMerges: number
}
3 changes: 2 additions & 1 deletion report-viewer/src/model/factories/OptionsFactory.ts
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,8 @@ export class OptionsFactory extends BaseFactory {
return {
enabled: json['enabled'] as boolean,
minNeighborLength: json['min_neighbour_length'] as number,
maxGapSize: json['max_gap_size'] as number
maxGapSize: json['max_gap_size'] as number,
minimumRequiredMerges: json['min_required_merges'] as number
}
}

Expand Down
3 changes: 3 additions & 0 deletions report-viewer/src/views/InformationView.vue
Original file line number Diff line number Diff line change
Expand Up @@ -94,6 +94,9 @@
<TextInformation label="Max Gap Size"
>{{ options.mergingOptions.maxGapSize }} }}</TextInformation
>
<TextInformation label="Min Required Merges"
>{{ options.mergingOptions.minimumRequiredMerges }} }}</TextInformation
>
</div>
</div>
</ScrollableComponent>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,8 @@ describe('Test JSON to Options', async () => {
mergingOptions: {
enabled: false,
minNeighborLength: 0,
maxGapSize: 0
maxGapSize: 0,
minimumRequiredMerges: 3
}
})
})
Expand Down
3 changes: 2 additions & 1 deletion report-viewer/tests/unit/model/factories/ValidOptions.json
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@
"merging": {
"enabled": false,
"min_neighbour_length": 0,
"max_gap_size": 0
"max_gap_size": 0,
"min_required_merges": 3
}
}

0 comments on commit 22efb29

Please sign in to comment.