Change to_xml method options in the default error_formatter

[dpsk]

I think it's better to have something like this

  <?xml version="1.0" encoding="UTF-8"?>
  <error>
    <message>Contact not found</message>
  </error>

instead of:

  <?xml version="1.0" encoding="UTF-8"?>
  <hash>
    <error>Contact not found</error>
  </hash>

hash as root looks a little off.

[dblock]

I think it makes sense.

• Please add tests. Looks like we didn't have any tests for the xml output, bad on us.
• Add an entry in CHANGELOG.
• Possibly add something on XML format to README - it's not much used, so it's good to see people who actually use it make it better and document it properly.

[dpsk]

Okay, i will add these changes a bit later.

[dpsk]

I'm added specs and changelog entry. I didn't have anything to add to the documentation to be honest.
As for the specs, inside of the formatter to_s method is used, instead of to_xml.
For using to_xml methods in specs we need to require active_support/core_ext and add builder.
/~https://github.com/lifo/docrails/blob/master/activesupport/lib/active_support/builder.rb

Oh, to_s on hash instance behaves differently in 1.8 and 1.9 :(

[dblock]

I am confused. How is this XML?

[dpsk]

The object don't respond to the to_xml method, what you see is the output of to_s. For testing to_xml we need to add active_support and builder(/~https://github.com/lifo/docrails/blob/master/activesupport/lib/active_support/builder.rb) into dependencies, the question is should we?

[dblock]

Well, if we want to test XML, we definitely should!

[dpsk]

Sorry for al those messy commits. Now we have proper xml in the test and successfull build.

[dblock]

This is good, thanks, merging.

[coveralls]

Changes Unknown when pulling 7984e1f on dpsk:master into * on intridea:master*.

[coveralls]

Changes Unknown when pulling 7984e1f on dpsk:master into * on intridea:master*.

[coveralls]

Changes Unknown when pulling 7984e1f on dpsk:master into * on intridea:master*.