Merge pull request #107 from generalui/BrokenTestFix

android10 · android10 · commit 45be4d94c1a9 · 2016-02-22T10:42:54.000+01:00
Fixing broken tests that were referring to non-existing fragment ids.
diff --git a/presentation/src/androidTest/java/com/fernandocejas/android10/sample/test/view/activity/UserDetailsActivityTest.java b/presentation/src/androidTest/java/com/fernandocejas/android10/sample/test/view/activity/UserDetailsActivityTest.java
@@ -53,7 +53,7 @@ public UserDetailsActivityTest() {
 
   public void testContainsUserDetailsFragment() {
     Fragment userDetailsFragment =
-        userDetailsActivity.getFragmentManager().findFragmentById(R.id.fl_fragment);
+        userDetailsActivity.getFragmentManager().findFragmentById(R.id.fragmentContainer);
     assertThat(userDetailsFragment, is(notNullValue()));
   }
 
diff --git a/presentation/src/androidTest/java/com/fernandocejas/android10/sample/test/view/activity/UserListActivityTest.java b/presentation/src/androidTest/java/com/fernandocejas/android10/sample/test/view/activity/UserListActivityTest.java
@@ -45,7 +45,7 @@ public UserListActivityTest() {
 
   public void testContainsUserListFragment() {
     Fragment userListFragment =
-        userListActivity.getFragmentManager().findFragmentById(R.id.fragmentUserList);
+        userListActivity.getFragmentManager().findFragmentById(R.id.fragmentContainer);
     assertThat(userListFragment, is(notNullValue()));
   }
 

Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,7 @@ public UserDetailsActivityTest() {`
`53`	`53`
`54`	`54`	`public void testContainsUserDetailsFragment() {`
`55`	`55`	`Fragment userDetailsFragment =`
`56`		`- userDetailsActivity.getFragmentManager().findFragmentById(R.id.fl_fragment);`
	`56`	`+ userDetailsActivity.getFragmentManager().findFragmentById(R.id.fragmentContainer);`
`57`	`57`	`assertThat(userDetailsFragment, is(notNullValue()));`
`58`	`58`	`}`
`59`	`59`
Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@ public UserListActivityTest() {`
`45`	`45`
`46`	`46`	`public void testContainsUserListFragment() {`
`47`	`47`	`Fragment userListFragment =`
`48`		`- userListActivity.getFragmentManager().findFragmentById(R.id.fragmentUserList);`
	`48`	`+ userListActivity.getFragmentManager().findFragmentById(R.id.fragmentContainer);`
`49`	`49`	`assertThat(userListFragment, is(notNullValue()));`
`50`	`50`	`}`
`51`	`51`