Skip to content

Commit

Permalink
Remove icon and update text for 0%
Browse files Browse the repository at this point in the history
  • Loading branch information
andrewiggins committed Dec 6, 2020
1 parent b63bbca commit 55b9089
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion dist/util.js
Original file line number Diff line number Diff line change
Expand Up @@ -8246,7 +8246,7 @@ function formatDifference({ absolute, percentChange: relative }) {
// show 0% - 0% but the actual result is actually not zero (i.e. -0.5% to
// 0.4%) let's still show the result as unsure to avoid a situation where we
// would display something like "slower ❌ 0% - 0% (0.00ms - 0.00ms)"
word = `<strong>unsure ❔</strong>`; // bold blue
word = `<strong>same</strong>`; // bold blue
rel = formatConfidenceInterval(relative, (n) => colorizeSign(n, percent));
abs = formatConfidenceInterval(absolute, (n) => colorizeSign(n, milli));
} else if (absolute.low > 0 && relative.low > 0) {
Expand Down
2 changes: 1 addition & 1 deletion src/utils/tachometer.js
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,7 @@ function formatDifference({ absolute, percentChange: relative }) {
// show 0% - 0% but the actual result is actually not zero (i.e. -0.5% to
// 0.4%) let's still show the result as unsure to avoid a situation where we
// would display something like "slower ❌ 0% - 0% (0.00ms - 0.00ms)"
word = `<strong>unsure ❔</strong>`; // bold blue
word = `<strong>same</strong>`; // bold blue
rel = formatConfidenceInterval(relative, (n) => colorizeSign(n, percent));
abs = formatConfidenceInterval(absolute, (n) => colorizeSign(n, milli));
} else if (absolute.low > 0 && relative.low > 0) {
Expand Down
4 changes: 2 additions & 2 deletions tests/getCommentBody.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -377,11 +377,11 @@ newCommentSuite("Shows unsure if change is 0% - 0%", async () => {
const summaryId = getSummaryId({ reportId });
const actual = html.querySelector(`#${summaryId}`).text;
const expected =
"report-id: unsure ❔ -0% - -0% (-0.00ms - -0.00ms)local-framework vs base-framework";
"report-id: same -0% - -0% (-0.00ms - -0.00ms)local-framework vs base-framework";
assert.equal(
actual,
expected,
"Expected unsure to show for rounded 0% difference"
"Expected same to show for rounded 0% difference"
);
});

Expand Down

0 comments on commit 55b9089

Please sign in to comment.