I think it should be documented, at least so that tempest can test things that are actually known and documented behaviors
Thanks, Shoham On Wed, Mar 23, 2016 at 1:07 PM, Sean Dague <[email protected]> wrote: > On 03/22/2016 04:44 PM, Shoham Peller wrote: > > Thank you for your answer Sean. Well understood. > > However, I think that if not fix this issue, at least we should supply > > guidelines on the matter. > > > > For example tempest's "test_delete_security_group_without_passing_id" > > test actually checks this behavior by addressing a security group with > > an "empty id" (/v2.1/{tenant_id}/os-security-groups/), and expects nova > > to return 404. The comment on the code says: > > # Negative test:Deletion of a Security Group with out passing ID should > > Fail > > > > If we'll release guidelines on the matter, and the right way to use the > > API, and that the trailing slashes issue is just a known bug as you say, > > this test is rendered useless. > > > > What do you think? > > I think it's known behavior, I'm still not convinced it's a bug. It's > the way we document that things work. If you feel we need to be more > explicit in documentation about it, I think that would be appropriate. > > -Sean > > -- > Sean Dague > http://dague.net > > __________________________________________________________________________ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: [email protected]?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev >
__________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: [email protected]?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
