google / google/go-github

Add integration tests

オープン
#98 コメント 5 件 リアクション 0 件 担当者 0 名 GitHub で見る
enhancement
主要言語
Go
スター
11.3k
フォーク
2.5k
平均マージ
1日 20時間
マージ済み PR(30日)
52

説明

This is an ongoing work item to add integration tests for the entire library. Unlike the existing unit tests that spin up a local HTTP server that mimics certain behavior of the API, these integration tests will call against the live GitHub API. As such, they will likely take a lot longer to execute and will not be run near as often. I've set things up with aff0a13d884ff5dc09bccb6c1ce081a82213bfab and will continue working from there.

I'm not 100% settled yet on how extensive these tests need to be. For example, we don't need to test the value of every field in a response. The biggest goal is to have a test suite that can be run when things like the [default media type changes](https://developer.github.com/changes/2014-04-09-reminder-about-default-media-type-change/) to make sure the library is still working properly. I also have a separate test suite that I'm working on to identify fields in responses that we aren't mapping into our Go structs.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

コミット aff0a13d884ff5dc09bccb6c1ce081a82213bfab の統合テスト設定から始め、ローカル HTTP サーバーを使用する既存のユニットテストと比較してください。先に必要な範囲を定義してから進めてください。完了時には、すべてのレスポンスフィールドをチェックせずに、デフォルトのメディアタイプの変更などの互換性問題を検出する、実際の GitHub API に対するテストスイートになっている必要があります。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
go
領域
api, testing
issue の種類
機能追加
難易度
5/5
見積もり時間
1週間以上
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
20/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。