Skip to content

Commit

Permalink
Replaced references to breakpoints-01 with breakpoints-02
Browse files Browse the repository at this point in the history
  • Loading branch information
bvaughn committed Jul 8, 2024
1 parent da3f51b commit fe68096
Show file tree
Hide file tree
Showing 7 changed files with 11 additions and 11 deletions.
2 changes: 1 addition & 1 deletion packages/e2e-tests/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,7 @@ You'll also probably need to specifically share that recording as "Public", espe

We use 1 specific Playwright test recording which is a replay of the breakpoints-05 test from this repo. Similar to the instructions above, to update this to a more recent version, you can visit the dashboard for FE E2E Tests and select a more recent recording ID for that test.

We also now have a "golden recording" of one of our own `breakpoints-01` E2E test runs. This serves as a testbed for checking more advanced behaviors like the React and Redux routines. If we ever need to update this, just copy-paste the recording ID from a test run in our "Frontend E2E tests" workspace.
We also now have a "golden recording" of one of our own `breakpoints-02` E2E test runs. This serves as a testbed for checking more advanced behaviors like the React and Redux routines. If we ever need to update this, just copy-paste the recording ID from a test run in our "Frontend E2E tests" workspace.

### Test Suite Dashboard Tests

Expand Down
6 changes: 3 additions & 3 deletions packages/e2e-tests/examples.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion packages/e2e-tests/tests/jump-to-code-02_redux-j2c.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import { getByTestName, waitFor } from "../helpers/utils";
import test from "../testFixture";

// trunk-ignore(gitleaks/generic-api-key)
test.use({ exampleKey: "breakpoints-01" });
test.use({ exampleKey: "breakpoints-02" });

async function checkForJumpButton(actionListItem: Locator, shouldBeEnabled: boolean) {
const jumpButton = getEventJumpButton(actionListItem);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ const sourceUrl = "react-dom.production.js";
const lineNumber = 150;

// trunk-ignore(gitleaks/generic-api-key)
test.use({ exampleKey: "breakpoints-01" });
test.use({ exampleKey: "breakpoints-02" });

test(`logpoints-10: too-many-points-to-find UX`, async ({
pageWithMeta: { page, recordingId, testScope },
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,12 +11,12 @@ import { getFocusBeginTime, getFocusEndTime, setFocusRange } from "../helpers/ti
import test, { expect } from "../testFixture";

// We need 500...10k hits
// Line 248 has 847 hits
// Line 301 has 534 hits
const sourceUrl = "react-dom.production.js";
const lineNumber = 248;
const lineNumber = 301;

// trunk-ignore(gitleaks/generic-api-key)
test.use({ exampleKey: "breakpoints-01" });
test.use({ exampleKey: "breakpoints-02" });

test(`logpoints-11: too-many-points-to-run-analysis UX`, async ({
pageWithMeta: { page, recordingId, testScope },
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ import test, { expect } from "../testFixture";

// Why is this even getting confused as an API key?
// trunk-ignore(gitleaks/generic-api-key)
test.use({ exampleKey: "breakpoints-01" });
test.use({ exampleKey: "breakpoints-02" });

test("react_devtools-02: RDT integrations (Chromium)", async ({
pageWithMeta: { page, recordingId, testScope },
Expand Down
2 changes: 1 addition & 1 deletion test/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ Tests are defined in the `manifest.js` file.
// Example browser based e2e tests
{
"example": "doc_rr_basic.html",
"script": "breakpoints-01.js",
"script": "breakpoints-02.js",
"targets": ["gecko", "chromium"]
},

Expand Down

0 comments on commit fe68096

Please sign in to comment.