Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1,100 changes: 560 additions & 540 deletions package-lock.json

Large diffs are not rendered by default.

36 changes: 18 additions & 18 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,9 +16,9 @@
},
"dependencies": {
"@emotion/styled": "^11.14.1",
"@mui/icons-material": "^7.3.7",
"@mui/material": "^7.3.7",
"@mui/system": "^7.3.7",
"@mui/icons-material": "^7.3.9",
"@mui/material": "^7.3.9",
"@mui/system": "^7.3.9",
"@reduxjs/toolkit": "^2.11.2",
"assert": "^2.1.0",
"buffer": "^6.0.3",
Expand All @@ -28,7 +28,7 @@
"express": "^5.2.1",
"fast-kde": "^0.2.2",
"format": "^0.2.2",
"json-e": "^4.8.1",
"json-e": "^4.8.2",
"material-ui-popup-state": "^5.3.6",
"moize": "^6.1.7",
"notistack": "^3.0.2",
Expand All @@ -37,8 +37,8 @@
"react-chartjs-2": "^5.3.1",
"react-dom": "^19.2.4",
"react-redux": "^9.2.0",
"react-router": "^7.13.0",
"react-virtuoso": "^4.18.1",
"react-router": "^7.13.1",
"react-virtuoso": "^4.18.3",
"stream-browserify": "^3.0.0",
"taskcluster-client-web": "^87.1.3",
"taskcluster-lib-scopes": "^11.0.0",
Expand Down Expand Up @@ -87,10 +87,10 @@
"@babel/preset-react": "^7.28.5",
"@babel/preset-typescript": "^7.28.5",
"@babel/runtime": "^7.28.6",
"@eslint/js": "^9.39.2",
"@eslint/js": "^9.39.4",
"@fetch-mock/jest": "^0.2.20",
"@jest/types": "^30.2.0",
"@swc/core": "^1.15.10",
"@swc/core": "^1.15.18",
"@swc/jest": "^0.2.39",
"@testing-library/dom": "^10.4.1",
"@testing-library/jest-dom": "^6.9.1",
Expand All @@ -99,24 +99,24 @@
"@types/jest": "^30.0.0",
"@types/jest-axe": "^3.5.9",
"@types/material-ui": "^0.21.18",
"@types/node": "^25.0.9",
"@types/react": "^19.2.8",
"@types/node": "^25.2.3",
"@types/react": "^19.2.14",
"@types/react-dom": "^19.2.3",
"babel-loader": "^10.0.0",
"babel-loader": "^10.1.1",
"babel-preset-jest": "^30.2.0",
"case-sensitive-paths-webpack-plugin": "^2.4.0",
"copy-webpack-plugin": "^13.0.1",
"core-js": "^3.48.0",
"css-loader": "^7.1.3",
"eslint": "^9.39.2",
"css-loader": "^7.1.4",
"eslint": "^9.39.4",
"eslint-config-prettier": "^10.1.8",
"eslint-import-resolver-typescript": "^4.4.4",
"eslint-plugin-import": "^2.32.0",
"eslint-plugin-jest": "^29.12.1",
"eslint-plugin-jest": "^29.15.0",
"eslint-plugin-jest-dom": "^5.5.0",
"eslint-plugin-react": "^7.37.5",
"eslint-plugin-testing-library": "^7.15.4",
"globals": "^17.3.0",
"eslint-plugin-testing-library": "^7.16.0",
"globals": "^17.4.0",
"history": "^5.3.0",
"html-webpack-plugin": "^5.6.6",
"jest": "^30.2.0",
Expand All @@ -133,8 +133,8 @@
"ts-loader": "^9.5.4",
"ts-node": "^10.9.2",
"typescript": "^5.9.3",
"typescript-eslint": "^8.54.0",
"webpack": "^5.104.1",
"typescript-eslint": "^8.56.0",
"webpack": "^5.105.4",
"webpack-cli": "^6.0.1",
"webpack-dev-server": "^5.2.3",
"webpack-merge": "^6.0.1"
Expand Down
49 changes: 39 additions & 10 deletions src/__tests__/App.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -21,38 +21,67 @@ describe('App', () => {
render(<App />);

// Title appears
const title = screen.getAllByText(/PerfCompare/i)[1];
expect(title).toBeInTheDocument();
const title = await screen.findByRole('heading', { level: 1 });
expect(title).toHaveTextContent('PerfCompare');

act(() => void jest.runAllTimers());
const homeText = screen.getByText('Compare with a base or over time');
expect(homeText).toBeInTheDocument();
// The Tagline text appears
const taglineText = await screen.findByText(
/Analyze results of performance tests to detect regressions and identify opportunities for improvement./i,
);
expect(taglineText).toBeInTheDocument();
await waitForAllFetches();
});

test('Should display File bug link', async () => {
test('Should display a link to the Docs with the correct tooltip', async () => {
render(<App />);

const link = await screen.findByRole('link', {
name: /report it on Bugzilla/,
name: 'Docs',
});
expect(link).toHaveAttribute(
'href',
'https://bugzilla.mozilla.org/enter_bug.cgi?product=Testing&component=PerfCompare&status_whiteboard=[pcf]',
'https://firefox-source-docs.mozilla.org/testing/perfdocs/perfcompare.html',
);
expect(link).toHaveAttribute('title', 'View the documentation');
await waitForAllFetches();
});

test('Should display PerfCompare Matrix channel link', async () => {
test('Should display a link to the GitHub repo with the correct tooltip', async () => {
render(<App />);
const link = await screen.findByRole('link', {
name: 'Source',
});
expect(link).toHaveAttribute(
'href',
'https://github.com/mozilla/perfcompare',
);
expect(link).toHaveAttribute('title', 'View the source code on GitHub');
await waitForAllFetches();
});

test('Should display PerfCompare Matrix channel link with the correct tooltip', async () => {
render(<App />);
const link = await screen.findByRole('link', {
name: '#perfcompare:mozilla.org on Matrix',
name: 'Matrix',
});
expect(link).toHaveAttribute(
'href',
'https://matrix.to/#/#perfcompare:mozilla.org',
);
expect(link).toHaveAttribute('title', 'Chat with us on Matrix');
await waitForAllFetches();
});

test('Should display the correct link to file a new bug on Bugzilla with the correct tooltip', async () => {
render(<App />);
const link = await screen.findByRole('link', {
name: 'Bugzilla',
});
expect(link).toHaveAttribute(
'href',
'https://bugzilla.mozilla.org/enter_bug.cgi?product=Testing&component=PerfCompare&status_whiteboard=[pcf]',
);
expect(link).toHaveAttribute('title', 'Report an issue on Bugzilla');
await waitForAllFetches();
});

Expand Down
2 changes: 1 addition & 1 deletion src/__tests__/CompareResults/RevisionRow.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -265,7 +265,7 @@ describe('Expanded row', () => {
const directionOfChange = roles[4]?.childNodes[0];
expect(directionOfChange).toHaveTextContent('Improvement');

const cliffsDelta = roles[5]?.childNodes[1];
const cliffsDelta = roles[5]?.childNodes[0];
expect(cliffsDelta).toHaveTextContent('.1');
});

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -355,7 +355,7 @@ exports[`Results View The table should match snapshot and other elements should
>
<span
aria-label="A summary of all values from Base runs using a mean."
class="MuiBox-root css-4g6ai3"
class="MuiBox-root css-1tkq7a4"
data-mui-internal-clone-element="true"
>
Base
Expand All @@ -371,7 +371,7 @@ exports[`Results View The table should match snapshot and other elements should
>
<span
aria-label="A summary of all values from New runs using a mean."
class="MuiBox-root css-4g6ai3"
class="MuiBox-root css-1tkq7a4"
data-mui-internal-clone-element="true"
>
New
Expand Down Expand Up @@ -416,8 +416,7 @@ exports[`Results View The table should match snapshot and other elements should
role="columnheader"
>
<span
aria-label="Cliff’s Delta quantifies the magnitude of the difference between Base and New values."
class="MuiBox-root css-4g6ai3"
class="MuiBox-root css-1tkq7a4"
data-mui-internal-clone-element="true"
>
<button
Expand Down Expand Up @@ -509,8 +508,7 @@ exports[`Results View The table should match snapshot and other elements should
role="columnheader"
>
<span
aria-label="An improvement or regression being shown here means that the effect size is meaningful, and the difference has a significant p-value."
class="MuiBox-root css-4g6ai3"
class="MuiBox-root css-1tkq7a4"
data-mui-internal-clone-element="true"
>
<button
Expand Down Expand Up @@ -543,7 +541,7 @@ exports[`Results View The table should match snapshot and other elements should
>
<span
aria-label="The total number of tasks/jobs that ran for this metric."
class="MuiBox-root css-4g6ai3"
class="MuiBox-root css-1tkq7a4"
data-mui-internal-clone-element="true"
>
Total Runs
Expand Down Expand Up @@ -637,7 +635,7 @@ exports[`Results View The table should match snapshot and other elements should
</div>
</div>
<div
class="fexjzun"
class="MuiBox-root css-56tgmi"
>
<span
class="f1rpr1m5"
Expand Down Expand Up @@ -730,7 +728,6 @@ exports[`Results View The table should match snapshot and other elements should
class="delta cell"
role="cell"
>

-
</div>
<div
Expand Down Expand Up @@ -920,7 +917,6 @@ exports[`Results View The table should match snapshot and other elements should
class="delta cell"
role="cell"
>

-
</div>
<div
Expand Down Expand Up @@ -1110,7 +1106,6 @@ exports[`Results View The table should match snapshot and other elements should
class="delta cell"
role="cell"
>

-
</div>
<div
Expand Down Expand Up @@ -1300,7 +1295,6 @@ exports[`Results View The table should match snapshot and other elements should
class="delta cell"
role="cell"
>

-
</div>
<div
Expand Down
Loading
Loading