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

* 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