ruby-grape / grape / 27504305221 / 28

97%

master: 97%

push

github

Detect the MultiXML constant to avoid the multi_xml deprecation (#2765)

multi_xml 0.9.0 deprecated the `MultiXml` constant in favor of `MultiXML`
(removed in v1.0); the legacy constant warns on access/use. Grape detected
the backend via `::MultiXml` and called `Grape::Xml.parse`, so users on
multi_xml >= 0.9 would hit the deprecation.

Detect `::MultiXML` first, falling back to the legacy `::MultiXml` constant
and then `ActiveSupport::XmlMini`. Unlike multi_json, no facade is needed:
`parse` is canonical (not renamed) and Grape only calls `parse`, so a direct
alias suffices on every backend.

Cover both multi_xml lines in CI: pin gemfiles/multi_xml.gemfile to >= 0.9
and add gemfiles/multi_xml_0_8.gemfile (< 0.9), each running the
spec/integration/multi_xml suite. The integration spec now drives a real
Grape API that parses an XML request body through Grape::Xml.parse.

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-authored-by: Daniel (dB.) Doubrovkine <dblock@dblock.org>

1102 of 1225 branches covered (89.96%)

Branch coverage included in aggregate %.

3529 of 3643 relevant lines covered (96.87%)

961.2 hits per line