diff --git a/services/core/java/com/android/server/notification/RankingHelper.java b/services/core/java/com/android/server/notification/RankingHelper.java index 221b2bb0886ad..32de3166faae8 100644 --- a/services/core/java/com/android/server/notification/RankingHelper.java +++ b/services/core/java/com/android/server/notification/RankingHelper.java @@ -561,7 +561,7 @@ public class RankingHelper implements RankingConfig { updateConfig(); return; } - if (channel.getImportance() < NotificationManager.IMPORTANCE_NONE + if (channel.getImportance() < NotificationManager.IMPORTANCE_MIN || channel.getImportance() > NotificationManager.IMPORTANCE_MAX) { throw new IllegalArgumentException("Invalid importance level"); } diff --git a/services/tests/notification/src/com/android/server/notification/RankingHelperTest.java b/services/tests/notification/src/com/android/server/notification/RankingHelperTest.java index 303054e777934..bc5c29d514a55 100644 --- a/services/tests/notification/src/com/android/server/notification/RankingHelperTest.java +++ b/services/tests/notification/src/com/android/server/notification/RankingHelperTest.java @@ -18,6 +18,7 @@ package com.android.server.notification; import static android.app.NotificationManager.IMPORTANCE_DEFAULT; import static android.app.NotificationManager.IMPORTANCE_HIGH; import static android.app.NotificationManager.IMPORTANCE_LOW; +import static android.app.NotificationManager.IMPORTANCE_NONE; import static junit.framework.Assert.assertNull; import static junit.framework.Assert.fail; @@ -504,12 +505,23 @@ public class RankingHelperTest { @Test public void testCreateChannel_blocked() throws Exception { - mHelper.setImportance(PKG, UID, NotificationManager.IMPORTANCE_NONE); + mHelper.setImportance(PKG, UID, IMPORTANCE_NONE); mHelper.createNotificationChannel(PKG, UID, new NotificationChannel("bananas", "bananas", IMPORTANCE_LOW), true); } + @Test + public void testCreateChannel_ImportanceNone() throws Exception { + try { + mHelper.createNotificationChannel(PKG, UID, + new NotificationChannel("bananas", "bananas", IMPORTANCE_NONE), true); + fail("Was allowed to create a blocked channel"); + } catch (IllegalArgumentException e) { + // yay + } + } + @Test public void testUpdate() throws Exception {