Skip to content

Commit

Permalink
refactor: tweaks
Browse files Browse the repository at this point in the history
  • Loading branch information
mdjastrzebski committed Jun 16, 2024
1 parent 2c458ac commit e58efac
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 5 deletions.
6 changes: 6 additions & 0 deletions .changeset/ten-toes-turn.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
---
'reassure': minor
'@callstack/reassure-compare': minor
---

Expose "Warmup Runs" and "Render Issues" in markdown report
10 changes: 5 additions & 5 deletions packages/compare/src/output/markdown.ts
Original file line number Diff line number Diff line change
Expand Up @@ -166,7 +166,7 @@ function buildDurationDetails(title: string, entry: MeasureEntry) {
`Mean: ${formatDuration(entry.meanDuration)}`,
`Stdev: ${formatDuration(entry.stdevDuration)} (${formatPercent(relativeStdev)})`,
entry.durations ? `Runs: ${formatRunDurations(entry.durations)}` : '',
entry.warmupDurations ? `Warmup: ${formatRunDurations(entry.warmupDurations)}` : '',
entry.warmupDurations ? `Warmup runs: ${formatRunDurations(entry.warmupDurations)}` : '',
]
.filter(Boolean)
.join(`<br/>`);
Expand Down Expand Up @@ -206,15 +206,15 @@ function buildRenderIssuesTable(entries: Array<CompareEntry | AddedEntry>) {
function buildRenderIssuesList(issues: RenderIssues | undefined) {
if (issues == null) return '';

const output = [];
const output = ['Render issues:'];
if (issues?.initialUpdateCount) {
output.push(`* Initial updates: ${formatInitialUpdates(issues.initialUpdateCount)}`);
output.push(` - Initial updates: ${formatInitialUpdates(issues.initialUpdateCount)}`);
}
if (issues?.redundantUpdates) {
output.push(`* Redundant updates: ${formatRedundantUpdates(issues.redundantUpdates)}`);
output.push(` - Redundant updates: ${formatRedundantUpdates(issues.redundantUpdates)}`);
}

return output.join('\n');
return output.join('<br/>');
}

function formatInitialUpdates(count: number | undefined) {
Expand Down

0 comments on commit e58efac

Please sign in to comment.