htmlパースからnvapiに置き換える

Negima1072 commented 1 year ago

はじめまして、Negima1072と申します。ライブラリの方向性からなるべくnvapi等に置き換えるほうが良いのではと思ったのでPRさせていただきます。

合わせて以下の修正をしております。

Negima1072 commented 1 year ago

@tor4kichi 作業が完了しました。 RankingとUserDetailはnvapiに置き換え、 Searchを新たに追加しました(WithCeApiとWithPageがあったため) 前者に関しては一部でPR前と互換が取れていない箇所があります。また、既存のものに倣ってテスト等も作成しましたが、不自然な点等がありましたらご指摘いただければ幸いです。ご確認のほどよろしくお願いします。

tor4kichi commented 1 year ago

PRありがとうございます。もし使えれば自作アプリにも取り込みたいAPIがあってとても助かります。

手元で試してみていくつか修正必要な点ありましたので列挙させて頂きます。

LiveSearchの実際のレスポンスを確認すると ProgramProvider は常にnullであったため当該プロパティへのテストを除外したい
LiveSearchのTimeshiftSetting.EndTime がnullの場合があるので DateTimeOffset? EndTime へ変更必要
UserSearch/VideoKeywordSearch/VideoTagSearch の string TotalCount でJsonパースエラーのため int TotalCountへ変更必要
VideoTestにコメントアウト済みのHTMLパースの動画ランキングをテストするコードがありコンパイルエラー（当該テストコード削除でOK）
nvapiの動画ランキング取得コードに対するテストが必要
ListSearchは404でテストに失敗しました。停止済みAPIの可能性？
- 試したURL→ https://nvapi.nicovideo.jp/v1/search/mylist?keyword=%E6%AD%8C

以上となります。私の方で出来るPR修正はやれたらやりたいんですがPR周りはいまいちわからないので箇条書きでお伝えしております。マージ以外で私が対応したほうが話が早いような点があればやりますのでお知らせください。

Negima1072 commented 1 year ago

@tor4kichi レビューありがとうございます。

LiveSearchの実際のレスポンスを確認すると ProgramProvider は常にnullであったため当該プロパティへのテストを除外したい

修正しました。 https://github.com/tor4kichi/NiconicoToolkit/pull/6/commits/ff8c1a69065db5d04228a6e18467cea2a51773a2

LiveSearchのTimeshiftSetting.EndTime がnullの場合があるので DateTimeOffset? EndTime へ変更必要

修正しました。 https://github.com/tor4kichi/NiconicoToolkit/pull/6/commits/a330aa39e2d9a509dd16d1354f965f77ce9c82be

UserSearch/VideoKeywordSearch/VideoTagSearch の string TotalCount でJsonパースエラーのため int TotalCountへ変更必要

修正しました。 https://github.com/tor4kichi/NiconicoToolkit/pull/6/commits/a330aa39e2d9a509dd16d1354f965f77ce9c82be

VideoTestにコメントアウト済みのHTMLパースの動画ランキングをテストするコードがありコンパイルエラー（当該テストコード削除でOK）

修正しました。 https://github.com/tor4kichi/NiconicoToolkit/pull/6/commits/ff8c1a69065db5d04228a6e18467cea2a51773a2

nvapiの動画ランキング取得コードに対するテストが必要

修正しました。 https://github.com/tor4kichi/NiconicoToolkit/pull/6/commits/28d5cf5087649f0a8ede833e6da5186c4cd3ea2c

ListSearchは404でテストに失敗しました。停止済みAPIの可能性？試したURL→ https://nvapi.nicovideo.jp/v1/search/mylist?keyword=%E6%AD%8C

修正しました。 https://github.com/tor4kichi/NiconicoToolkit/pull/6/commits/1ca1440114477dc25dc1d58ec43a7f5d2970ae5f 正しいURLはv1/search/listでした

ご確認お願いしますm( )m

tor4kichi commented 1 year ago

ご対応ありがとうございます！検索の方はList含めて完璧に動きました！

動画ランキングなんですがテスト失敗していて２点ご確認頂きたいです。

Enum.Parse<Ranking.Video.RankingGenre>(genre) について、C#のEnum.Parseはケースセンシティブのため DataRow["all"] -> DataRow["All"] に修正お願いします。
上記修正後にテストを動作させると https://nvapi.nicovideo.jp/v1/ranking/genre/all?page=&term=hour にアクセスした上で Assert.IsTrue(res.Meta.IsSuccess); の段階で失敗が発生します。（エラー内容は400 InvalidParameter）
- pageの値が一部空欄になっているのもありますが、それを訂正した上でブラウザでアクセスしてみても、同様に400 InvalidParameterが返ってきます。URLの確認をお願いしたいです。

Negima1072 commented 1 year ago

@tor4kichi テストの使い方がいまいちわからないので確認できなくすみませんm( )m 1つ目に関してはcommitの修正で大丈夫でしょうか? 2つ目に関してはpageの値が不適切だったのでqueryの処理を他のものと揃えました。ブラウザで400が出たのはheaderが足りないからだと思います。ご確認お願いしますー

tor4kichi commented 1 year ago

@Negima1072 ご対応ありがとうございました！ランキングの方も動作確認しました。おっしゃる通りヘッダー不足のようで問題ありませんでした🙇‍♂️

tor4kichi / NiconicoToolkit

htmlパースからnvapiに置き換える #6