A Patch with redesign to use mocks instead of using test-only Methods about bigvisiblecruise

[GoogleCodeExporter]

I failed to mentioned, that I'd also updated NUnit and RhinoMock libraries - 
which is
why the patch is reasonably large.
... I also forgot to delete the offending property ProjectStatusDocument - in 
the
patch. It's unused and can be deleted.

Original comment by [email protected] on 13 Dec 2008 at 1:03

[GoogleCodeExporter]

Thanks for the patch. I'll get it reviewed soon.

Original comment by [email protected] on 14 Dec 2008 at 9:12

[GoogleCodeExporter]

I added the patch. Thanks for cleanup and I like the introduction of the stubs 
- 
feels much cleaner. I haven't had much time to work on the project lately, so I 
appreciate the effort to help me clean up the code. It's very appreciated.

Patch is applied and in the trunk. This will make it in for the next release.

Thanks again.
- Ben

Original comment by [email protected] on 15 Dec 2008 at 1:41

• Changed state: RequestImplemented

[GoogleCodeExporter]

No worries, I'm happy to work on this at the moment. I've already done a fair 
bit
since that patch, yesterday ;-)
eg I've added a gradient to the background colors and I'll be adding a simple
animation to move from one color to another tonight.
If the patches are hard to keep up with, I'm happy to commit directly, if you 
trust
me ;-)

Original comment by [email protected] on 15 Dec 2008 at 1:59

[GoogleCodeExporter]

Good to hear. I've added a few minor mods as well. So, try and update before 
submitting - it will make my life a little easier :)

Original comment by [email protected] on 15 Dec 2008 at 3:55