generated from Seekra/repository-template
fix(settings): return router.push promise
The settings page composable function to go to a settings page now returns the promise router.push returns.
This commit is contained in:
@@ -26,10 +26,10 @@ vi.mock('vue-router', () => ({
|
|||||||
|
|
||||||
describe('useSettingsPage', () => {
|
describe('useSettingsPage', () => {
|
||||||
describe('goToSettingsPage', () => {
|
describe('goToSettingsPage', () => {
|
||||||
test('goes to correct route', () => {
|
test('goes to correct route', async () => {
|
||||||
const { goToSettingsPage } = useSettingsPage();
|
const { goToSettingsPage } = useSettingsPage();
|
||||||
|
|
||||||
goToSettingsPage('a.b.c');
|
await goToSettingsPage('a.b.c');
|
||||||
expect(pushRoute).toHaveBeenCalledWith('/settings/a/b/c');
|
expect(pushRoute).toHaveBeenCalledWith('/settings/a/b/c');
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|||||||
@@ -24,7 +24,7 @@ export const useSettingsPage = function useSettingsPage () {
|
|||||||
const router = useRouter();
|
const router = useRouter();
|
||||||
|
|
||||||
const goToSettingsPage = function goToSettingsPage (pagePath) {
|
const goToSettingsPage = function goToSettingsPage (pagePath) {
|
||||||
router.push(`/settings/${normalizePagePath(pagePath)}`);
|
return router.push(`/settings/${normalizePagePath(pagePath)}`);
|
||||||
};
|
};
|
||||||
|
|
||||||
return { goToSettingsPage };
|
return { goToSettingsPage };
|
||||||
|
|||||||
Reference in New Issue
Block a user