Commit 0fa061ba authored by Anita Woodruff's avatar Anita Woodruff Committed by Commit Bot

[Android Tests] SiteChannelsManagerTest: use custom matcher in all cases

- Should have been using matchesChannelStatus in all cases where channel
statuses are checked, for better error messaging on fail.

Bug: 
Change-Id: If36da189c1707e3e788bbe48cfc9ccf8eccbe71e
Reviewed-on: https://chromium-review.googlesource.com/545716Reviewed-by: default avatarPeter Beverloo <peter@chromium.org>
Commit-Queue: Peter Beverloo <peter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481897}
parent 443acabd
......@@ -71,7 +71,7 @@ public class SiteChannelsManagerTest {
assertThat(Arrays.asList(mSiteChannelsManager.getSiteChannels()), hasSize(1));
NotificationSettingsBridge.SiteChannel channel = mSiteChannelsManager.getSiteChannels()[0];
assertThat(channel.getOrigin(), is("https://chromium.org"));
assertThat(channel.getStatus(), is(NotificationChannelStatus.ENABLED));
assertThat(channel.getStatus(), matchesChannelStatus(NotificationChannelStatus.ENABLED));
}
@Test
......@@ -82,7 +82,7 @@ public class SiteChannelsManagerTest {
assertThat(Arrays.asList(mSiteChannelsManager.getSiteChannels()), hasSize(1));
NotificationSettingsBridge.SiteChannel channel = mSiteChannelsManager.getSiteChannels()[0];
assertThat(channel.getOrigin(), is("https://example.com"));
assertThat(channel.getStatus(), is(NotificationChannelStatus.BLOCKED));
assertThat(channel.getStatus(), matchesChannelStatus(NotificationChannelStatus.BLOCKED));
}
@Test
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment