feat(core): ability to pass an argument to usePermissions

[Cavdy]

PR Checklist

Please check if your PR fulfills the following requirements:

• [ ] The commit message follows our guidelines: https://refine.dev/docs/guides-concepts/contributing/#commit-convention

Bugs / Features

• [ ] Related issue(s) linked
• [ ] Tests for the changes have been added
• [ ] Docs have been added / updated
• [ ] Changesets have been added https://refine.dev/docs/guides-concepts/contributing/#creating-a-changeset

What is the current behavior?

What is the new behavior?

added ability to pass an argument to usePermissions.

fixes #5607

Notes for reviewers

[changeset-bot[bot]]

🦋 Changeset detected

Latest commit: d3e0daa0181e3d50d7a1f8e6d442cc64736316cf

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[Cavdy]

@BatuhanW test case written but not sure if I got it right

[alicanerdurmaz]

@BatuhanW test case written but not sure if I got it right

Hello @Cavdy, It looks neat 🚀 But, I have some suggestions:

In the following code, I mocked the getPermissions function to assert that it is called with "params"

   it("should accept params", async () => {
        const mockGetPermissions = jest.fn(() => Promise.resolve(["admin"]));
        const { result } = renderHook(
            ({ params }: { params: string }) => usePermissions({ params }),
            {
                initialProps: { params: "admin" },
                wrapper: TestWrapper({
                    authProvider: {
                        login: () => Promise.resolve({ success: true }),
                        check: () => Promise.resolve({ authenticated: true }),
                        onError: () => Promise.resolve({}),
                        logout: () => Promise.resolve({ success: true }),
                        
                        getPermissions: mockGetPermissions,
                    },
                    dataProvider: MockJSONServer,
                    resources: [{ name: "posts" }],
                }),
            },
        );

        await waitFor(() => {
            expect(result.current.isSuccess).toBeTruthy();
        });
        
        expect(mockGetPermissions).toHaveBeenCalledWith("admin");
        expect(result.current.data).toEqual(["admin"]);
    });

Also, I believe we need to test with { v3LegacyAuthProviderCompatible: true } prop. I think the assertions will remain the same but in this case we will pass extra prop.

[Cavdy]

Thanks for the suggestion @alicanerdurmaz

I have updated the test

[nx-cloud[bot]]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit d3e0daa0181e3d50d7a1f8e6d442cc64736316cf. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution

---

✅ Successfully ran 24 targets

• lerna run cypress:run --scope with-remix-material-ui -- --record --key --group with-remix-material-ui
• lerna run cypress:run --scope with-remix-headless -- --record --key --group with-remix-headless
• lerna run cypress:run --scope with-remix-antd -- --record --key --group with-remix-antd
• lerna run cypress:run --scope with-nextjs-next-auth -- --record --key --group with-nextjs-next-auth
• lerna run cypress:run --scope with-nextjs -- --record --key --group with-nextjs
• lerna run cypress:run --scope table-react-table-basic -- --record --key --group table-react-table-basic
• lerna run cypress:run --scope table-react-table-advanced -- --record --key --group table-react-table-advanced
• lerna run cypress:run --scope table-material-ui-use-update-many -- --record --key --group table-material-ui-use-update-many
• lerna run cypress:run --scope table-material-ui-use-delete-many -- --record --key --group table-material-ui-use-delete-many
• lerna run cypress:run --scope table-material-ui-use-data-grid -- --record --key --group table-material-ui-use-data-grid
• lerna run cypress:run --scope table-material-ui-table-filter -- --record --key --group table-material-ui-table-filter
• lerna run cypress:run --scope table-material-ui-data-grid-pro -- --record --key --group table-material-ui-data-grid-pro
• lerna run cypress:run --scope table-material-ui-cursor-pagination -- --record --key --group table-material-ui-cursor-pagination
• lerna run cypress:run --scope table-material-ui-advanced -- --record --key --group table-material-ui-advanced
• lerna run cypress:run --scope table-mantine-basic -- --record --key --group table-mantine-basic
• lerna run cypress:run --scope table-mantine-advanced -- --record --key --group table-mantine-advanced
• lerna run cypress:run --scope table-handson -- --record --key --group table-handson
• lerna run cypress:run --scope table-chakra-ui-basic -- --record --key --group table-chakra-ui-basic
• lerna run cypress:run --scope table-chakra-ui-advanced -- --record --key --group table-chakra-ui-advanced
• lerna run cypress:run --scope table-antd-use-update-many -- --record --key --group table-antd-use-update-many
• lerna run cypress:run --scope table-antd-use-table -- --record --key --group table-antd-use-table
• lerna run cypress:run --scope table-antd-use-editable-table -- --record --key --group table-antd-use-editable-table
• lerna run cypress:run --scope table-antd-use-delete-many -- --record --key --group table-antd-use-delete-many
• lerna run cypress:run --scope table-antd-table-filter -- --record --key --group table-antd-table-filter

---

Sent with 💌 from NxCloud.

[Cavdy]

@BatuhanW done the changes... Thanks for the feedback