Rexml or Nokogiri?
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 25/100
調査の方向性
Start with lib/dbus/xml.rb and spec/bus_and_xml_backend_spec.rb, especially the test case that switches to Nokogiri without restoring the backend. Run the affected specs and inspect the reported failures to establish the supported parser behavior; done means the parser configuration and dependency behavior are decided and the test suite is consistent with that decision.
索引モデルが issue の本文から書いたものです。
説明
PR #87 introduced hard dependency on ReXML. However, the irony is that the Nokogiri is actually preferred and the whole test suite is executed against Nokogiri. On top of that, if Nokogiri is not available, there are ~170 test failures such as:
2) DBus::Service when a private bus is set up #exists? is true for an existing service
Failure/Error: @doc = Nokogiri.XML(xml)
NameError:
uninitialized constant DBus::IntrospectXMLParser::NokogiriParser::Nokogiri
# ./lib/dbus/xml.rb:81:in `initialize'
# ./lib/dbus/xml.rb:127:in `new'
# ./lib/dbus/xml.rb:127:in `parse'
# ./lib/dbus/proxy_object_factory.rb:33:in `introspect_into'
# ./lib/dbus/proxy_object.rb:80:in `introspect'
# ./spec/bus_driver_spec.rb:20:in `block (4 levels) in <top (required)>'
# ./spec/spec_helper.rb:119:in `with_service_by_activation'
# ./spec/bus_driver_spec.rb:11:in `block (4 levels) in <top (required)>'
# ./spec/spec_helper.rb:95:in `block (2 levels) in with_private_bus'
# ./spec/spec_helper.rb:66:in `with_env'
# ./spec/spec_helper.rb:94:in `block in with_private_bus'
# ./spec/spec_helper.rb:66:in `with_env'
# ./spec/spec_helper.rb:77:in `with_private_bus'
# ./spec/bus_driver_spec.rb:10:in `block (3 levels) in <top (required)>'
as soon as the BusAndXmlBackendTest tests introspection reading nokogiri test case is executed, because it sets the backend to Nokogiri without reseting it back to the original value:
I'd like to clarify this situation.
If both backends are really equal, then the two test cases should be probably merged into one, where the XML backend is defined by the available package set (or by some env variable) and the test suite should be run twice, once with Nokogiri and the other time with ReXML.
It also is suboptimal to always pull in ReXML gem, when Nokogiri is going to be used. Ideally, there should be no implicit dependency. Maybe the default behavior should be to just gracefully fail if there is no XML parser available. Or only one XML parser could be supported ...
- 主要言語
- Ruby
- スター
- 174
- フォーク
- 51
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
mvidner/ruby-dbus のほかの issue
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
-
NetworkManager.Settings.Connection cannot round-trip, calling Update with what GetSettings returnedオープン
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
-
good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 45/100
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
-
UNIX_FD in practice?オープン
難易度 4/5 3〜5日 初心者へのやさしさ 32/100
mvidner/ruby-dbus の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
難易度 2/5 半日 初心者へのやさしさ 72/100
TheOdinProject/curriculum#31479 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
taskcluster/taskcluster#9284 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Class option relations are appended to the command on every run対応中かも @auraseb が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100