#8647 fix vitest warnings #1467

Merged
jon merged 91 commits from 8647_fix_warnings into dev 2025-04-16 11:18:11 +00:00
2 changed files with 9 additions and 10 deletions
Showing only changes of commit 6778a309dd - Show all commits

View File

@ -25,7 +25,7 @@ describe('useArrayData', () => {
arrayData.fetch({});
await flushPromises();
const router = useRouter();
const routerReplace = useRouter().replace.mock.calls[0][0];
expect(axios.get.mock.calls[0][1].params).toEqual({

View File

@ -11,14 +11,8 @@ export function useArrayData(key, userOptions) {
let route = null;
Outdated
Review

Esto da warnings? No serà que hay q mockearlo?

Esto da warnings? No serà que hay q mockearlo?

Resuelve la #8028

Resuelve la #8028

Warning en DevTools no en vitest

Warning en DevTools no en vitest
let router = null;
const initializeRouting = () => {
if (!route) route = useRoute();
if (!router) router = useRouter();
return { route, router };
};
// Si no hay key, intentamos obtenerla del route
if (!key) {
const { route: initialRoute, router: initialRouter } = initializeRouting();
key = initialRoute?.meta?.moduleName;
route = initialRoute;
router = initialRouter;
@ -30,15 +24,20 @@ export function useArrayData(key, userOptions) {
const store = arrayDataStore.get(key);
let canceller = null;
const { route: initialRoute, router: initialRouter } = (() => {
if (!route) route = useRoute();
if (!router) router = useRouter();
return { route, router };
})();
onMounted(() => {
setOptions();
reset(['skip']);
const { route: initialRoute, router: initialRouter } = initializeRouting();
route = initialRoute;
router = initialRouter;
const searchUrl = store.searchUrl;
if (route.query[searchUrl]) {
const params = JSON.parse(query[searchUrl]);
const query = route.query[searchUrl];
if (query) {
const params = JSON.parse(query);
const filter =
params?.filter && typeof params?.filter == 'object'
? params?.filter