Fixed some UI tests related to hw_model, p_table and hostgroups.

* Updated data as per tests.
* Drop length arguments to `gen_*` methods. They produce 10 chars
  long strings by default
* Removed skips related to old bugs.
* Many other miscellaneous fixes.
We are using variable keyword args in ui/factory. So I think it's fine to use name=name here in such instances.  **kwargs