I've just done a few tests to check if we could make this Configuration thing just simple.
As I suspected, all tests pass just fine and the examples webapp works fine as well. I'm attaching the patch for review. I'd vote for applying it (at least to trunk) unless we have a good answer for the current design. Cheers Remi
configcleanup.patch
Description: Binary data
Le 17 févr. 2013 à 21:53, Remi VANKEISBELCK a écrit : > It's done, I have refactored the unit tests so that they use a > MockServletContext whenever they need to access the configuration. > > The current test base was relying on ugly statics, it ain't the case now, we > create and stop the StripesFilter (via MockServletContext) when a utest needs > to access the configuration. > > This StripesFilter.getConfiguration() thing is pretty abscure to me, so again > please tell me why it's done like this if you have any idea... I'd quite like > to remove all this for a simpler mechanism... > > Cheers > > Remi
------------------------------------------------------------------------------ The Go Parallel Website, sponsored by Intel - in partnership with Geeknet, is your hub for all things parallel software development, from weekly thought leadership blogs to news, videos, case studies, tutorials, tech docs, whitepapers, evaluation guides, and opinion stories. Check out the most recent posts - join the conversation now. http://goparallel.sourceforge.net/
_______________________________________________ Stripes-development mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/stripes-development
