Replying via email since I can't seem to find this comment on github...

If you're referring to the fact that there's a missing .isTrue() at the 
end of that line, I fixed that yesterday.

Or are you questioning the purpose of the test?

-Andy

Christian Edward Gruber wrote:
>
> In core/src/test/java/dagger/internal/BindingTest.java:
>
> > +
> > +import static org.fest.assertions.Assertions.assertThat;
> > +import org.junit.Before;
> > +import org.junit.Test;
> > +
> > +public final class BindingTest {
> > + private Binding<String> wrappedBinding;
> > + private Binding<String> singletonBinding;
> > +
> > + @Before public void setUp() {
> > + wrappedBinding = new StringBinding();
> > + singletonBinding = Linker.scope(wrappedBinding);
> > + }
> > +
> > + @Test public void testSingletonBindingIsSingleton() {
> > + assertThat(singletonBinding.isSingleton());
>
> This feels like an error-prone check or something.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/square/dagger/pull/170/files#r3174236>.