1) printing to the console in the tests:I did it because there is no other
way to verify that the encryption is happening correctly since it's random.
This can be removed if we don't want to show we tested that. Let me know

2) the extra line can be added bit please add it to the coding standards

3)array list: I don't understand your question Tbh, what is the issue with
generate keys?
On 2013-05-26 4:51 PM, "williamhc" <notifications@github.com> wrote:

> In tests/models/PuzzleTests.java:
>
> > +
> > +import junit.framework.Test;
> > +import junit.framework.TestSuite;
> > +
> > +public class PuzzleTests
> > +{
> > +	public static TestSuite suite;
> > +
> > +    public static Test suite()
> > +    {
> > +        suite = new TestSuite("Puzzle tests");
> > +      //  suite.addTestSuite(CalculateGPATest.class);
> > +      //  suite.addTestSuite(CalculateGPA2Test.class);
> > +       return suite;
> > +    }
> > +}
>
> It's good practice to have an empty line at the end of every text file.
> Because stupid stuff<http://stackoverflow.com/questions/2287967/why-is-it-recommended-to-have-empty-line-in-the-end-of-file>.
>
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/williamhc/PG13/pull/5/files#r4395226>
> .
>