listof1 already generates a list so this isn't firstChar -- it's actually nonEmptyListofChars.

```scala
lazy val genNonEmptyString: Gen[String] = Gen.listOf1(Gen.alphaChar).map(_.mkString)
```

Also, note that these should be lazy val's or you can produce scenarios where the generators are null.