This is a good solution. The only thing I would change is not reassigning the sorted lists to s1, s2. Instead, assign them to new l1, l2 variables.

Approved.

Duplicate issue below

Clever use of the Set equals method.

256 looks correct to me.

Here is a TypeScript translation for review: https://www.codewars.com/kumite/61ae28da87f3400033383e2f?sel=61ae28da87f3400033383e2f

I made the first change (size in the ArrayList constructor). I'm not too worried about the second change, since this code isn't seen by the user. It's just to verify the tests are correct.

I am understanding your code :)

Very clean with the comments :)

2 suggestions

• Specify the size of the ArrayList in its constructor
• Inspired by some of the JS solutions, there's this idea using StringBuilder

   public String[] rotate(final String text) {
      final int len = text.length();
      final List<String> rotations = new ArrayList<>(len);
      final StringBuilder lastRotation = new StringBuilder(text);
      for (int i = 0; i < len; i++) {
         lastRotation.append(lastRotation.charAt(0)).deleteCharAt(0);
         rotations.add(lastRotation.toString());
      }
      return rotations.toArray(new String[len]);
   }

The should contain ORDER BY species test is a little overly zealous. Most databases (including PostgreSQL) support ordering by the column number. So ORDER BY 6 would be valid (if column order can be guaranteed), as well as ORDER BY species.

That is not the definition of Camel Case. If you want to restrict your definition to lower case letters first then you should clarify "lower camel case". However, "camel case" is by definition ambiguous.

https://en.wikipedia.org/wiki/Camel_case

By definition the first letter is lowercase in camelCase.