search

dddeastanglia / DDDEastAnglia

DDD East Anglia website
https://www.dddeastanglia.com
7 stars 10 forks source link

Sessions pages in admin section #275

Closed adrianbanks closed 10 years ago

adrianbanks commented 10 years ago

This is based on #270 as it shares a lot of the styling changes, so that should be merged first.

Style the session details, edit and delete pages to match the newly-created users pages.

alastairs commented 10 years ago

Where are the tests for the new actions? I think I might have missed this in the user's admin pages too... :frowning:

adrianbanks commented 10 years ago

I couldn't see what to test other than the implementation when I looked at it originally. Looking at it again, I think there is some behaviour for some of the actions, but not all. Any suggestions?

alastairs commented 10 years ago

Well, off the top of my head, there's the interactions with the repositories to verify: ensuring the controller delegates correctly.

adrianbanks commented 10 years ago

there's the interactions with the repositories to verify

Hmm, feels a bit like testing the implementation. I'll take a look at see what I can do.