Right.  Just like +.  That's actually the whole point of that style rule,
right?

On Mon Sep 16 2013 at 6:39:15 PM, Christian Edward Gruber <
notifications@github.com> wrote:

In compiler/src/test/java/dagger/**tests/integration/validation/**
CyclicModuleIncludesTest.java:

> +          "  @Module(includes = Scissors.class)",
> +          "  static class Lizard {}",
> +          "  @Module(includes = Lizard.class)",
> +          "  static class Spock {}",
> +          "}"));
> +
> +  @Test public void cyclicModuleSelfIncludes() {
> +    ASSERT.about(javaSource()).**that(javaFile).processedWith(**daggerProcessors()).**failsToCompile()
> +        .withErrorContaining("**CyclicModules.**SelfReferencingModule includes itself directly")
> +            .in(javaFile).onLine(4);
> +  }
> +
> +  @Test public void cyclicModuleIncludes_full_**cycle() {
> +    ASSERT.about(javaSource()).**that(javaFile).processedWith(**daggerProcessors()).**failsToCompile()
> +        .withErrorContaining("0. CyclicModules.Rock included by CyclicModules.Paper")
> +            .in(javaFile).onLine(6).and()

Ok, Ill buy that, but it doesn't help this case:

ASSERT.about(javaSource()).**that(javaFile).processedWith(**daggerProcessors()).**failsToCompile()
    .withErrorContaining("0. CyclicModules.Rock included by
CyclicModules.Paper")
        .in(javaFile).onLine(6).and()
    .withErrorContaining(...)
        .in(javaFile).onLine(6).and()
    .withErrorContaining(...)
        .in(javaFile).onLine(6).and()

just becomes

ASSERT.about(javaSource()).**that(javaFile).processedWith(**daggerProcessors())
    .failsToCompile()
    .withErrorContaining("0. CyclicModules.Rock included by
CyclicModules.Paper")
        .in(javaFile).onLine(6).and()
    .withErrorContaining(...)
        .in(javaFile).onLine(6).and()
    .withErrorContaining(...)
        .in(javaFile).onLine(6).and()

There's still no initial and() Adding and() to the start would just do this:

ASSERT.about(javaSource()).**that(javaFile).processedWith(**daggerProcessors())
    .failsToCompile()
    .withErrorContaining("0. CyclicModules.Rock included by
CyclicModules.Paper")
        .in(javaFile).onLine(6)
    .and().withErrorContaining(...**)
        .in(javaFile).onLine(6)
    .and().withErrorContaining(...**)
        .in(javaFile).onLine(6)

So the first withError... ends up mis-aligned with the second and third
one.

—
Reply to this email directly or view it on
GitHub<https://github.com/google/dagger/pull/11/files#r6393037>
.