Automation failure : test_search_permissions

SatelliteQE / robottelo

Robottelo is a test suite that exercises The Foreman.

GNU General Public License v3.0

61 stars 114 forks source link

Closed tkolhar closed 9 years ago

tkolhar commented 9 years ago

TestNames : tests.foreman.api.test_permission.PermissionsTestCase.test_search_permissions

the test says : Items in the second set but not the first: u'Docker/ImageSearch' u'Container' u'DockerRegistry' more details :

the search function is failing please fix it thanks

oshtaier commented 9 years ago

'API' u'Docker/ImageSearch' u'Container' u'DockerRegistry'

refer to 'UI' 'Docker/ImageSearch' 'Docker/Container' 'Docker/DockerRegistry'

'UI' implementation sounds much better and probably should be used as a standard. Should we fix automation as is? Or wait for better naming?

elyezer commented 9 years ago

@oshtaier good point. I think worth opening a bug.

Ichimonji10 commented 9 years ago

@oshtaier FYI, upstream and downstream systems expose different permissions. Keep that in mind when running any permissions-related tests.

elyezer commented 9 years ago

And now we have get_server_software helper which helps us differentiate from upstream and downstream.

Ichimonji10 commented 9 years ago

Thank you for that @elyezer. :smile:

oshtaier commented 9 years ago