Skip to content

Commit

Permalink
Misc improvement for integration_tests
Browse files Browse the repository at this point in the history
1. Include integration_tests:versioniong to settings.gradle.
2. Migrate more assertions to Google Truth for AccessibilityEventTest.
3. Bump minSdk to 21 for androidx_test.

Signed-off-by: utzcoz <utzcoz@outlook.com>
  • Loading branch information
utzcoz committed May 4, 2024
1 parent 1baf605 commit b01fdf3
Show file tree
Hide file tree
Showing 9 changed files with 16 additions and 15 deletions.
2 changes: 1 addition & 1 deletion integration_tests/androidx_test/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ android {

defaultConfig {
testApplicationId 'org.robolectric.integrationtests.axt'
minSdk 19
minSdk 21
targetSdk 34
multiDexEnabled true
testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
Expand Down
10 changes: 5 additions & 5 deletions integration_tests/androidx_test/src/main/AndroidManifest.xml
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,11 @@
<activity android:name="org.robolectric.integrationtests.axt.EspressoActivity"
android:label="Activity Label"
android:exported="true" >
<intent-filter >
<action android:name="android.intent.action.VIEW" />
<category android:name="android.intent.category.DEFAULT" />
</intent-filter>
</activity>
<intent-filter>
<action android:name="android.intent.action.VIEW" />
<category android:name="android.intent.category.DEFAULT" />
</intent-filter>
</activity>
<activity android:name="org.robolectric.integrationtests.axt.EspressoScrollingActivity"
android:exported="true" />
<activity android:name="org.robolectric.integrationtests.axt.ActivityWithPlatformMenu"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
package="org.robolectric.integrationtests.axt">

<uses-sdk
android:minSdkVersion="19"
android:minSdkVersion="21"
android:targetSdkVersion="34"/>

<application>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
package="org.robolectric.integrationtests.axt">

<uses-sdk
android:minSdkVersion="19"
android:minSdkVersion="21"
android:targetSdkVersion="34"/>

<application
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
package="org.robolectric.integrationtests.axt">

<uses-sdk
android:minSdkVersion="19"
android:minSdkVersion="21"
android:targetSdkVersion="34"/>

<application>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
package="org.robolectric.integrationtests.axt">

<uses-sdk
android:minSdkVersion="19"
android:minSdkVersion="21"
android:targetSdkVersion="34"/>

<application>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
package="org.robolectric.integrationtests.axt">

<uses-sdk
android:minSdkVersion="19"
android:minSdkVersion="21"
android:targetSdkVersion="34"/>

<application>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
package android.view.accessibility;

import static com.google.common.truth.Truth.assertWithMessage;
import static org.junit.Assert.assertFalse;

import android.os.Build;
import android.os.Message;
Expand Down Expand Up @@ -269,9 +268,10 @@ static void assertEqualAccessibilityRecord(
.isEqualTo(expectedRecord.getDisplayId());
}

assertFalse(
"one of the parcelableData is null",
expectedRecord.getParcelableData() == null ^ receivedRecord.getParcelableData() == null);
assertWithMessage("one of the parcelableData is null")
.that(
expectedRecord.getParcelableData() == null ^ receivedRecord.getParcelableData() == null)
.isFalse();
if (expectedRecord.getParcelableData() != null && receivedRecord.getParcelableData() != null) {
assertWithMessage("parcelableData has incorrect value")
.that(((Message) receivedRecord.getParcelableData()).what)
Expand Down
1 change: 1 addition & 0 deletions settings.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -42,4 +42,5 @@ include ":integration_tests:play_services"
include ":integration_tests:sparsearray"
include ":integration_tests:nativegraphics"
include ":integration_tests:room"
include ":integration_tests:versioning"
include ':testapp'

0 comments on commit b01fdf3

Please sign in to comment.