Good catch. I think for clarity it might be good to do something like `data = if ::File.directory?(...)`. Then both can return a value for that, if it does exist, based on the entries, if it doesn't an empty array. Then both can just do the load. That seems a little clearer, at least in terms of intention, to me. What do you think?