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

Streamline addon test suites #1184

Open
jasonvarga opened this issue Jan 8, 2020 · 1 comment
Open

Streamline addon test suites #1184

jasonvarga opened this issue Jan 8, 2020 · 1 comment

Comments

@jasonvarga
Copy link
Member

@jasonvarga jasonvarga commented Jan 8, 2020

As shown in #1159 and probably some other issues, testing addons needs some love.

I think it would be good to have our own abstract TestCase for developers to extend that does extra necessary bootstrapping, rather than them needing to extend from Testbench's and do a bunch of wiring.

There's also the ExceptionHandler issue, and Statamic config loading, both of which could probably be done as part of this abstracted class.

There's a bunch of useful fake classes used in our own tests that could be useful to addon devs. Stuff should probably be moved out from our tests directory and into a real Statamic\Testing namespace.

@github-actions
Copy link

@github-actions github-actions bot commented Jan 7, 2021

This issue has not had recent activity and has been marked as stale — by me, a robot. Simply reply to keep it open and send me away. If you do nothing, I will close it in a week. I have no feelings, so whatever you do is fine by me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
3.0.0-final
  
Awaiting triage
Development

Successfully merging a pull request may close this issue.

1 participant