Tickets :: Comment on [#6861] yaml dumper should quote values with comments

Search Results: 4 of 248 [ <<First <Prev Next> Last>> ] [ Return to Search Results ]

* Your Email Address
* Spam protection	Enter the letters below: \ /.__ . . . __. >< \| \\|\ \| \|(__ / \\|__/\| \\|\__\|.__)
Comment	> I just figured out why the tests were failing for me. I have syck > installed, and just realized that Horde calls syck, which outputs > different YAML than Horde/Yaml does. When I disabled syck temporarily > all of the tests pass. > > > > FWIW, you might consider making your tests do round-trip tests like so: > > > > $this->assertEquals($inputPhpStruct, load(dump($inputPhpStruct))) > > > > Also you might consider an option to "disable" syck use and use this > option for the tests to ensure that testing is actually testing the > Horde/Yaml code! > > > > Thanks for accepting the patch. > > > > Alan
Attachment
Watch this ticket