I can see what's happening. If you kept `build` as `build`, the diff would've been smaller. That was all.

On a second thought, these names pose a more serious problem; they are very non-descriptive. What is the distinguishing feature of `build` and `build2`? It is not clear from the name.

How about:

```ruby
let!(:failed_build)  { Factory(:build, repository: repo, request: request, state: :failed) }
let!(:passed_build)  { Factory(:build, repository: repo, request: request, state: :passed) }
```