Enable lint in tests for modules that require no fixes

PiperOrigin-RevId: 693313908
This commit is contained in:
ibaker 2024-11-05 05:37:16 -08:00 committed by Copybara-Service
parent 12cb803486
commit 286273c10e
15 changed files with 61 additions and 17 deletions

View File

@ -29,6 +29,10 @@ android {
} }
} }
lintOptions {
checkTestSources true
}
compileOptions { compileOptions {
sourceCompatibility JavaVersion.VERSION_1_8 sourceCompatibility JavaVersion.VERSION_1_8
targetCompatibility JavaVersion.VERSION_1_8 targetCompatibility JavaVersion.VERSION_1_8

View File

@ -35,11 +35,6 @@ android {
lint { lint {
baseline = file("lint-baseline.xml") baseline = file("lint-baseline.xml")
} }
lintOptions {
// TODO: b/353490583 - Move this to common_library_config.gradle once it's enabled for all
// libraries.
checkTestSources true
}
publishing { publishing {
singleVariant('release') { singleVariant('release') {
withSourcesJar() withSourcesJar()

View File

@ -22,6 +22,11 @@ android {
} }
} }
lintOptions {
// TODO: b/353490583 - Enable this once the violations are fixed.
checkTestSources false
}
publishing { publishing {
singleVariant('release') { singleVariant('release') {
withSourcesJar() withSourcesJar()

View File

@ -27,6 +27,11 @@ android {
test.assets.srcDir '../test_data/src/test/assets' test.assets.srcDir '../test_data/src/test/assets'
} }
lintOptions {
// TODO: b/353490583 - Enable this once the violations are fixed.
checkTestSources false
}
publishing { publishing {
singleVariant('release') { singleVariant('release') {
withSourcesJar() withSourcesJar()

View File

@ -28,6 +28,11 @@ android {
} }
} }
lintOptions {
// TODO: b/353490583 - Enable this once the violations are fixed.
checkTestSources false
}
// TODO(Internal: b/372449691): Remove packagingOptions once AGP is updated // TODO(Internal: b/372449691): Remove packagingOptions once AGP is updated
// to version 8.5.1 or higher. // to version 8.5.1 or higher.
packagingOptions { packagingOptions {

View File

@ -33,6 +33,11 @@ android {
test.assets.srcDir '../test_data/src/test/assets/' test.assets.srcDir '../test_data/src/test/assets/'
} }
lintOptions {
// TODO: b/353490583 - Enable this once the violations are fixed.
checkTestSources false
}
publishing { publishing {
singleVariant('release') { singleVariant('release') {
withSourcesJar() withSourcesJar()

View File

@ -34,12 +34,6 @@ android {
test.assets.srcDir '../test_data/src/test/assets/' test.assets.srcDir '../test_data/src/test/assets/'
} }
lintOptions {
// TODO: b/353490583 - Move this to common_library_config.gradle once it's enabled for all
// libraries.
checkTestSources true
}
publishing { publishing {
singleVariant('release') { singleVariant('release') {
withSourcesJar() withSourcesJar()

View File

@ -24,6 +24,11 @@ android {
sourceSets.test.assets.srcDir '../test_data/src/test/assets/' sourceSets.test.assets.srcDir '../test_data/src/test/assets/'
lintOptions {
// TODO: b/353490583 - Enable this once the violations are fixed.
checkTestSources false
}
publishing { publishing {
singleVariant('release') { singleVariant('release') {
withSourcesJar() withSourcesJar()

View File

@ -27,6 +27,11 @@ android {
test.assets.srcDir '../test_data/src/test/assets/' test.assets.srcDir '../test_data/src/test/assets/'
} }
lintOptions {
// TODO: b/353490583 - Enable this once the violations are fixed.
checkTestSources false
}
publishing { publishing {
singleVariant('release') { singleVariant('release') {
withSourcesJar() withSourcesJar()

View File

@ -34,6 +34,11 @@ android {
test.assets.srcDir '../test_data/src/test/assets/' test.assets.srcDir '../test_data/src/test/assets/'
} }
lintOptions {
// TODO: b/353490583 - Enable this once the violations are fixed.
checkTestSources false
}
publishing { publishing {
singleVariant('release') { singleVariant('release') {
withSourcesJar() withSourcesJar()

View File

@ -33,11 +33,6 @@ android {
buildFeatures { buildFeatures {
aidl true aidl true
} }
lintOptions {
// TODO: b/353490583 - Move this to common_library_config.gradle once it's enabled for all
// libraries.
checkTestSources true
}
} }
dependencies { dependencies {
api project(modulePrefix + 'lib-common') api project(modulePrefix + 'lib-common')

View File

@ -13,7 +13,14 @@
// limitations under the License. // limitations under the License.
apply from: "$gradle.ext.androidxMediaSettingsDir/common_library_config.gradle" apply from: "$gradle.ext.androidxMediaSettingsDir/common_library_config.gradle"
android.namespace 'androidx.media3.test.exoplayer.playback' android {
namespace 'androidx.media3.test.exoplayer.playback'
lintOptions {
// TODO: b/353490583 - Enable this once the violations are fixed.
checkTestSources false
}
}
dependencies { dependencies {
androidTestImplementation 'androidx.test:rules:' + androidxTestRulesVersion androidTestImplementation 'androidx.test:rules:' + androidxTestRulesVersion

View File

@ -33,6 +33,11 @@ android {
} }
sourceSets.main.assets.srcDir '../test_data/src/test/assets/' sourceSets.main.assets.srcDir '../test_data/src/test/assets/'
lintOptions {
// TODO: b/353490583 - Enable this once the violations are fixed.
checkTestSources false
}
} }
dependencies { dependencies {

View File

@ -34,6 +34,11 @@ android {
test.assets.srcDir '../test_data/src/test/assets/' test.assets.srcDir '../test_data/src/test/assets/'
} }
lintOptions {
// TODO: b/353490583 - Enable this once the violations are fixed.
checkTestSources false
}
publishing { publishing {
singleVariant('release') { singleVariant('release') {
withSourcesJar() withSourcesJar()

View File

@ -26,6 +26,10 @@ android {
buildFeatures { buildFeatures {
compose true compose true
} }
lintOptions {
// TODO: b/353490583 - Enable this once the violations are fixed.
checkTestSources false
}
publishing { publishing {
singleVariant('release') { singleVariant('release') {
withSourcesJar() withSourcesJar()