I agree on that one, let me know when you finish reviewing to fix all,
thanks.
On Feb 23, 2014 7:25 PM, "Carlos Miranda" <notifications@github.com> wrote:

> In src/test/java/com/jcabi/github/RtPublicKeysITCase.java:
>
> > @@ -113,4 +111,32 @@ public final void removesKey() throws Exception {
> >          );
> >      }
> >
> > +    /**
> > +     * RtPublicKeys should be able to create a key.
> > +     *
> > +     * @throws Exception If a problem occurs.
> > +     */
> > +    @Test
> > +    public final void createsKey() throws Exception {
> > +        final PublicKeys keys = this.keys();
> > +        final PublicKey key = keys.create("rsa", "rsa sh");
> > +        MatcherAssert.assertThat(
> > +            keys.iterate().iterator().next(),
> > +            Matchers.equalTo(key)
>
> This assertion will fail if there is more than one public key on the repo
> you are testing on, I don't think that's a safe assumption. Better to use
> Matchers.hasItem() here I think.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/jcabi/jcabi-github/pull/554/files#r9978686>
> .
>