|
| 1 | +Android 进行单元测试难在哪-part1 |
| 2 | +--- |
| 3 | + |
| 4 | +> * 原文链接 : [Against Android Unit Tests](http://philosophicalhacker.com/2015/04/10/against-android-unit-tests/) |
| 5 | +* 原文作者 : [Matthew Dupree](http://philosophicalhacker.com/) |
| 6 | +* [译文出自 : 开发技术前线 www.devtf.cn](http://www.devtf.cn) |
| 7 | +* 译者 : [chaossss](https://github.com/chaosssss) |
| 8 | +* 校对者: [tiiime](https://github.com/tiiime) |
| 9 | +* 状态 : 完成 |
| 10 | + |
| 11 | + |
| 12 | + |
| 13 | +正如我在[序](issue-8/Android 进行单元测试难在哪-序.md)中所说,在 Android 中难于进行测试是众多 Android 开发者的共识。上一篇博文发出后,有许多同行回复了我,并对我的观点表示支持: |
| 14 | + |
| 15 | +> — Andy Dyer (@dammitandy) April 13, 2015 |
| 16 | +
|
| 17 | +> @philosohacker大大发话了!他写了一篇有关如何能更好地测试 Android 应用的博文,[地址戳我](http://t.co/gQCJKIOrSN) |
| 18 | +> — Andy Dyer (@dammitandy) April 13, 2015 |
| 19 | +
|
| 20 | +> 卧槽,我终于知道在 Android 上测试应用的正确姿势了!感谢:[地址戳我](http://t.co/8gIqfoFVnL) |
| 21 | +
|
| 22 | +> 非常赞同“Android SDK 排斥 Android 测试单元”的看法。你应该让你的代码和 SDK 的的联系尽可能地小。 |
| 23 | +
|
| 24 | +> — Pascal Hartig (@passy) April 12, 2015 |
| 25 | +
|
| 26 | +所以在 Android 中进行单元测试困难的问题,我提到的这些已经说得足够清楚了。但事实上,问题的根源比这更显而易见。难于进行单元测试的一部分原因是:为了启动 Roboletric 使你的测试单元能够以一种合理的速度运作,你往往需要做许多繁复且没有意义的事。不过在我看来,这并不是关键因素,关键在于:Google 设计的 Android SDK 和 Google 官方提倡的应用架构模式让测试变得困难,在某些情况下,进行测试甚至是不可能的。 |
| 27 | + |
| 28 | +这样说可能让人感觉是泛泛而谈,为了证明我的结论,我今天将会用一整个系列的博文来论证我的观点。在接下来的博文里,我会尽可能解释为什么官方鼓励的应用开发方式会使进行单元测试困难,甚至不可能。除此以外,我还会在这些博文里通过重构应用架构探索增强 Android 可测试性的办法,使得应用的代码不需要直接依赖于 Android SDK。通过介绍 Android 怎么让测试变得困难,我会延伸出我想到的重构应用的几个建议,我会在系列博文的第四篇里把它们列出来。 |
| 29 | + |
| 30 | +## 一个(看似)简单的测试示例 |
| 31 | + |
| 32 | +要分析这个问题,我们不妨先从一个简单的测试范例开始研究。示例里面用的是 Google IOsched 应用的源码,因为我在第一篇博文里就说道:IOsched 就是 Android 开发者的参考模板。 |
| 33 | + |
| 34 | +IOsched 应用里有一个有关 IO 大会细节的页面: |
| 35 | + |
| 36 | + |
| 37 | + |
| 38 | +页面里有一个“+”按钮,我们能通过这个按钮将 IO 大会添加日历的对应日期里,如果 IO 大会已经被添加到日历中,“+”按钮则会变成“check”按钮;如果用户再次点击“check”按钮,IO 大会则会从日历中被移除。打开相应的源码我们会发现:这个按钮的状态储存在一个叫作 mStarred 的实例变量里,Activity 会根据按钮的状态启动一个 Service,Service 将在它的 onStop() 方法中将对应的 IO 大会添加到用户的日历中/将对应的 IO 大会从用户的日历中移除。 |
| 39 | + |
| 40 | +```java |
| 41 | +@Override |
| 42 | +public void onStop() { |
| 43 | + super.onStop(); |
| 44 | + if (mInitStarred != mStarred) { |
| 45 | + if (UIUtils.getCurrentTime(this) < mSessionStart) { |
| 46 | + // Update Calendar event through the Calendar API on Android 4.0 or new versions. |
| 47 | + Intent intent = null; |
| 48 | + if (mStarred) { |
| 49 | + // Set up intent to add session to Calendar, if it doesn't exist already. |
| 50 | + intent = new Intent(SessionCalendarService.ACTION_ADD_SESSION_CALENDAR, |
| 51 | + mSessionUri); |
| 52 | + intent.putExtra(SessionCalendarService.EXTRA_SESSION_START, |
| 53 | + mSessionStart); |
| 54 | + intent.putExtra(SessionCalendarService.EXTRA_SESSION_END, |
| 55 | + mSessionEnd); |
| 56 | + intent.putExtra(SessionCalendarService.EXTRA_SESSION_ROOM, mRoomName); |
| 57 | + intent.putExtra(SessionCalendarService.EXTRA_SESSION_TITLE, mTitleString); |
| 58 | + } else { |
| 59 | + // Set up intent to remove session from Calendar, if exists. |
| 60 | + intent = new Intent(SessionCalendarService.ACTION_REMOVE_SESSION_CALENDAR, |
| 61 | + mSessionUri); |
| 62 | + intent.putExtra(SessionCalendarService.EXTRA_SESSION_START, |
| 63 | + mSessionStart); |
| 64 | + intent.putExtra(SessionCalendarService.EXTRA_SESSION_END, |
| 65 | + mSessionEnd); |
| 66 | + intent.putExtra(SessionCalendarService.EXTRA_SESSION_TITLE, mTitleString); |
| 67 | + } |
| 68 | + intent.setClass(this, SessionCalendarService.class); |
| 69 | + startService(intent); |
| 70 | + |
| 71 | + if (mStarred) { |
| 72 | + setupNotification(); |
| 73 | + } |
| 74 | + } |
| 75 | + } |
| 76 | +} |
| 77 | +``` |
| 78 | + |
| 79 | +在这样的代码里写一个测试单元看起来是个明智的选择,但问题就来了:你无法为这样的代码写一个测试单元。你可能就不懂了,为什么不能写啊?别急,为了找到问题所在,我们不妨回到 Android 系统里,并想一想通常情况下,是什么让代码可以被测试。 |
| 80 | + |
| 81 | +我们都知道,一个测试单元包含了三个步骤:准备,测试,断言。为了顺利地展示我们测试中的断言步骤,我们需要获得程序在执行测试步骤中的代码后的状态。我们不妨将这个状态称为“测试后状态”。对一个测试单元来说,只有三种办法能获得测试后状态。 |
| 82 | + |
| 83 | +第一种办法就是检查测试步骤中执行的方法的返回值,写下这样的测试代码倒不难: |
| 84 | + |
| 85 | +```java |
| 86 | +public void testSquare() { |
| 87 | + |
| 88 | + MathNerd mathNerd = new MathNerd(); |
| 89 | + |
| 90 | + int result = mathNerd.square(2); |
| 91 | + |
| 92 | + assertEquals(result, 4); |
| 93 | +} |
| 94 | +``` |
| 95 | + |
| 96 | +第二种办法就是获得被测试对象中那些可访问的属性的引用,并断言你取得的这些属性就是你想要的: |
| 97 | + |
| 98 | +```java |
| 99 | +public void testSquare() { |
| 100 | + |
| 101 | + MathNerd mathNerd = new MathNerd(); |
| 102 | + |
| 103 | + int result = mathNerd.square(2); |
| 104 | + |
| 105 | + assertTrue(mathNerd.didDoMath()); |
| 106 | +} |
| 107 | +``` |
| 108 | + |
| 109 | +第三种办法是检查对象中被注入的依赖的状态。这些依赖或许是不可访问的,但因为你在测试函数中对它们进行注入,那么你肯定会持有它们的引用 |
| 110 | + |
| 111 | +```java |
| 112 | +public void testSquare() { |
| 113 | + |
| 114 | + Calculator calculator = new Calculator(); |
| 115 | + |
| 116 | + TerribleMathStudent terribleMathStudent = new TerribleMathStudent(calculator); |
| 117 | + |
| 118 | + //terribleMathStudent uses calculator to do this |
| 119 | + int result = terribleMathStudent.square(2); |
| 120 | + The IOsched app has a IO session detail screen that looks like this: |
| 121 | + assertTrue(calculator.didDoMath()); |
| 122 | +} |
| 123 | +``` |
| 124 | + |
| 125 | +那么我们现在已经把能够在一个测试单元中完成断言步骤的方法都列出来了,接下来就该解释为什么刚刚的 Android 代码无法被测试了:答案很简单,我们根本没有办法在 onStop() 方法里完成测试单元的断言步骤——因为 onStop() 方法根本就没有返回值。SessionDetailActivity 里也不存在能帮助我们验证 SessionCalendarService 是否通过正确的命令启动的可访问属性。最后,启动 SessionCalendarService 的代码也不属于注入到 SessionDetailActivity 中的依赖。 |
| 126 | + |
| 127 | +你以为这样就完了吗?很不幸,还有比这更糟糕的。在 onStop() 里完成测试单元的准备步骤也是不可能的。为了在测试单元中完成准备步骤,你必须能够改变影响程序测试后状态的所有因素,不妨把它们叫作“预测试状态”。而我们没有任何办法能够在 onStop() 方法里改变预测试状态,所以我们不能对应用进行单元测试。 |
| 128 | + |
| 129 | +onStop() 方法里,预测试状态是一个叫作 mInitStarred 的布尔值,你看看写在 onStop() 里的代码估计就懂我的意思了: |
| 130 | + |
| 131 | +```java |
| 132 | +@Override |
| 133 | +public void onStop() { |
| 134 | + super.onStop(); |
| 135 | + if (mInitStarred != mStarred) { |
| 136 | + if (UIUtils.getCurrentTime(this) < mSessionStart) { |
| 137 | + // Update Calendar event through the Calendar API on Android 4.0 or new versions. |
| 138 | + Intent intent = null; |
| 139 | + if (mStarred) { |
| 140 | + // Set up intent to add session to Calendar, if it doesn't exist already. |
| 141 | + intent = new Intent(SessionCalendarService.ACTION_ADD_SESSION_CALENDAR, |
| 142 | + mSessionUri); |
| 143 | + intent.putExtra(SessionCalendarService.EXTRA_SESSION_START, |
| 144 | + mSessionStart); |
| 145 | + intent.putExtra(SessionCalendarService.EXTRA_SESSION_END, |
| 146 | + mSessionEnd); |
| 147 | + intent.putExtra(SessionCalendarService.EXTRA_SESSION_ROOM, mRoomName); |
| 148 | + intent.putExtra(SessionCalendarService.EXTRA_SESSION_TITLE, mTitleString); |
| 149 | + } else { |
| 150 | + // Set up intent to remove session from Calendar, if exists. |
| 151 | + intent = new Intent(SessionCalendarService.ACTION_REMOVE_SESSION_CALENDAR, |
| 152 | + mSessionUri); |
| 153 | + intent.putExtra(SessionCalendarService.EXTRA_SESSION_START, |
| 154 | + mSessionStart); |
| 155 | + intent.putExtra(SessionCalendarService.EXTRA_SESSION_END, |
| 156 | + mSessionEnd); |
| 157 | + intent.putExtra(SessionCalendarService.EXTRA_SESSION_TITLE, mTitleString); |
| 158 | + } |
| 159 | + intent.setClass(this, SessionCalendarService.class); |
| 160 | + startService(intent); |
| 161 | + |
| 162 | + if (mStarred) { |
| 163 | + setupNotification(); |
| 164 | + } |
| 165 | + } |
| 166 | + } |
| 167 | +} |
| 168 | +``` |
| 169 | + |
| 170 | +这个布尔值会在 ContentProvider 的查询操作完成后通过回调方法被设置: |
| 171 | + |
| 172 | +```java |
| 173 | +private void onSessionQueryComplete(Cursor cursor) { |
| 174 | + |
| 175 | + final boolean inMySchedule = cursor.getInt(SessionsQuery.IN_MY_SCHEDULE) != 0; |
| 176 | + |
| 177 | + //... |
| 178 | + |
| 179 | + if (!mIsKeynote) { |
| 180 | + showStarredDeferred(mInitStarred = inMySchedule, false); |
| 181 | + } |
| 182 | + |
| 183 | + //... |
| 184 | +} |
| 185 | +``` |
| 186 | + |
| 187 | +让 mInitStarred 在 Loader 回调方法里被初始化的原因是:如果用户向他们的日历添加了在 SessionDetailActivity 中选择的 IO 大会事件,那么用于标识该 IO 大会是否被添加到用户日历中的变量将会被存储到数据库中包含所有 IO 大会的表中。我们都知道,Google 希望我们使用 Loader 去获得数据库中的数据,所以 mInitStarred 在 Loader 回调方法里被初始化。 |
| 188 | + |
| 189 | +> 注:但是 Google 自己用 Loader 取数据都非常困难,详见视频:[Click me!](https://www.youtube.com/watch?v=qrPoIF6A9gM) |
| 190 | +
|
| 191 | +mInitStarred 用于判断应用是否必须启动 SessinCalendarService 更新用户的日历,添加/移除某个 IO 大会的信息。如果从数据库中查询获得的信息表示用户的日历已经添加了该大会,而且添加按钮已经转换为“check”,我们就不需要进行任何操作。 |
| 192 | + |
| 193 | +所以,如果我们要在 onStop() 方法里进行测试,我们想要确定 onStop() 方法是否真正启动了一个根据“+”按钮的状态去更新用户日历的 Service。但很不幸,mInitStarred 是一个在 Loader 回调中被初始化的私有实例变量,如果要在测试单元里改变它的值,一定会非常麻烦。那我们现在不妨再来想想改变测试单元预测试状态的办法。 |
| 194 | + |
| 195 | +第一个办法是,改变影响对象预测试状态的公有属性: |
| 196 | + |
| 197 | +> 注:改变测试状态的公有属性显然有很多办法。但如果,我举个例子,预测试状态由被测试对象的依赖决定,那我们当然也可以通过改变对象依赖的公有属性改变被测试对象的预测试状态。 |
| 198 | +
|
| 199 | +```java |
| 200 | +public void testSquare() { |
| 201 | + |
| 202 | + TerribleMathStudent terribleMathStudent = new TerribleMathStudent(); |
| 203 | + terribleMathStudent.setCalculator(new BrokenCalculator()); |
| 204 | + |
| 205 | + //terribleMathStudent uses broken calculator to do this, so this student really sucks |
| 206 | + int result = terribleMathStudent.square(2); |
| 207 | + |
| 208 | + assertNotEquals(result, 4); |
| 209 | + |
| 210 | + //we pity the terrible math student, so we give him a working calculator |
| 211 | + terribleMathStudent.setCalculator(new Calculator()); |
| 212 | + |
| 213 | + result = terribleMathStudent.square(2); |
| 214 | + |
| 215 | + assertEquals(result, 4); |
| 216 | +} |
| 217 | +``` |
| 218 | + |
| 219 | +第二个办法是改变传入被测试方法的参数: |
| 220 | + |
| 221 | +```java |
| 222 | +public void testSquare() { |
| 223 | + |
| 224 | + MathNerd mathNerd = new MathNerd(); |
| 225 | + |
| 226 | + //Math nerds only use calculators when they need them |
| 227 | + int result = mathNerd.squareBigNumber(new BrokenCalculator(), 54321); |
| 228 | + |
| 229 | + assertNotEquals(result, 2950771041); |
| 230 | + |
| 231 | + result = mathNerd.squareBigNumber(new Calculator(), 54321); |
| 232 | + |
| 233 | + assertEquals(result, 2950771041); |
| 234 | +} |
| 235 | +``` |
| 236 | + |
| 237 | +就像改变测试后状态一样,这两个办法都不能解决我们的问题,所以我们根本不可能在 onStop() 方法里面完成测试单元的准备步骤,问题的根源在于:SessionDetailActivity 里根本不存在能用于改变 mInitStarred 的公有属性。除此以外,onStop() 方法也不会把 mInitStarred 当作一个参数,所以我们无法通过改变 onStop() 的参数值来改变预测试状态。 |
| 238 | + |
| 239 | +## 结论 |
| 240 | + |
| 241 | +经过上面一系列源码的分析,每一个希冀进行单元测试的开发者都站在了悬崖的边缘上:即便是 Google 官方所谓的开发模板应用 - IOsched 中一个看起来非常简单的模块,要进行单元测试都困难重重。如果你曾经为了在 Activity 里为你的业务逻辑添加相应的测试单元绞尽脑汁,我现在可以告诉你为什么会这么蛋疼:大部分情况下,要在 Activity 里添加相应的测试单元只是不能实现而已。某些情况下,因为你没有办法改变被测试对象的预测试状态,使得你无法完成单元测试的准备步骤。在另一些情况下,因为你无法访问相关的测试后状态,使得你无法完成对象的断言步骤。而 SessionDetailActivity 的 onStop() 方法就是完美的示例,因为上面的两种情况在它身上体现地淋漓尽致。 |
| 242 | + |
| 243 | +Android 里还有许多类似的代码也是无法测试的,这让我不禁怀疑:让我们写下无法测试的代码的罪魁祸首,会不会就是带有 Android 平台特性的应用架构方式。所以在下一篇博文里,我将会深入探索这个假设。 |
0 commit comments