Tree building sorts children by name so we must name our storages.

See build_smartproxy_affinity_node:
https://github.com/ManageIQ/manageiq/blob/e405e1ad52072a0c7f341481d6457841e849450e/vmdb/app/controllers/ops_controller/settings/common.rb#L1272

This fixes a sporadic test failure where the storages are in unexpected order because they have no name.
I think we should update all of our factories that use sequences for names to pad with 0s.  Maybe have a factory helper method.

I'm thinking something like

```ruby
# spec/support/factory_girl_helper.rb
def seq_padded_for_sorting(n)
  n.to_s.rjust(13, '0') # pad to 13 for max region size
end

# spec/factories/storage.rb

  factory :storage do
    sequence(:name) { |n| "storage_#{seq_padded_for_sorting(n)}" }
  end
```

Thoughts?  I'm wondering if other sporadic failures are caused by this.