Traits fixtures use a target that is not RFC 9535 (`@.x-oai-traits`)
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 84/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- javascript, yaml
- 領域
- testing
調査の方向性
tests/v1.0/pass/actions-traits-example.yaml、tests/v1.1/pass/actions-traits-example.yaml、tests/v1.2-dev/pass/actions-traits-example.yamlのtargetエントリを更新し、仕様の修正されたtraitsの例に一致させてください。まず、これらのfixtureとissue内の例を比較し、その後、fixtureに対してスキーマテストスイートを実行してください。3つすべてが修正されたtargetを使用し、期待される操作を選択すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
The traits fixtures in the schema test suite use a target that is not a valid RFC 9535 query:
tests/v1.0/pass/actions-traits-example.yamltests/v1.1/pass/actions-traits-example.yamltests/v1.2-dev/pass/actions-traits-example.yaml
- target: $.paths.*.get[[email protected]]
In RFC 9535, a member name in shorthand notation cannot contain - (section 2.5.1.1: name-char = name-first / DIGIT, and name-first excludes -), so @.x-oai-traits is a syntax error; it would have to be @['x-oai-traits']. The fixtures pass because the schema only checks pattern: ^\$ on target. But an implementation that follows the "RFC9535 Compliance" section ("MUST fully implement RFC9535") has to reject these overlays, so the "pass" fixtures cannot be used to test tools.
Even in bracket notation, the target would filter the members of each get operation, not the operations themselves, and paged is an array element, not a member. #229 fixed the same example in the specification text (now in 1.1.0 and 1.2.0) to:
- target: $.paths[?(@.get['x-oai-traits'][?(@ == 'paged')])].get
Suggestion: use that target in the three fixtures, so that they match the specification's traits example.
Found while running the fixtures against an RFC 9535 implementation that passes the JSONPath Compliance Test Suite; the corrected target parses and selects the expected operations on the specification's example description.
- 主要言語
- JavaScript
- スター
- 121
- フォーク
- 38
- 平均マージ
- 1日 17時間
- マージ済み PR（30日）
- 9
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
OAI/Overlay-Specification のほかの issue
-
ci: update the agenda template to lead us down the path of checking the spec repo対応中かも @baywet が 7 日前に担当しました。 オープン
OAI/Overlay-Specification#411 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
ci: add automation to sync from main -> dev -> vX.Y-dev branches対応中かも @baywet が 7 日前に担当しました。 オープン
OAI/Overlay-Specification#410 · コメント 1 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
Revisit: should Overlays declare their target document format? (follow-up to #268)対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
OAI/Overlay-Specification#367 · コメント 3 件 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
OAI/Overlay-Specification#358 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
OAI/Overlay-Specification#299 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
OAI/Overlay-Specification の issue をすべて見る
似ている issue
-
Notifications pagination accepts NaN and negative LIMIT/OFFSET対応中かも @Senthil455 が今日担当しました。 オープンBUGS MEDIUM
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
elixpo/blogs.elixpo#419 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
immich-app/immich#32426 ·
メンテナーはふだん 1 日以内に返信
-
automated issue report
難易度 1/5 1時間未満 初心者へのやさしさ 70/100
lirantal/discoprint#45 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
scanaislop/aislop#482 ·
メンテナーはふだん 1 日以内に返信