単体では動作させられないため実際は結合テスト。
成果物
Github.Uploader.RequestParameter.unittest.201705041425
開発環境
- Linux Mint 17.3 MATE 32bit
- SQLite 3.8.2
- Python 3.4.3
前回まで
ソースコード変更
冗長なコードをコメントアウトして書き換えた。その後テストしてクリア。
追加しなかった
def CreateBasicOrTwoFactor(self, username, password, two_factor_secret=None) if None is not two_factor: return TwoFactorAuthentication(username, password, two_factor_secret) else: return BasicAuthentication(username, password)
アカウント登録(GitHubUserRegister.py
)にて上記の処理を頻繁に行うのでメソッドが欲しいと思ったがやめた。理由は以下。
- DBでなく文字列を引数にとるのでGitHub認証確認していないまま各認証クラスのインスタンスを返すことになる
- コンストラクタでDBを引数で渡されているのに使わないのは不自然
- たった4行なので毎回書いても大差ない
所感
バグなし。テストケースも少ないが。