Item8638: Add QUERY macro

pencil
Priority: Enhancement
Current State: Closed
Released In: 1.1.0
Target Release: minor
Applies To: Engine
Component:
Branches:
Reported By: CrawfordCurrie
Waiting For:
Last Change By: KennethLavrsen
May be reverted if it doesn't pass 14-day muster.

-- CrawfordCurrie - 28 Feb 2010

Removed the hard-coded dependency on JSON.pm. If we really need JSON, we need to add it to the DEPENDENCIES, and/or the CPAN libs. For now, one single test fails without it (the JSON one). Should make this one expected failure, but...

-- OlivierRaginel - 02 Mar 2010

I changed the Fn_QUERY test because it fails locally after I installed JSON.pm. I am not entirely comfortable with using JSON.pm to test the JSON produced by JSON.pm. For this reason - I've marked this for your feedback, Crawford. Are you happy with the updated test?

-- MichaelTempest - 22 Aug 2010

Yes, certainly, no problem. We are not testing JSON.pm, we are testing the code that uses JSON.pm. The core code could equally use a different JSON module, or even boil it's own - but the test doesn't know, or care, about that.

-- CrawfordCurrie - 23 Aug 2010

 
Topic revision: r14 - 04 Oct 2010, KennethLavrsen
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License    Legal Imprint    Privacy Policy