xfy
58a97d1725
问题
- AnimatedWebp.kt 原写死 WEBP_FILES = (1..152).map{...}, 与磁盘文件耦合却无校验
- 加 153.webp → 永远不会被随机到; 删某个 → random() 偶发命中不存在的资源
- 这是静默失败, 运行期无报错, 难排查
改动
- core/build.gradle.kts defaultConfig: 构建期扫描 assets/animations/ 生成
buildConfigField("String[]", "WEBP_FILES", "new String[]{...}")
含 require(webpFiles.isNotEmpty()) 防空目录构建
- AnimatedWebp.kt: WEBP_FILES 从 private 硬编码改为 internal val = BuildConfig.WEBP_FILES.toList()
(internal 让同模块测试可访问)
- 新增 AnimatedWebpFilesTest: 2 个守卫测试
1. webpFilesMatchDirectoryContents: WEBP_FILES 必须与 assets/animations/ 一一对应
2. webpFilesUseZeroPaddedThreeDigitNames: 锁定 NNN.webp 命名约定
TDD 流程
- 先写测试 → 编译失败(WEBP_FILES 是 private) → 红灯成立
- 改实现 → 测试 PASS → 绿灯
- 突变验证: 临时改 WEBP_FILES = (1..150) 漏掉 151/152
→ webpFilesMatchDirectoryContents 立即 FAIL, 守卫有效
- 恢复后全量 146 个测试 0 失败
设计说明
- 当前实现里 WEBP_FILES 与目录都来自同一次构建期扫描, 二者天然一致
- 测试的核心价值是锁定「两者一致」不变量, 防止有人回退成硬编码
(突变验证已证明: 回退后测试立即失败)
- BuildConfig 生成结果验证: String[] 含 152 个元素 001~152.webp
验证
- ./gradlew :core:testDebugUnitTest → 146 tests, 0 failures, 0 errors
- ./gradlew :app:assembleDebug → BUILD SUCCESSFUL
- BuildConfig.java: public static final String[] WEBP_FILES = new String[]{"001.webp",...,"152.webp"}
94 lines
3.0 KiB
Plaintext
94 lines
3.0 KiB
Plaintext
import org.jetbrains.kotlin.gradle.dsl.JvmTarget
|
|
|
|
plugins {
|
|
alias(libs.plugins.androidLibrary)
|
|
alias(libs.plugins.composeCompiler)
|
|
}
|
|
|
|
android {
|
|
namespace = "plus.rua.project.shared"
|
|
compileSdk = libs.versions.android.compileSdk.get().toInt()
|
|
|
|
defaultConfig {
|
|
minSdk = libs.versions.android.minSdk.get().toInt()
|
|
consumerProguardFiles("proguard-rules.pro")
|
|
|
|
// 构建期扫描 assets/animations/ 生成 WebP 文件列表，避免运行期硬编码 (1..152)
|
|
// 与 assets/ 目录耦合却不校验，导致增删文件后隐性 bug
|
|
val webpFiles = layout.projectDirectory
|
|
.dir("src/main/assets/animations")
|
|
.asFile
|
|
.listFiles { f -> f.extension.equals("webp", ignoreCase = true) }
|
|
?.map { it.name }
|
|
?.sorted()
|
|
?: emptyList()
|
|
require(webpFiles.isNotEmpty()) { "assets/animations/ 不应为空，请检查目录" }
|
|
// 拼成 Java 数组字面量: new String[]{"001.webp","002.webp",...}
|
|
val quoted = webpFiles.joinToString(",") { name -> "\"$name\"" }
|
|
val arrayLiteral = "new String[]{$quoted}"
|
|
buildConfigField("String[]", "WEBP_FILES", arrayLiteral)
|
|
}
|
|
|
|
buildTypes {
|
|
debug {
|
|
buildConfigField("boolean", "ENABLE_TRACE", "true")
|
|
}
|
|
release {
|
|
isMinifyEnabled = false
|
|
consumerProguardFiles("proguard-rules.pro")
|
|
buildConfigField("boolean", "ENABLE_TRACE", "false")
|
|
}
|
|
create("trace") {
|
|
initWith(buildTypes.getByName("release"))
|
|
buildConfigField("boolean", "ENABLE_TRACE", "true")
|
|
}
|
|
}
|
|
|
|
buildFeatures {
|
|
compose = true
|
|
buildConfig = true
|
|
}
|
|
|
|
compileOptions {
|
|
sourceCompatibility = JavaVersion.VERSION_17
|
|
targetCompatibility = JavaVersion.VERSION_17
|
|
}
|
|
|
|
packaging {
|
|
resources {
|
|
excludes += listOf(
|
|
"/META-INF/{AL2.0,LGPL2.1}",
|
|
"/META-INF/LICENSE*",
|
|
"/META-INF/NOTICE*",
|
|
"META-INF/DEPENDENCIES",
|
|
"**/*.kotlin_metadata",
|
|
"**/*.kotlin_module",
|
|
)
|
|
}
|
|
}
|
|
}
|
|
|
|
dependencies {
|
|
implementation(platform(libs.compose.bom))
|
|
|
|
implementation(libs.androidx.activity.compose)
|
|
implementation(libs.androidx.lifecycle.viewmodelCompose)
|
|
implementation(libs.androidx.lifecycle.runtimeCompose)
|
|
|
|
implementation(libs.compose.runtime)
|
|
implementation(libs.compose.foundation)
|
|
implementation(libs.compose.animation)
|
|
implementation(libs.compose.material.icons)
|
|
implementation(libs.compose.material3)
|
|
implementation(libs.compose.ui)
|
|
|
|
implementation(libs.kotlinx.datetime)
|
|
implementation(libs.tyme4kt)
|
|
implementation(libs.sketch.compose)
|
|
implementation(libs.sketch.animated.webp)
|
|
implementation(libs.androidx.profileinstaller)
|
|
|
|
testImplementation("org.jetbrains.kotlin:kotlin-test-junit:${libs.versions.kotlin.get()}")
|
|
testImplementation(libs.kotlinx.coroutines.test)
|
|
}
|