Thanks everyone for the input.

I will spend my Friday working on this project.

About the number of asserts per test: my understanding is that we want a
single logical assert, so we should not be put off by multiple assert lines.

So long assertion is after arrangement and action, of course.

I will review on Friday.

Thanks again!
On 2013-11-26 8:01 AM, "Felix Müller" <notifications@github.com> wrote:

> In
> src/test/java/org/sonar/plugins/scala/cobertura/ScalaCoberturaParserTest.java:
>
> > +import org.sonar.plugins.scala.language.ScalaFile;
> > +import org.sonar.plugins.scala.language.ScalaPackage;
> > +
> > +public class ScalaCoberturaParserTest {
> > +
> > +    private ScalaCoberturaParser underTest;
> > +
> > +    @Before
> > +    public void setUp() {
> > +        underTest = new ScalaCoberturaParser();
> > +    }
> > +
> > +    @Test
> > +    public void should_create_ScalaFile_resource_when_deep_package() {
> > +        Resource resource = underTest.getResource("com.mock.scalapackage.MockScalaClass");
> > +        assertNotNull(resource);
>
> I go with gshakhn. Try to use camel case. Furthermore, try to code only
> one assert pert test. When there are more, it's probably better to split
> one test method into many.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/SonarCommunity/sonar-scala/pull/2/files#r7928584>
> .
>