feat: add schemaLocation if present

[HassanAkbar]

Automatically support xsi:schemaLocation for all XML elements if present in XML

closes #64

[ronaldtse]

@HassanAkbar I've added a README and some specs to test for multiple schemaLocations. As you can see here, there could be multiple schemaLocations, and we need to support that. Can you help revise and update the feature/specs?

[HassanAkbar]

@ronaldtse I've updated the code according to the specs you added, I just changed how the schemaLocation object was being created in the test cases that you added.
Can you check that and let me know if that is correct or do I need to change that? Thanks

[ronaldtse]

Somehow this is failing...

class Ceramic < Lutaml::Model::Serializable
  attribute :type, :string
  attribute :glaze, :string
  attribute :color, :string

  xml do
    root 'Ceramic'
    namespace 'http://example.com/ceramic', 'cera'
    map_element 'Type', to: :type, namespace: :inherit
    map_element 'Glaze', to: :glaze
    map_attribute 'color', to: :color, namespace: 'http://example.com/color', prefix: 'clr'
  end
end

xml = <<~HERE
<cera:Ceramic
  xmlns:cera="http://example.com/ceramic"
  xmlns:clr="http://example.com/color"
  xsi:schemaLocation=
    "http://example.com/ceramic http://example.com/ceramic.xsd
     http://example.com/color http://example.com/color.xsd"
  clr:color="navy-blue">
  <cera:Type>Porcelain</cera:Type>
  <Glaze>Clear</Glaze>
</cera:Ceramic>
HERE

Ceramic.from_xml(xml)

@HassanAkbar any idea?

[ronaldtse]

Actually it works! Sorry!!

[ronaldtse]

No actually it doesn't?

$ bundle console
# ...
require_relative "../lib/lutaml/model/xml_adapter/nokogiri_adapter"
Lutaml::Model::Config.configure do |config|
  config.xml_adapter_type = :nokogiri
end

class Ceramic < Lutaml::Model::Serializable
  attribute :type, :string
  attribute :glaze, :string
  attribute :color, :string

  xml do
    root 'Ceramic'
    namespace 'http://example.com/ceramic', 'cera'
    map_element 'Type', to: :type, namespace: :inherit
    map_element 'Glaze', to: :glaze
    map_attribute 'color', to: :color, namespace: 'http://example.com/color', prefix: 'clr'
  end
end

xml_content = <<~HERE
<cera:Ceramic
  xmlns:cera="http://example.com/ceramic"
  xmlns:clr="http://example.com/color"
  xsi:schemaLocation=
    "http://example.com/ceramic http://example.com/ceramic.xsd
     http://example.com/color http://example.com/color.xsd"
  clr:color="navy-blue">
  <cera:Type>Porcelain</cera:Type>
  <Glaze>Clear</Glaze>
</cera:Ceramic>
HERE

a = Ceramic.from_xml(xml_content)
a.schema_location
# => nil

[ronaldtse]

@ronaldtse I've updated the code according to the specs you added, I just changed how the schemaLocation object was being created in the test cases that you added. Can you check that and let me know if that is correct or do I need to change that? Thanks

The tests are just illustrative but the way it works seems fine to me.

[HassanAkbar]

No actually it doesn't?

$ bundle console
# ...
require_relative "../lib/lutaml/model/xml_adapter/nokogiri_adapter"
Lutaml::Model::Config.configure do |config|
  config.xml_adapter_type = :nokogiri
end

class Ceramic < Lutaml::Model::Serializable
  attribute :type, :string
  attribute :glaze, :string
  attribute :color, :string

  xml do
    root 'Ceramic'
    namespace 'http://example.com/ceramic', 'cera'
    map_element 'Type', to: :type, namespace: :inherit
    map_element 'Glaze', to: :glaze
    map_attribute 'color', to: :color, namespace: 'http://example.com/color', prefix: 'clr'
  end
end

xml_content = <<~HERE
<cera:Ceramic
  xmlns:cera="http://example.com/ceramic"
  xmlns:clr="http://example.com/color"
  xsi:schemaLocation=
    "http://example.com/ceramic http://example.com/ceramic.xsd
     http://example.com/color http://example.com/color.xsd"
  clr:color="navy-blue">
  <cera:Type>Porcelain</cera:Type>
  <Glaze>Clear</Glaze>
</cera:Ceramic>
HERE

a = Ceramic.from_xml(xml_content)
a.schema_location
# => nil

@ronaldtse This is happening because the xmlns:xsi is not defined and nokogiri is treating xis:schemaLocation as a attribute name and not xsi as prefix and schemaLocation as attribute name which we are expecting.

I'll update the code to handle this scenario.

[ronaldtse]

Ahh @HassanAkbar yes you are right. We need to define the namespace xsi first. My example XML was mistaken. It’s fine then.

[HassanAkbar]

@ronaldtse Then I think this is done. Is there something else left to be done in this PR?

[ronaldtse]

This is all good, I've tested it with the (updated) sample in the README. Merging once tests pass.