api fixed and methods GET and DELETED added for TemplateCombination

[renzon]

fixes #332

Results:

>>> from nailgun.config import ServerConfig
>>> from nailgun.entities import HostGroup, Environment, ConfigTemplate, TemplateCombination
>>> cfg = ServerConfig('https://foo.bar', 
...                    auth=('baz', '****'), verify=False)
>>> def creat_ent(cls, **kwargs):
...     return cls(cfg, **kwargs)
>>> hostgroup=creat_ent(HostGroup, name='group').create()
>>> env=creat_ent(Environment, name='env').create()
>>> config_template=creat_ent(
...     ConfigTemplate,snippet=False,name='comb',template='cat', template_kind=8,
...     template_combinations=[{'hostgroup_id': hostgroup.id, 'environment_id': env.id}]
... ).create()
>>> combination_dct=config_template.template_combinations[0]
>>> combination_dct
{u'environment_id': 6, u'provisioning_template_name': u'comb', u'environment_name': u'env', u'hostgroup_id': 5, u'config_template_name': u'comb', u'hostgroup_name': u'group', u'provisioning_template_id': 110, u'id': 5, u'config_template_id': 110}
>>> combination=creat_ent(TemplateCombination, id=combination_dct['id'])
>>> combination=combination.read()  # Get working
>>> combination
nailgun.entities.TemplateCombination(nailgun.config.ServerConfig(url='https://foo.bar', verify=False, auth=('baz', '****')), environment=nailgun.entities.Environment(nailgun.config.ServerConfig(url='https://foo.bar', verify=False, auth=('baz', '****')), id=6), config_template=nailgun.entities.ConfigTemplate(nailgun.config.ServerConfig(url='https://foo.bar', verify=False, auth=('baz', '****')), id=110), id=5, hostgroup=nailgun.entities.HostGroup(nailgun.config.ServerConfig(url='https://foo.bar', verify=False, auth=('baz', '****')), id=5))
>>> combination.delete() # Delete working
{u'environment_id': 6, u'provisioning_template_id': 110, u'created_at': u'2016-12-16T12:21:02.687Z', u'updated_at': u'2016-12-16T12:21:02.687Z', u'hostgroup_id': 5, u'id': 5}

[renzon]

OBS: New version of nailgun must be publish after this merge. After publication SatelliteQE/robottelo#4099 can be reviewed and merged as well.

[coveralls]

Coverage remained the same at 98.578% when pulling 7802ba6 on renzon:332 into b5fd3d1 on SatelliteQE:master.

[oshtaier]

class TemplateCombination(
        Entity,
        EntityDeleteMixin 
        EntityReadMixin,
        EntityUpdateMixin):

[oshtaier]

Yeah, only get and delete methods are present using that base URL

[oshtaier]

Please, use direct nailgun calls for testing and providing results. Robotello is redundant layer in that case

[coveralls]

Coverage remained the same at 98.578% when pulling 62099f8 on renzon:332 into b5fd3d1 on SatelliteQE:master.

[coveralls]

Coverage remained the same at 98.578% when pulling 62099f8 on renzon:332 into b5fd3d1 on SatelliteQE:master.

[coveralls]

Coverage remained the same at 98.578% when pulling 62099f8 on renzon:332 into b5fd3d1 on SatelliteQE:master.

[renzon]

@oshtaier normally I would use directly nailgun. But robottelo test was already done. In fact it was the reason I found those bugs on nailgun. So I took the lazy path and just posted the tests using it =P. But if you think directly call is really needed, I can make it ;)

[renzon]

I don't know why is travis failing on docs creation only on Python 3.3.

[svtkachenko]

@renzon If I remember correctly it's because latest sphinx requires python 3.4+

[svtkachenko]

I think usually we put mixins in alphabetical order.

[renzon]

Fixed ;)

[oshtaier]

already wrote about that above:)

[coveralls]

Coverage remained the same at 98.578% when pulling 0620990 on renzon:332 into b5fd3d1 on SatelliteQE:master.

[oshtaier]

Direct call will instantly show whether every mixin works and you really can perform all actions that you like

[oshtaier]

About travis - yes, we had discussion with @omaciel and @rochacbruno about that, but we didn't have any final decision. We have to whether remove 3.3 verification at all or make sphinx static for version when it worked for 3.3

[oshtaier]

TemplateCombination(Entity, EntityDeleteMixin, EntityReadMixin):

[renzon]

@oshtaier I edited #342 (comment) to have nailgun session

[renzon]

@oshtaier @svtkachenko Entities Mixings order fixed

[coveralls]

Coverage remained the same at 98.578% when pulling 2f46d4d on renzon:332 into b5fd3d1 on SatelliteQE:master.

[svtkachenko]

ACK

[abalakh]

LGTM, ACK

[renzon]

@oshtaier please check the nailgun session and changes, merging if you think it is ok. Thanks.