6.0.0-alpha10
▾
Tasks
New Task
Search
Photos
Wiki
▾
Tickets
New Ticket
Search
dev.horde.org
Toggle Alerts Log
Help
5/15/25
H
istory
A
ttachments
C
omment
W
atch
Download
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
N
ew Ticket
M
y Tickets
S
earch
Q
uery Builder
R
eports
Saved Queries
Open Bugs
Bugs waiting for Feedback
Open Bugs in Releases
Open Enhancements
Enhancements waiting for Feedback
Bugs with Patches
Enhancements with Patches
Release Showstoppers
Stalled Tickets
New Tickets
Horde 5 Showstoppers