Skip to content
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.

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

Bug introduced between 3.2.56 -> 3.2.57 #14125

Closed
XbNz opened this issue Sep 3, 2024 · 1 comment
Closed

Bug introduced between 3.2.56 -> 3.2.57 #14125

XbNz opened this issue Sep 3, 2024 · 1 comment

Comments

@XbNz
Copy link

XbNz commented Sep 3, 2024

Package

filament/filament

Package Version

3.2.57

Laravel Version

11

Livewire Version

3

PHP Version

8.3

Problem description

I'm gonna go ahead and ignore the template here because the issue is clear.

The assertTableActionDoesNotExist – and I'm assuming some other assertion helpers as well – are no longer asserting after this upgrade. This happens when the table action code is present.

Example:

3.2.56:

$response->assertOk();
$response->assertTableActionDoesNotExist('changeState');

OK (1 test, 2 assertions)


3.2.57:

$response->assertOk();
$response->assertTableActionDoesNotExist('changeState');

OK (1 test, 1 assertions)

As you can see, it can easily be missed especially when it is part of an existing test suite. I use this assertion for my most important "user has role to do ..." test cases and I assume many others do as well, so I believe it's quite important to at the very least introduce backwards compatibility, especailly given that it was a minor version upgrade.

I think a simple fix would be to force the implementation of the $checkActionUsing and $record parameters in v4, where a breaking change is permissable.

Expected behavior

N/A

Steps to reproduce

N/A

Reproduction repository (issue will be closed if this is not valid)

N/A

Relevant log output

N/A

Donate 💰 to fund this issue

  • You can donate funding to this issue. We receive the money once the issue is completed & confirmed by you.
  • 100% of the funding will be distributed between the Filament core team to run all aspects of the project.
  • Thank you in advance for helping us make maintenance sustainable!
Fund with Polar
Copy link

github-actions bot commented Sep 3, 2024

Hey @XbNz! We're sorry to hear that you've hit this issue. 💛

However, it looks like you forgot to fill in the reproduction repository URL. Can you edit your original post and then we'll look at your issue?

We need a public GitHub repository which contains a Laravel app with the minimal amount of Filament code to reproduce the problem. Please do not link to your actual project, what we need instead is a minimal reproduction in a fresh project without any unnecessary code. This means it doesn't matter if your real project is private / confidential, since we want a link to a separate, isolated reproduction. That would allow us to download it and review your bug much easier, so it can be fixed quicker. Please make sure to include a database seeder with everything we need to set the app up quickly.

Also, it doesn't look like you've provided much information on how to replicate the issue. Please edit your original post with clear steps we need to take.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

1 participant