From ba1332d1b4dd525a2ef313246482708cb2016052 Mon Sep 17 00:00:00 2001 From: Maurice Lam Date: Wed, 6 Dec 2017 12:55:23 -0800 Subject: [PATCH] Changes needed to upgrade to new gradle version - Add ignore for various lint errors that is introduced in the lint update - Remove GLIF Pixel themes as they are unused and causing lint errors - Add flavor dimension "compat" as it is now required - Replace *Compile with *Implementation in the gradle files, as needed by the newer Gradle version - Use build tools 27.0.0 to avoid the Gradle warning - Use new Robolectric configuration mechanism and removed our old hack, per http://robolectric.org/getting-started/ Reference: https://developer.android.com/studio/build/gradle-plugin-3-0-0-migration.html Test: Existing tests pass Bug: 70287447 Change-Id: I505476ae3695cd06349766c5e6280299f779f272 --- .../view/NavigationBarButton.java | 2 + .../items/ExpandableSwitchItemTest.java | 3 +- .../setupwizardlib/items/SwitchItemTest.java | 3 +- .../util/DimensionConsistencyTest.java | 3 +- library/lint.xml | 4 + library/main/res/values/styles.xml | 20 ++- .../util/WizardManagerHelper.java | 12 -- library/platform/res/values-v23/styles.xml | 4 +- ...ecyclerViewScrollHandlingDelegateTest.java | 3 +- library/rules.gradle | 10 +- library/self.gradle | 25 ++-- .../setupwizardlib/GlifLayoutTest.java | 2 +- .../setupwizardlib/items/ButtonItemTest.java | 5 +- .../setupwizardlib/items/ItemGroupTest.java | 5 +- .../PatchedGradleManifestFactory.java | 126 ------------------ .../SuwLibRobolectricTestRunner.java | 29 ---- .../setupwizardlib/span/LinkSpanTest.java | 4 +- .../ListViewScrollHandlingDelegateTest.java | 3 +- .../template/RequireScrollMixinTest.java | 3 +- .../ScrollViewScrollHandlingDelegateTest.java | 3 +- .../util/GlifDimensionTest.java | 3 +- .../setupwizardlib/util/GlifStyleTest.java | 3 +- .../setupwizardlib/util/PartnerTest.java | 2 - .../util/WizardManagerHelperTest.java | 3 +- .../view/FillContentLayoutTest.java | 3 +- .../view/IllustrationVideoViewTest.java | 2 - tools/gradle/android.properties | 4 +- 27 files changed, 54 insertions(+), 235 deletions(-) delete mode 100644 library/test/robotest/src/com/android/setupwizardlib/robolectric/PatchedGradleManifestFactory.java diff --git a/library/gingerbread/src/com/android/setupwizardlib/view/NavigationBarButton.java b/library/gingerbread/src/com/android/setupwizardlib/view/NavigationBarButton.java index 5172c47..d7a3c2e 100644 --- a/library/gingerbread/src/com/android/setupwizardlib/view/NavigationBarButton.java +++ b/library/gingerbread/src/com/android/setupwizardlib/view/NavigationBarButton.java @@ -16,6 +16,7 @@ package com.android.setupwizardlib.view; +import android.annotation.SuppressLint; import android.content.Context; import android.content.res.ColorStateList; import android.graphics.PorterDuff; @@ -30,6 +31,7 @@ import android.widget.Button; * Button for navigation bar, which includes tinting of its compound drawables to be used for dark * and light themes. */ +@SuppressLint("AppCompatCustomView") public class NavigationBarButton extends Button { public NavigationBarButton(Context context) { diff --git a/library/gingerbread/test/robotest/src/com/android/setupwizardlib/items/ExpandableSwitchItemTest.java b/library/gingerbread/test/robotest/src/com/android/setupwizardlib/items/ExpandableSwitchItemTest.java index 4fb3e28..3020ed3 100644 --- a/library/gingerbread/test/robotest/src/com/android/setupwizardlib/items/ExpandableSwitchItemTest.java +++ b/library/gingerbread/test/robotest/src/com/android/setupwizardlib/items/ExpandableSwitchItemTest.java @@ -32,7 +32,6 @@ import android.widget.FrameLayout; import android.widget.ImageView; import android.widget.TextView; -import com.android.setupwizardlib.BuildConfig; import com.android.setupwizardlib.R; import com.android.setupwizardlib.robolectric.SuwLibRobolectricTestRunner; @@ -44,7 +43,7 @@ import org.robolectric.annotation.Config; import java.util.ArrayList; @RunWith(SuwLibRobolectricTestRunner.class) -@Config(constants = BuildConfig.class, sdk = { Config.OLDEST_SDK, Config.NEWEST_SDK }) +@Config(sdk = { Config.OLDEST_SDK, Config.NEWEST_SDK }) public class ExpandableSwitchItemTest { private TextView mSummaryView; diff --git a/library/gingerbread/test/robotest/src/com/android/setupwizardlib/items/SwitchItemTest.java b/library/gingerbread/test/robotest/src/com/android/setupwizardlib/items/SwitchItemTest.java index d391d80..fa5bbba 100644 --- a/library/gingerbread/test/robotest/src/com/android/setupwizardlib/items/SwitchItemTest.java +++ b/library/gingerbread/test/robotest/src/com/android/setupwizardlib/items/SwitchItemTest.java @@ -31,7 +31,6 @@ import android.widget.FrameLayout; import android.widget.ImageView; import android.widget.TextView; -import com.android.setupwizardlib.BuildConfig; import com.android.setupwizardlib.R; import com.android.setupwizardlib.robolectric.SuwLibRobolectricTestRunner; @@ -40,7 +39,7 @@ import org.junit.runner.RunWith; import org.robolectric.annotation.Config; @RunWith(SuwLibRobolectricTestRunner.class) -@Config(constants = BuildConfig.class, sdk = { Config.OLDEST_SDK, Config.NEWEST_SDK }) +@Config(sdk = { Config.OLDEST_SDK, Config.NEWEST_SDK }) public class SwitchItemTest { private SwitchCompat mSwitch; diff --git a/library/gingerbread/test/robotest/src/com/android/setupwizardlib/util/DimensionConsistencyTest.java b/library/gingerbread/test/robotest/src/com/android/setupwizardlib/util/DimensionConsistencyTest.java index 43e7f03..7a08235 100644 --- a/library/gingerbread/test/robotest/src/com/android/setupwizardlib/util/DimensionConsistencyTest.java +++ b/library/gingerbread/test/robotest/src/com/android/setupwizardlib/util/DimensionConsistencyTest.java @@ -25,7 +25,6 @@ import android.util.DisplayMetrics; import android.util.TypedValue; import android.view.ContextThemeWrapper; -import com.android.setupwizardlib.BuildConfig; import com.android.setupwizardlib.R; import com.android.setupwizardlib.robolectric.SuwLibRobolectricTestRunner; @@ -35,7 +34,7 @@ import org.junit.runner.RunWith; import org.robolectric.annotation.Config; @RunWith(SuwLibRobolectricTestRunner.class) -@Config(constants = BuildConfig.class, sdk = Config.ALL_SDKS) +@Config(sdk = Config.ALL_SDKS) public class DimensionConsistencyTest { // Visual height of the framework switch widget diff --git a/library/lint.xml b/library/lint.xml index ca22c65..625b20d 100644 --- a/library/lint.xml +++ b/library/lint.xml @@ -1,7 +1,11 @@ + + + + diff --git a/library/main/res/values/styles.xml b/library/main/res/values/styles.xml index 0e7685b..e1e229f 100644 --- a/library/main/res/values/styles.xml +++ b/library/main/res/values/styles.xml @@ -32,9 +32,6 @@ top|bottom - - - - @@ -94,11 +88,10 @@ gravity - - - @@ -211,7 +204,10 @@ false -