search

quattor / CCM

Configuration Cache Manager
www.quattor.org
Other
4 stars 13 forks source link

TextRender: add support for jsonpretty type #186

Closed kwaegema closed 6 years ago

kwaegema commented 6 years ago

Requires quattor/CAF#265

jrha commented 6 years ago

@kwaegema can you address @stdweird's comment?

jrha commented 6 years ago 
Failed test 'Correct JSON pretty rendered'
at src/test/perl/textrender.t line 72.
       got: ''
  expected: '{XY"a"Z"a",XY"b"Z"1",XY"c"Z1,XY"d"Ztrue,XY"e"Zfalse,XY"f"Z1.5,XY"g"Z[XYY"g1",XYY"g2"XY],XY"h"Z{XYY"a"Z"a",XYY"b"Z"1",XYY"c"Z1,XYY"d"Ztrue,XYY"e"ZfalseXY}X}X'

Doesn't look right to me...

stdweird commented 6 years ago

@jrha @kwaegema is on holiday till july 8th or so. i can try to get this resolved, but we have an urgent deadline for the openstack stuff.

jrha commented 6 years ago

Sure, no worries. Just wanted to make sure the error wasn't lost.

kwaegema commented 6 years ago

@jrha @stdweird comment addressed, fixes check