2011 02-25-jenkinsci study

Copyright(c) Yusuke Yamamoto All rights reserved.

Jenkinsでリグレッションを起こす方法

2011/2/25山本裕介 @yusukey

http://samuraism.jp/@yusukey

http://samuraism.jp

http://samuraism.jp


話す内容• Twitter4Jについて

• Jenkinsを使っているのにリグレッション発生

• 原因・経緯

• 学んだこと


Twitter4Jについて


Java用Twitter APIライブラリ30名を超すコントリビュータ• JDK1.4.2～• Google App Engine

• Android

に対応LOC: 5.3万行くらいの規模


Twitter4Jの開発で利用しているツールIntelliJ IDEA: github / Jiraに対応

JUnit: テストツール

Maven: ビルドツール

Jira: issue管理ツール

スライドに登場するロゴ、製品名は各社、各コミュニティの登録商標です


Twitter4Jの開発で利用しているツール

Maven Central Repository

Hudson / Jenkins: CIツール

github: ソーシャルコーディングハブ

Sonatype Nexus: Mavenプロジェクト管理2010年1月～

2009年4月～

スライドに登場するロゴ、製品名は各社、各コミュニティの登録商標です

2011年2月～2009年3月～

2008年6月～


Twitter4Jの開発エコシステム

開発マシン CIサーバ github.com

service hook

repo1.maven.org

oss.sonatype.org

release:perform

promote

git push origin master


2007年より

3年半リグレッションなし♥♥


2010年10月2日リグレッション発生

♥♥


発生した問題• Twitter4J 2.1.5

• OAuth署名に失敗してメソッドコールができないとの報告

401:Authentication credentials were missing or incorrect.


原因・経緯


直接の原因• 文字列の分離を行うStringUtil.split()にバグ

• 特定の条件でOAuth署名に失敗

StringUtil.split(“foo,bar”,”,”); » {“foo”, “bar”}

StringUtil.split(“foobar”,”,”); » new String[0]

public static String[] split(String str, String separator){ String[] returnValue; int index; if(-1 == str.indexOf(separator)){ returnValue = new String[0]; }else{...


なぜStringUtil.splitを実装？• 元々String#splitを使用

• J2MEでString#split(@since JDK1.4)がなくエラーが発生するというユーザーがいたため

• J2MEはJDKのサブセット、JDK1.3(?)ベース

Caused by: java.lang.NoSuchMethodError: java.lang.String: method split(Ljava/lang/String;)[Ljava/lang/String; not found at twitter4j.conf.PropertyConfiguration.setFieldsWithTreePath(Ljava/util/Properties;Ljava/lang/String;)V(PropertyConfiguration.java:196) at twitter4j.conf.PropertyConfiguration.<init>(Ljava/lang/String;)V(PropertyConfiguration.java:121)


Jenkins使っていればリグレッションは起きないよね？


不具合のあるままリリースした原因１

• 当時Twitter４Jはほとんどテストされていなかった！

• なぜ？: 2010年8月末のBasic認証廃止に伴うテストケースのリファクタリングが原因


誤ったTestCaseのリファクタリング

TwitterTestUnitテストメソッド

OAuthTestOAuth認証初期化

BasicTestBasic認証初期化

TwitterTestUnitテストメソッド

OAuth初期化

Mavenに認識されない

before

after


不具合のあるままリリースした原因２

failしたテストケースを１部無視していた

なぜ？: Twitter APIは不安定なのでAPIの問題と勘違い

TwitterTestUnit AsyncTwitterTest

fail


学んだこと


学んだこと（１）• failするテストは定期的に確認！

• 特にage欄の確認は重要


学んだこと（２）• テストケースのリファクタリングは慎重に• テスト数が異様に減っていたら注意• コードカバレッジにも注目

怪しい兆候


ご静聴ありがとうございました

♥♥

2011 02-25-jenkinsci study

Technology