nit: we prefer "expected foo, found bar" to "expected foo but found bar"