Skip to content

Commit 099ac51

Browse files
author
Anders Breid
authored
Fix bug not validating Subscription name. (#396)
* Fix bug not validating Subscription name, improved validation tests
1 parent 31d07f2 commit 099ac51

File tree

2 files changed

+108
-105
lines changed

2 files changed

+108
-105
lines changed

src/main/java/com/ericsson/ei/subscription/SubscriptionValidator.java

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -87,18 +87,22 @@ public static void validateSubscription(Subscription subscription)
8787
*/
8888
private static void validateSubscriptionName(String subscriptionName)
8989
throws SubscriptionValidationException {
90-
String invalidSubscriptionNameRegex = null;
91-
invalidSubscriptionNameRegex = RegExProvider.SUBSCRIPTION_NAME;
90+
String invalidSubscriptionNameRegex = RegExProvider.SUBSCRIPTION_NAME;
9291

9392
if (subscriptionName == null) {
9493
throw new SubscriptionValidationException(
9594
"Required field SubscriptionName has not been set.");
96-
} else if (invalidSubscriptionNameRegex == null
95+
}
96+
97+
if (invalidSubscriptionNameRegex == null
9798
|| invalidSubscriptionNameRegex.isEmpty()) {
9899
throw new SubscriptionValidationException(
99100
"A valid regular expression for validating subscription name is not provided.");
100-
} else if (Pattern.matches(invalidSubscriptionNameRegex,
101-
subscriptionName)) {
101+
}
102+
103+
Pattern pattern = Pattern.compile(invalidSubscriptionNameRegex);
104+
Matcher matcher = pattern.matcher(subscriptionName);
105+
if (matcher.find()) {
102106
throw new SubscriptionValidationException(
103107
"Wrong format of SubscriptionName: " + subscriptionName);
104108
}
@@ -224,8 +228,7 @@ private static void validateRestPostMediaType(String restPostMediaType)
224228
*/
225229
public static void validateEmail(String email)
226230
throws SubscriptionValidationException {
227-
String validEmailRegEx = null;
228-
validEmailRegEx = RegExProvider.NOTIFICATION_META;
231+
String validEmailRegEx = RegExProvider.NOTIFICATION_META;
229232

230233
if (validEmailRegEx == null || validEmailRegEx.isEmpty()) {
231234
throw new SubscriptionValidationException(

0 commit comments

Comments
 (0)