[Freeipa-devel] [PATCH] Added error handler for ipa_cmd().

Adam Young ayoung at redhat.com
Wed Sep 29 21:05:46 UTC 2010


On 09/29/2010 04:53 PM, Adam Young wrote:
> On 09/29/2010 03:17 PM, Endi Sukma Dewata wrote:
>> ----- "Adam Young"<ayoung at redhat.com>  wrote:
>>
>>> Endi, can you add in a Unit test for this? If need be, extend the
>>> ipa_cmd so that it looks for an optional command line param that makes
>>> it try to fetch a .json file that doesn't exist:
>>>
>>>
>>> Something like
>>> var suffix = $.bbq.get("cmd_suffix");
>>> if (suffix){
>>> method += suffix;
>>> }
>>> method += '.json';
>>>
>>> ...
>> Attached is a new patch based on the latest master with unit tests.
>> ipa_cmd() error can be simulated by overriding $.ajax with a mockup
>> function. Thanks!
>>
>> -- 
>> Endi S. Dewata
> ACK
>
> _______________________________________________
> Freeipa-devel mailing list
> Freeipa-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/freeipa-devel
Pushed to master




More information about the Freeipa-devel mailing list