やってみる

アウトプットすべく己を導くためのブログ。その試行錯誤すらたれ流す。

RequestParameter.pyの単体テストをした

単体では動作させられないため実際は結合テスト

成果物

GitHubGithub.Uploader.RequestParameter.unittest.201705041425

開発環境

前回まで

ソースコード変更

冗長なコードをコメントアウトして書き換えた。その後テストしてクリア。

追加しなかった

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行なので毎回書いても大差ない

所感

バグなし。テストケースも少ないが。