It's a tiny detail, but (honest question) wouldn't it be better to check if the return value is not None? Checking for len() reminds me too much of Java.

```clj
(assert (not (is None (read-file "README.md"))))
```