This return is unreachable.
Merge pull request #562 from martinpovolny/user_wo_group_fix

Creating user: tests, cleanups.
Merge pull request #580 from mzazrivec/none_storage_type_display_fix

Fix error ocurring when displaying storage with null storage type
Merge pull request #583 from blomquisg/bz1140191-security-groups-404

Handle Openstack 404 Errors
These old ruby 1.8.7 based fleecing tests are unused.
Merge pull request #587 from romanblanco/erb2haml_ui_5

Code readability changes.
Remove the GC disable/enable in the before/after blocks.

It's not clear why were were doing this but the code was implemented in commit:
ff4b5f010ab115a3eec40703b40096ecbe44db34

And refined in commit: 3e637725c2c2ce1fea69c5a1491046370f8d7408, with a message including:
"...Hopefully this doesn't reintroduce the GC segfault."

I believe this was added back when we were using ruby 1.8.7 and possibly libxml-ruby/rexml with soap4r.  Since we're way beyond 1.8.7 with 1.9.3 and soon 2.0, we should remove this and fix it if it re-occurs.
Cap & U table styling fix

added no-hover class to Cap & U tables, removed fieldsets

https://bugzilla.redhat.com/show_bug.cgi?id=1135659
VMWareMetadata is not called anywhere.

It looks like it was last modified for conversion to binary_struct here: 00a7dd423c738574008fe1895a5e19f27e32ac98

And the last substantial change was in 2009 here:
05d517b984ea1ecd89793063ece88ebd0be97927

The test contains invalid ruby 1.9 syntax, see below, and also contains binary string literals that may cause issues in ruby 2.0+.

```
lib/metadata/VMWare/test/vmdk.rb:114:13: E: unexpected token tCOLON
    when   7: return "NTFS"
            ^
lib/metadata/VMWare/test/vmdk.rb:115:5: E: unexpected token kWHEN
    when  12: return "FAT32"
    ^^^^
lib/metadata/VMWare/test/vmdk.rb:116:5: E: unexpected token kWHEN
    when 131: return "ext2"
    ^^^^
lib/metadata/VMWare/test/vmdk.rb:119:1: E: unexpected token kEND
end
^^^
lib/metadata/VMWare/test/vmdk.rb:190:18: E: unexpected token tCOLON
    when "SPARSE":  parseDiskSparse(disk)
                 ^
lib/metadata/VMWare/test/vmdk.rb:191:5: W: else without rescue is useless
    else            raise "Unrecognized Disk Type - #{disk["type"]}"
    ^^^^
lib/metadata/VMWare/test/vmdk.rb:193:1: E: unexpected token kEND
end
^^^
```
Corrected name of test file
Snapshot creation disabled for Templates

Implemented the supports_snapshots? method

https://bugzilla.redhat.com/show_bug.cgi?id=1005285
Merge pull request #609 from romanblanco/erb2haml_ui_5

Corrected name of test file
Merge pull request #608 from jrafanie/remove_unused_code_and_1_9_incompatible_test

VMWareMetadata is not called anywhere.
Merge pull request #606 from jrafanie/remove_gc_disable

Remove the GC disable/enable in the before/after blocks.
Merge pull request #586 from tinaafitz/manageiq_system_policy_model_change

ManageIQ domain - Correct /system/policy instance relationship values.
Added supports_snapshots? method to base/sub class.

- Added supports_snapshots? method to base  class that returns false and added supports_snapshots? to VmVmware subclass to return true as that allows snapshot functionality for VM .
- Updated hover text for buttons to show 'Revert' in the hover text for Revert button instead of 'Remove', and changed word 'Operations' to be lowercase in the hover text message.

https://bugzilla.redhat.com/show_bug.cgi?id=1141174
https://bugzilla.redhat.com/show_bug.cgi?id=1141375
added no-hover and row0 row1 classes to vm disk table

https://bugzilla.redhat.com/show_bug.cgi?id=1139076
SCVMM - VM vendor property should be set to 'microsoft'

the vendor icon was being parsed from the OS table on the VM. Instead, it should reflect the vendor of the provider.
converted tabs to spaces

https://bugzilla.redhat.com/show_bug.cgi?id=1139076
Addressed rubocop comments.

https://bugzilla.redhat.com/show_bug.cgi?id=1141174
https://bugzilla.redhat.com/show_bug.cgi?id=1141375