Skip to content

Commit

Permalink
Addressing comments
Browse files Browse the repository at this point in the history
  • Loading branch information
RushanNanayakkara committed Oct 22, 2024
1 parent c8c7b5b commit e758d90
Showing 1 changed file with 35 additions and 20 deletions.
Original file line number Diff line number Diff line change
@@ -1,18 +1,21 @@
/*
* Copyright (c) 2019-2024, WSO2 Inc. (http://www.wso2.org) All Rights Reserved.
* Copyright (c) 2019-2024, WSO2 LLC. (http://www.wso2.com).
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* WSO2 LLC. licenses this file to you under the Apache License,
* Version 2.0 (the "License"); you may not use this file except
* in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
* Unless required by applicable law or agreed to in writing,
* software distributed under the License is distributed on an
* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
* KIND, either express or implied. See the License for the
* specific language governing permissions and limitations
* under the License.
*/

package org.wso2.carbon.identity.governance.service.notification;

import org.wso2.carbon.identity.governance.exceptions.notiification.NotificationTemplateManagerException;
Expand All @@ -37,6 +40,7 @@ public interface NotificationTemplateManager {
default void addNotificationTemplateType(String notificationChannel, String displayName, String tenantDomain)
throws NotificationTemplateManagerException {

throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 43 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L43

Added line #L43 was not covered by tests
}

/**
Expand All @@ -48,10 +52,12 @@ default void addNotificationTemplateType(String notificationChannel, String disp
* @param applicationUuid Application UUID.
* @throws NotificationTemplateManagerException If an error occurred while adding the template to the registry.
*/
@Deprecated
default void addNotificationTemplateType(String notificationChannel, String displayName,
String tenantDomain, String applicationUuid)
throws NotificationTemplateManagerException {

throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 60 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L60

Added line #L60 was not covered by tests
}

/**
Expand All @@ -65,7 +71,7 @@ default void addNotificationTemplateType(String notificationChannel, String disp
default List<String> getAllNotificationTemplateTypes(String notificationChannel, String tenantDomain)
throws NotificationTemplateManagerException {

return null;
throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 74 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L74

Added line #L74 was not covered by tests
}

/**
Expand All @@ -80,6 +86,7 @@ default void deleteNotificationTemplateType(String notificationChannel, String t
String tenantDomain)
throws NotificationTemplateManagerException {

throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 89 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L89

Added line #L89 was not covered by tests
}

/**
Expand All @@ -94,7 +101,7 @@ default boolean isNotificationTemplateTypeExists(String notificationChannel, Str
String tenantDomain)
throws NotificationTemplateManagerException {

return false;
throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 104 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L104

Added line #L104 was not covered by tests
}

/**
Expand All @@ -108,6 +115,7 @@ default boolean isNotificationTemplateTypeExists(String notificationChannel, Str
default void resetNotificationTemplateType(String notificationChannel, String templateType, String tenantDomain)
throws NotificationTemplateManagerException {

throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 118 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L118

Added line #L118 was not covered by tests
}

/**
Expand All @@ -120,7 +128,7 @@ default void resetNotificationTemplateType(String notificationChannel, String te
default List<NotificationTemplate> getAllNotificationTemplates(String notificationChannel, String tenantDomain)
throws NotificationTemplateManagerException {

return null;
throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 131 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L131

Added line #L131 was not covered by tests
}

/**
Expand All @@ -136,7 +144,7 @@ default List<NotificationTemplate> getNotificationTemplatesOfType(String notific
String templateDisplayName, String tenantDomain)
throws NotificationTemplateManagerException {

return null;
throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 147 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L147

Added line #L147 was not covered by tests
}

/**
Expand All @@ -153,7 +161,7 @@ default List<NotificationTemplate> getNotificationTemplatesOfType(String notific
String templateDisplayName, String tenantDomain, String applicationUuid)
throws NotificationTemplateManagerException {

return null;
throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 164 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L164

Added line #L164 was not covered by tests
}

/**
Expand All @@ -170,7 +178,7 @@ default NotificationTemplate getNotificationTemplate(String notificationChannel,
String tenantDomain)
throws NotificationTemplateManagerException {

return null;
throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 181 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L181

Added line #L181 was not covered by tests
}

/**
Expand All @@ -188,7 +196,7 @@ default NotificationTemplate getNotificationTemplate(String notificationChannel,
String tenantDomain, String applicationUuid)
throws NotificationTemplateManagerException {

return null;
throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 199 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L199

Added line #L199 was not covered by tests
}

/**
Expand All @@ -202,6 +210,7 @@ default NotificationTemplate getNotificationTemplate(String notificationChannel,
default void addNotificationTemplate(NotificationTemplate notificationTemplate, String tenantDomain)
throws NotificationTemplateManagerException {

throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 213 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L213

Added line #L213 was not covered by tests
}

/**
Expand All @@ -215,6 +224,7 @@ default void addNotificationTemplate(NotificationTemplate notificationTemplate,
default void addNotificationTemplate(NotificationTemplate notificationTemplate, String tenantDomain,
String applicationUuid) throws NotificationTemplateManagerException {

throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 227 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L227

Added line #L227 was not covered by tests
}

/**
Expand All @@ -228,6 +238,7 @@ default void addNotificationTemplate(NotificationTemplate notificationTemplate,
default void updateNotificationTemplate(NotificationTemplate notificationTemplate, String tenantDomain)
throws NotificationTemplateManagerException {

throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 241 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L241

Added line #L241 was not covered by tests
}

/**
Expand All @@ -241,6 +252,7 @@ default void updateNotificationTemplate(NotificationTemplate notificationTemplat
default void updateNotificationTemplate(NotificationTemplate notificationTemplate, String tenantDomain,
String applicationUuid) throws NotificationTemplateManagerException {

throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 255 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L255

Added line #L255 was not covered by tests
}

/**
Expand All @@ -256,6 +268,7 @@ default void deleteNotificationTemplate(String notificationChannel, String templ
String tenantDomain)
throws NotificationTemplateManagerException {

throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 271 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L271

Added line #L271 was not covered by tests
}


Expand All @@ -273,6 +286,7 @@ default void deleteNotificationTemplate(String notificationChannel, String templ
String tenantDomain, String applicationUuid)
throws NotificationTemplateManagerException {

throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 289 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L289

Added line #L289 was not covered by tests
}

/**
Expand All @@ -287,6 +301,7 @@ default void deleteNotificationTemplate(String notificationChannel, String templ
default void addDefaultNotificationTemplates(String notificationChannel, String tenantDomain)
throws NotificationTemplateManagerException {

throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 304 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L304

Added line #L304 was not covered by tests
}

/**
Expand All @@ -297,7 +312,7 @@ default void addDefaultNotificationTemplates(String notificationChannel, String
*/
default List<NotificationTemplate> getDefaultNotificationTemplates(String notificationChannel) {

return null;
throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 315 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L315

Added line #L315 was not covered by tests
}

/**
Expand All @@ -312,7 +327,7 @@ default boolean isNotificationTemplateExists(String notificationChannel, String
String tenantDomain)
throws NotificationTemplateManagerException {

return false;
throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 330 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L330

Added line #L330 was not covered by tests
}

/**
Expand All @@ -329,7 +344,7 @@ default boolean isNotificationTemplateExists(String notificationChannel, String
String tenantDomain, String applicationUuid)
throws NotificationTemplateManagerException {

return false;
throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 347 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L347

Added line #L347 was not covered by tests
}

/**
Expand All @@ -341,7 +356,7 @@ default boolean isNotificationTemplateExists(String notificationChannel, String
default List<NotificationTemplate> getAllSystemNotificationTemplatesOfType(String notificationChannel,
String templateDisplayName) throws NotificationTemplateManagerException {

return null;
throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 359 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L359

Added line #L359 was not covered by tests
}

/**
Expand All @@ -357,6 +372,6 @@ default NotificationTemplate getSystemNotificationTemplate(String notificationCh
String locale)
throws NotificationTemplateManagerException {

return null;
throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 375 in components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java

View check run for this annotation

Codecov / codecov/patch

components/org.wso2.carbon.identity.governance/src/main/java/org/wso2/carbon/identity/governance/service/notification/NotificationTemplateManager.java#L375

Added line #L375 was not covered by tests
}
}

0 comments on commit e758d90

Please sign in to comment.