I'd get rid of these constants. Symbols in Ruby are immutable values and it would more idiomatic to just use the symbols directly in your code (ex. have your test method return `:good` instead of `IndexResult::GOOD`).

You would only want to use constants if they pointed to a meaningful value (like a numeric score value or something). Even then, there's a slightly more Ruby way to do it.

If you're goal is to accomplish the Ruby equivalent of an enum you could do something like this:

```ruby
RESULT = {
  :good => 85.0,
  :bad => 55.0,
  ...
}

RESULT[:good] #=> 85.0
```
All that said, it might be worth re-evaluating what your evaluator result model contains in general. It would be good to contain index recommendations, the offending index or query operation or some sort of scored value rather than a somewhat arbitrary value like `:good`.