Why the string check? It doesn't seem to be covered by a test.