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

Implement UnitTest builder pattern for AllReady.Areas.Admin.Controllers.CampaignController #2283

Closed
Paul-Hadfield opened this Issue Jan 20, 2018 · 1 comment

Comments

Projects
None yet
1 participant
@Paul-Hadfield
Collaborator

Paul-Hadfield commented Jan 20, 2018

Before working on #2219, I will implement the Builder pattern for the constructor of the controller in relevant unit tests. This is taking what was learnt from working on #2228, which lead to #2263

@Paul-Hadfield

This comment has been minimized.

Collaborator

Paul-Hadfield commented Jan 20, 2018

I'll pick this one up

Paul-Hadfield added a commit to Paul-Hadfield/allReady that referenced this issue Jan 20, 2018

Introduce builder pattern for HTBox#2283
Another builder pattern, this time ahead of introducing new parameter in HTBox#2219

MisterJames added a commit that referenced this issue Mar 7, 2018

Merge pull request #2312 from stimms/feature/use_builder_in_test_2
Using builder pattern in tests. Fixes #2283
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment