Hi everyone, First of all, congratulations for all the great work you are doing! I hope to see the S4 community grow fast and healthy.
I just wanted to give a suggestion on an issue. > - test package names: I started using prefixing by test.s4, in order to > avoid any confusion with other classes from code. I'm ok for prefixing > tests > with org.apache.s4, but I think it should at least be org.apache.s4.test. > Which naming scheme do we choose? > > In my experience, putting tests in the same package as the class being tested helps a lot with interface design, i.e. you don't need to make a method public to make it testable but you can use the default package visibility and yet not include it in the package-level API. This makes it cleaner what the API itself is. An easy way to separate tests and source code is to have a /src subdirectory and a /test subdirectory with mirrored source trees below (so you have src/org/apache/s4/ and test/org/apache/s4/ and classes in these two locations are in the same package). Hope this helps. Cheers, -- Gianmarco De Francisci Morales
