Yep.  Read the comment right before I closed it.

On Mon Sep 16 2013 at 6:46:14 PM, Christian Edward Gruber <
notifications@github.com> wrote:

> In
> compiler/src/test/java/dagger/tests/integration/validation/LibraryModuleTest.java:
>
> > +  }
> > +
> > +  @Test public void unusedProviderMethodsFailOnNonLibrary() {
> > +    JavaFileObject source = JavaFileObjects.forSourceString("Library", Joiner.on("\n").join(
> > +        "import dagger.Module;",
> > +        "import dagger.ObjectGraph;",
> > +        "import dagger.Provides;",
> > +        "import java.lang.Override;",
> > +        "@Module(library = false)",
> > +        "class TestModule {",
> > +        "  @Provides String string() {",
> > +        "    return \"string\";",
> > +        "  }",
> > +        "}"));
> > +    ASSERT.about(javaSource()).that(source).processedWith(daggerProcessors()).failsToCompile()
> > +        .withErrorContaining("Graph validation failed:").in(source).onLine(6).and()
>
> Fair. I can fix that up, but I'd rather fix it up upstream, or separately
> so we can push it back up. (You closed it, so this comment is just more of
> an ACK)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/google/dagger/pull/11/files#r6393169>
> .
>