Yes. `#annotation_provider` is now uses `@provider` instance variable to return value (this would bypass a DB call in `AnnotationProvider.authenticate_provider` on all but the initial call). So, without `attr_reader` `#annotation_provider` returns `nil`, and we won't be able to find annotations.

Specs use a factory, and the failures would appear slightly different:
```
  1) Travis::Services::UpdateAnnotation when annotation is enabled creates the annotation if it doesn't exist already
     Failure/Error: lambda { @annotation = service.run }.should issue_queries(7)
     NameError:
       undefined local variable or method `annotations' for #<AnnotationProvider:0x007fbe4440f100>
```