illegal offset type in tests

I have many of my tests fail with the same error, and not sure why.

As an example test that is failing, its complaining about the ->test line

    public function test_missing_fields_when_creating_user(): void
    {
        $this->tempUser->assignRole(RoleConstants::SUPER_ADMIN);

        Livewire::actingAs($this->tempUser)
            ->test(UserResource\Pages\CreateUser::class)
            ->call('create')
            ->assertHasErrors(['data.name', 'data.email', 'data.password', 'data.password_confirmation']);

        $this->tempUser->removeRole(RoleConstants::SUPER_ADMIN);
    }

    public function test_missing_fields_when_creating_user(): void
    {
        $this->tempUser->assignRole(RoleConstants::SUPER_ADMIN);

        Livewire::actingAs($this->tempUser)
            ->test(UserResource\Pages\CreateUser::class)
            ->call('create')
            ->assertHasErrors(['data.name', 'data.email', 'data.password', 'data.password_confirmation']);

        $this->tempUser->removeRole(RoleConstants::SUPER_ADMIN);
    }

Solution

I Finally found it.

So basically where im creating the temp users, I have to cast the uuid as a string when creating

        $user->id = (string) Str::uuid();

        $user->id = (string) Str::uuid();

Else it was detecting as a LazyLoadFromUuid or some named object

Jump to solution

Jamie CeeOP•11/18/24, 11:26 AM

And the page its referencing

Jamie CeeOP•11/18/24, 1:41 PM

THese tests used to work, until I implemented the spatie perrmissions plugin

awcodes•11/18/24, 1:56 PM

Make sure you are seeding the roles and permissions too.

Aawcodes Make sure you are seeding the roles and permissions too.

Jamie CeeOP•11/18/24, 2:11 PM

I have this in my TestCase setup flow

Jamie CeeOP•11/18/24, 2:18 PM

Done a dd in the TestCase, I can see the role definitely exists, and returns true if I check the user having the role

awcodes•11/18/24, 2:18 PM

And what’s in the policy?

Aawcodes And what’s in the policy?

Jamie CeeOP•11/18/24, 2:19 PM

Along the lines of this, but I've even tested by just returning true in every function, same outcome

awcodes•11/18/24, 2:21 PM

Hm. Filament honors the policy and your policy is checking permissions not roles. But I would expect it to work with true.

Jamie CeeOP•11/18/24, 2:23 PM

I've just tried adding ->givePermissionTo() on the user, but still getting the same Illegal offset type error. Im just so confused

Jamie CeeOP•11/18/24, 2:24 PM

But even then, its Super Admin im adding, does that not cover all permissions?

awcodes•11/18/24, 2:25 PM

Permissions and Roles are separate in Spatie's package

awcodes•11/18/24, 2:25 PM

when you seed the roles you also have to seed the permissions and assign the permissions to the roles if you want to use both

Jamie CeeOP•11/18/24, 2:27 PM

Im not actually wiping the database currently during these tests, so those permissions do already exist in the db.

awcodes•11/18/24, 2:28 PM

do you have a repo you can share?

Jamie CeeOP•11/18/24, 2:30 PM

Sadly its a work one, so NDA.

I can try mimic as much of this as possible to remove redacted information to share. What particularly would you be looking for?

awcodes•11/18/24, 2:32 PM

i'm using pest, but i have these helpers:

awcodes•11/18/24, 2:33 PM

then in my tests i use to login the user with the appropriate permissions and role. and it works.

Jamie CeeOP•11/18/24, 2:33 PM

Ill try replicate and give it a go

awcodes•11/18/24, 2:36 PM

try using instead of . I think that might be an internal method. The docs say to use

Aawcodes try using `->can()` instead of `->checkPermissionTo()`. I think that might be an...

Jamie CeeOP•11/18/24, 2:37 PM

The checkPermissionTo() came with the generation command. Apologies, im also using the https://github.com/althinect/filament-spatie-roles-permissions repo

GitHub

GitHub - Althinect/filament-spatie-roles-permissions

Contribute to Althinect/filament-spatie-roles-permissions development by creating an account on GitHub.

awcodes•11/18/24, 2:38 PM

ah, don't know anything about the package. sorry. might get better support if you ask the question directly on that channel. #althinect-spatie-roles-permissions

Aawcodes ah, don't know anything about the package. sorry. might get better support if yo...

Jamie CeeOP•11/18/24, 2:38 PM

WIll do. Thank you

awcodes•11/18/24, 2:38 PM

if it's something in the package then it's not a filament question.

Solution

Aawcodes if it's something in the package then it's not a filament question. 🙂

Jamie Cee•11/18/24, 3:54 PM

I Finally found it.

So basically where im creating the temp users, I have to cast the uuid as a string when creating

        $user->id = (string) Str::uuid();

        $user->id = (string) Str::uuid();

Else it was detecting as a LazyLoadFromUuid or some named object