Skip to content

refactor: Refactor auth provider #5782

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

Merged
merged 18 commits into from
Jan 20, 2023
Merged

refactor: Refactor auth provider #5782

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
244845f
Add paywall to the audit route
BrunoQuaresma Jan 18, 2023
b83f9c2
refactor auth provider
BrunoQuaresma Jan 18, 2023
528892f
Move router to be inside of AppRouter
BrunoQuaresma Jan 18, 2023
9f0e0ee
Create dashboard provider
BrunoQuaresma Jan 18, 2023
1714ca5
Minor refactoring
BrunoQuaresma Jan 18, 2023
884dac7
Fix tests
BrunoQuaresma Jan 18, 2023
970e1a5
Merge branch 'bq/refactor-audit-route' into bq/refactor-auth-provider-2
BrunoQuaresma Jan 18, 2023
6e572e3
Merge branch 'main' of github.com:coder/coder into bq/refactor-auth-p…
BrunoQuaresma Jan 19, 2023
6442b3a
Fix Audit Log page test
BrunoQuaresma Jan 19, 2023
5747590
Add long time running test
BrunoQuaresma Jan 19, 2023
7768e95
Fix stories
BrunoQuaresma Jan 19, 2023
1173d16
Increase timeout
BrunoQuaresma Jan 19, 2023
ea4300b
Increse timeout
BrunoQuaresma Jan 19, 2023
d8826c3
Update site/src/components/WorkspaceStats/WorkspaceStats.stories.tsx
BrunoQuaresma Jan 19, 2023
e12f437
Update site/src/components/AuthProvider/AuthProvider.tsx
BrunoQuaresma Jan 19, 2023
9e7a875
Fix format
BrunoQuaresma Jan 19, 2023
3e589f1
Improve name
BrunoQuaresma Jan 19, 2023
dce50e3
Extract audit log paywall
BrunoQuaresma Jan 19, 2023
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
Fix tests
  • Loading branch information
@BrunoQuaresma
BrunoQuaresma committed Jan 18, 2023
commit 884dac7e523910c33ef46f1f6c849c57323013e5
27 changes: 0 additions & 27 deletions site/src/components/LicenseBanner/LicenseBanner.test.tsx
View file
Open in desktop

This file was deleted.

9 changes: 9 additions & 0 deletions site/src/components/WorkspaceStats/WorkspaceStats.stories.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
import { Story } from "@storybook/react"
import * as Mocks from "../../testHelpers/renderHelpers"
import { MockWorkspace } from "../../testHelpers/renderHelpers"
import {
WorkspaceStats,
WorkspaceStatsProps,
Expand All @@ -18,3 +19,11 @@ export const Example = Template.bind({})
Example.args = {
workspace: Mocks.MockWorkspace,
}

export const Outdated = Template.bind({})
Outdated.args = {
workspace: {
...MockWorkspace,
outdated: true,
},
}
36 changes: 0 additions & 36 deletions site/src/components/WorkspaceStats/WorkspaceStats.test.tsx
View file
Open in desktop

This file was deleted.

29 changes: 17 additions & 12 deletions site/src/testHelpers/renderHelpers.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ import {
screen,
waitForElementToBeRemoved,
} from "@testing-library/react"
import { AuthProvider } from "components/AuthProvider/AuthProvider"
import { DashboardLayout } from "components/Dashboard/DashboardLayout"
import { createMemoryHistory } from "history"
import { i18n } from "i18n"
Expand All @@ -28,9 +29,11 @@ export const WrapperComponent: FC<React.PropsWithChildren<unknown>> = ({
}) => {
return (
<HelmetProvider>
<HistoryRouter history={history}>
<ThemeProvider theme={dark}>{children}</ThemeProvider>
</HistoryRouter>
<ThemeProvider theme={dark}>
<HistoryRouter history={history}>
<AuthProvider>{children}</AuthProvider>
</HistoryRouter>
</ThemeProvider>
</HelmetProvider>
)
}
Expand Down Expand Up @@ -59,16 +62,18 @@ export function renderWithAuth(
<HelmetProvider>
<I18nextProvider i18n={i18n}>
<ThemeProvider theme={dark}>
<MemoryRouter initialEntries={[route]}>
<Routes>
<Route element={<RequireAuth />}>
<Route element={<DashboardLayout />}>
<Route path={path ?? route} element={ui} />
<AuthProvider>
<MemoryRouter initialEntries={[route]}>
<Routes>
<Route element={<RequireAuth />}>
<Route element={<DashboardLayout />}>
<Route path={path ?? route} element={ui} />
</Route>
</Route>
</Route>
{routes}
</Routes>
</MemoryRouter>
{routes}
</Routes>
</MemoryRouter>
</AuthProvider>
</ThemeProvider>
</I18nextProvider>
</HelmetProvider>,
Expand Down