Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Moved "Return to Regular Route" into root directions list #2521

Merged
merged 11 commits into from
Mar 29, 2024
Merged

fix: Moved "Return to Regular Route" into root directions list #2521

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
2d03044
Update Storybook with "Return to route" wording change
hannahpurcell Mar 26, 2024
b353c56
Test out moving the "return to route" to the root directions list
hannahpurcell Mar 26, 2024
24a8278
Only add Regular Route after detour is finished
hannahpurcell Mar 26, 2024
fc1d9aa
Updated wording to match figma
hannahpurcell Mar 26, 2024
27b25b0
Added check: "regular route" only appears when detour is finished
hannahpurcell Mar 27, 2024
2bfae55
Formatting of test
hannahpurcell Mar 27, 2024
fb5fea8
Switch to jest-dom matchers
hannahpurcell Mar 28, 2024
d41a21a
Async behavior not needed for these click events
hannahpurcell Mar 28, 2024
27a72c1
Similarly, async/await not needed for the first test either
hannahpurcell Mar 28, 2024
480144a
Update test title wording
hannahpurcell Mar 28, 2024
18636f1
Tuck in a check during a test, re-work later
hannahpurcell Mar 29, 2024
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
Prev Previous commit
Next Next commit
Update test title wording 
Co-authored-by: Josh Larson <jlarson@mbta.com>
  • Loading branch information
@hannahpurcell @joshlarson
hannahpurcell and joshlarson authored Mar 28, 2024
commit 480144a78bbb8646c61ebc814f910730bb92866f
2 changes: 1 addition & 1 deletion assets/tests/components/detours/diversionPage.test.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -272,7 +272,7 @@ describe("DiversionPage", () => {
expect(await screen.findByText("Regular Route")).toBeVisible()
})

test("does not show 'Regular Route' when detour not finished", async () => {
test("does not show 'Regular Route' when detour is not finished", async () => {
jest.mocked(fetchDetourDirections).mockResolvedValue(
ok(
detourShapeFactory.build({
Expand Down
Loading