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 f177e93
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 @@ -52,6 +56,7 @@ default void addNotificationTemplateType(String notificationChannel, String disp
String tenantDomain, String applicationUuid)
throws NotificationTemplateManagerException {

throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 59 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#L59

Added line #L59 was not covered by tests
}

/**
Expand All @@ -65,7 +70,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 73 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#L73

Added line #L73 was not covered by tests
}

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

throw new UnsupportedOperationException("Not implemented yet");

Check warning on line 88 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#L88

Added line #L88 was not covered by tests
}

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

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

Check warning on line 103 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#L103

Added line #L103 was not covered by tests
}

/**
Expand All @@ -108,6 +114,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 117 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#L117

Added line #L117 was not covered by tests
}

/**
Expand All @@ -120,7 +127,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 130 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#L130

Added line #L130 was not covered by tests
}

/**
Expand All @@ -136,7 +143,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 146 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#L146

Added line #L146 was not covered by tests
}

/**
Expand All @@ -153,7 +160,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 163 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#L163

Added line #L163 was not covered by tests
}

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

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

Check warning on line 180 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#L180

Added line #L180 was not covered by tests
}

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

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

Check warning on line 198 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#L198

Added line #L198 was not covered by tests
}

/**
Expand All @@ -202,6 +209,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 212 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#L212

Added line #L212 was not covered by tests
}

/**
Expand All @@ -215,6 +223,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 226 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#L226

Added line #L226 was not covered by tests
}

/**
Expand All @@ -228,6 +237,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 240 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#L240

Added line #L240 was not covered by tests
}

/**
Expand All @@ -238,9 +248,11 @@ default void updateNotificationTemplate(NotificationTemplate notificationTemplat
* @param applicationUuid Application UUID.
* @throws NotificationTemplateManagerException If an error occurred while updating the notification template.
*/
@Deprecated
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 f177e93

Please sign in to comment.