Re: [Freeipa-devel] [PATCH 0004] [Test] Test for caacl-add-service: incorrect error message when service does not exists
On 18.08.2016 15:02, Tomas Krizek wrote: Hi, NACK. The issue is not that the error message contains the "no such entry" string. That is actually a valid part of the error message if the service indeed does not exist. The problem is that the error message contained only the first character of the service's name instead of the entire name of the service. For example, the command ipa caacl-add-service test_caacl --services svc/master2.ipa.test --services svc/master1.ipa.test should end with these error messages if those services do not exist: member service:svc/master2.ipa.t...@ipa.test: no such entry member service:svc/master1.ipa.t...@ipa.test: no such entry There is also a typo in the file name of the test and I'm also not sure if there isn't a better place to put the test. +1 This should not be integration_test but only test_xmlrpc Martin^2 On 08/18/2016 01:48 PM, Ganna Kaihorodova wrote: Hello! Test for caacl-add-service: incorrect error message when service does not exists https://fedorahosted.org/freeipa/ticket/6171 Best regards, Ganna Kaihorodova Associate Software Quality Engineer : -- Tomas Krizek -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code
Re: [Freeipa-devel] [PATCH 0004] [Test] Test for caacl-add-service: incorrect error message when service does not exists
Hi, NACK. The issue is not that the error message contains the "no such entry" string. That is actually a valid part of the error message if the service indeed does not exist. The problem is that the error message contained only the first character of the service's name instead of the entire name of the service. For example, the command ipa caacl-add-service test_caacl --services svc/master2.ipa.test --services svc/master1.ipa.test should end with these error messages if those services do not exist: member service: svc/master2.ipa.t...@ipa.test: no such entry member service: svc/master1.ipa.t...@ipa.test: no such entry There is also a typo in the file name of the test and I'm also not sure if there isn't a better place to put the test. On 08/18/2016 01:48 PM, Ganna Kaihorodova wrote: Hello! Test for caacl-add-service: incorrect error message when service does not exists https://fedorahosted.org/freeipa/ticket/6171 Best regards, Ganna Kaihorodova Associate Software Quality Engineer : -- Tomas Krizek -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code