What else would `pending` be besides a string?  `nil`?  If it's just a string or nil this could be simplified to:

``` ruby
reason = pending || RSpec::Core::Pending::NO_REASON_GIVEN
```

If other values (besides a string or nil) get passed, it seems odd to ignore them and odd to pass them and maybe we should do something about that...?