Skip to content

Commit

Permalink
RANGER-3888: fixed unit tests for earlier commit in RANGER-3888 - #2
Browse files Browse the repository at this point in the history
  • Loading branch information
@mneethiraj
mneethiraj committed Oct 28, 2022
1 parent b1920de commit b6e4324
Show file tree
Hide file tree
Showing 3 changed files with 245 additions and 19 deletions.
54 changes: 38 additions & 16 deletions .../main/java/org/apache/ranger/plugin/model/validation/RangerValidityScheduleValidator.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -231,11 +231,22 @@ private boolean validateFieldSpec(RangerValidityRecurrence recurrence, RangerVal
}

if(ret) {
SimpleDateFormat formatter = new SimpleDateFormat("yyyy");
String startYear = formatter.format(startTime);
String endYear = formatter.format(endTime);
int minimum = field == RangerValidityRecurrence.RecurrenceSchedule.ScheduleFieldSpec.year ? Integer.valueOf(startYear) : field.minimum;
int maximum = field == RangerValidityRecurrence.RecurrenceSchedule.ScheduleFieldSpec.year ? Integer.valueOf(endYear) : field.maximum;
final int minimum;
final int maximum;

if (field == RecurrenceSchedule.ScheduleFieldSpec.year) {
SimpleDateFormat formatter = new SimpleDateFormat("yyyy");

minimum = Integer.valueOf(formatter.format(startTime));
maximum = Integer.valueOf(formatter.format(endTime));
} else if (field == RecurrenceSchedule.ScheduleFieldSpec.month) {
minimum = field.minimum + 1;
maximum = field.maximum + 1;
} else {
minimum = field.minimum;
maximum = field.maximum;
}

ret = validateRanges(recurrence, field, minimum, maximum, validationFailures);
}
return ret;
Expand Down Expand Up @@ -352,36 +363,47 @@ public int compare(Range me, Range other) {
if (StringUtils.isNotEmpty(spec)) {
// Range
if (spec.startsWith("-") || spec.endsWith("-")) {
validationFailures.add(new ValidationFailureDetails(0, fieldName, "", false, true, false, "incorrect range spec"));
validationFailures.add(new ValidationFailureDetails(0, fieldName, "", false, true, false, "incorrect range spec: " + spec));
ret = false;
} else {
String[] ranges = StringUtils.split(spec, "-");
if (ranges.length > 2) {
validationFailures.add(new ValidationFailureDetails(0, fieldName, "", false, true, false, "incorrect range spec"));
validationFailures.add(new ValidationFailureDetails(0, fieldName, "", false, true, false, "incorrect range spec: " + spec));
ret = false;
} else if (ranges.length == 2) {
int val1 = minValidValue, val2 = maxValidValue;
if (!StringUtils.equals(ranges[0], RangerValidityRecurrence.RecurrenceSchedule.WILDCARD)) {
val1 = Integer.valueOf(ranges[0]);
if (val1 < minValidValue || val1 > maxValidValue) {
validationFailures.add(new ValidationFailureDetails(0, fieldName, "", false, true, false, "incorrect lower range value"));
ret = false;
}
} else {
value = RangerValidityRecurrence.RecurrenceSchedule.WILDCARD;
}

if (!StringUtils.equals(ranges[1], RangerValidityRecurrence.RecurrenceSchedule.WILDCARD)) {
val2 = Integer.valueOf(ranges[1]);
} else {
value = RangerValidityRecurrence.RecurrenceSchedule.WILDCARD;
}

if (field == RecurrenceSchedule.ScheduleFieldSpec.year) { // for year, one bound (lower or upper) can be outside the range
if (val1 < minValidValue && val2 > maxValidValue) {
validationFailures.add(new ValidationFailureDetails(0, fieldName, "", false, true, false, "incorrect range: (" + val1 + ", " + val2 + "). valid range: (" + minValidValue + ", " + maxValidValue + ")"));
ret = false;
}
} else { // for month/dayOfMonth/dayOfWeek/hour/minute both bounds (lower and upper) must be within range
if (val1 < minValidValue || val1 > maxValidValue) {
validationFailures.add(new ValidationFailureDetails(0, fieldName, "", false, true, false, "incorrect lower range: " + val1 + ". valid range: (" + minValidValue + ", " + maxValidValue + ")"));
ret = false;
}

if (val2 < minValidValue || val2 > maxValidValue) {
validationFailures.add(new ValidationFailureDetails(0, fieldName, "", false, true, false, "incorrect upper range value"));
validationFailures.add(new ValidationFailureDetails(0, fieldName, "", false, true, false, "incorrect upper range: " + val2 + ". valid range: (" + minValidValue + ", " + maxValidValue + ")"));
ret = false;
}
} else {
value = RangerValidityRecurrence.RecurrenceSchedule.WILDCARD;
}

if (ret) {
if (val1 >= val2) {
validationFailures.add(new ValidationFailureDetails(0, fieldName, "", false, true, false, "incorrect range"));
validationFailures.add(new ValidationFailureDetails(0, fieldName, "", false, true, false, "incorrect range: min=" + val1 + ", max=" + val2));
ret = false;
} else {
value = RangerValidityRecurrence.RecurrenceSchedule.WILDCARD;
Expand All @@ -401,7 +423,7 @@ public int compare(Range me, Range other) {
if (!StringUtils.equals(ranges[0], RangerValidityRecurrence.RecurrenceSchedule.WILDCARD)) {
int val = Integer.valueOf(ranges[0]);
if (val < minValidValue || val > maxValidValue) {
validationFailures.add(new ValidationFailureDetails(0, fieldName, "", false, true, false, "incorrect value"));
validationFailures.add(new ValidationFailureDetails(0, fieldName, "", false, true, false, "incorrect value: " + val + ". Valid range: (" + minValidValue + "-" + maxValidValue + ")"));
ret = false;
} else {
if (!StringUtils.equals(value, RangerValidityRecurrence.RecurrenceSchedule.WILDCARD)) {
Expand Down
6 changes: 3 additions & 3 deletions agents-common/src/test/java/org/apache/ranger/plugin/policyengine/TestPolicyEngine.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -905,9 +905,9 @@ private void runValiditySchedulerTests(String resourceName) {
}
}

assertTrue(testCase.name, isValid == testCase.result.isValid);
assertTrue(testCase.name, isApplicable == testCase.result.isApplicable);
assertTrue(testCase.name + ", [" + validationFailures +"]", validationFailures.size() == testCase.result.validationFailureCount);
assertEquals(testCase.name + " - isValid (validationFailures: " + validationFailures + ")", testCase.result.isValid, isValid);
assertEquals(testCase.name + " - isApplicable (validationFailures: " + validationFailures + ")", testCase.result.isApplicable, isApplicable);
assertEquals(testCase.name + " - validationFailureCount (validationFailures: " + validationFailures +")", testCase.result.validationFailureCount, validationFailures.size());
}
}
TimeZone.setDefault(defaultTZ);
Expand Down
204 changes: 204 additions & 0 deletions ...on/src/test/resources/policyengine/validityscheduler/test-validity-schedules-invalid.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -210,5 +210,209 @@
"isApplicable": false,
"validationFailureCount": 2
}
},
{
"name": "invalid lower bound - minute",
"validitySchedules": [
{
"startTime": "2018/01/12 14:32:00", "endTime": "2050/01/23 06:30:00",
"recurrences": [
{
"schedule": { "minute": "-1", "hour": "0", "dayOfMonth": "1", "dayOfWeek": "1", "month": "1", "year": "2018" },
"interval": { "minutes": 10 }
}
]
}
],
"accessTime": "20180112-18:32:27.000-0800",
"result": { "isValid": false, "isApplicable": false, "validationFailureCount": 1
}
},
{
"name": "invalid upper bound - minute",
"validitySchedules": [
{
"startTime": "2018/01/12 14:32:00", "endTime": "2050/01/23 06:30:00",
"recurrences": [
{
"schedule": { "minute": "60", "hour": "0", "dayOfMonth": "1", "dayOfWeek": "1", "month": "1", "year": "2018" },
"interval": { "minutes": 10 }
}
]
}
],
"accessTime": "20180112-18:32:27.000-0800",
"result": { "isValid": false, "isApplicable": false, "validationFailureCount": 1
}
},
{
"name": "invalid lower bound - hour",
"validitySchedules": [
{
"startTime": "2018/01/12 14:32:00", "endTime": "2050/01/23 06:30:00",
"recurrences": [
{
"schedule": { "minute": "0", "hour": "-1", "dayOfMonth": "1", "dayOfWeek": "1", "month": "1", "year": "2018" },
"interval": { "minutes": 10 }
}
]
}
],
"accessTime": "20180112-18:32:27.000-0800",
"result": { "isValid": false, "isApplicable": false, "validationFailureCount": 1
}
},
{
"name": "invalid upper bound - hour",
"validitySchedules": [
{
"startTime": "2018/01/12 14:32:00", "endTime": "2050/01/23 06:30:00",
"recurrences": [
{
"schedule": { "minute": "0", "hour": "24", "dayOfMonth": "1", "dayOfWeek": "1", "month": "1", "year": "2018" },
"interval": { "minutes": 10 }
}
]
}
],
"accessTime": "20180112-18:32:27.000-0800",
"result": { "isValid": false, "isApplicable": false, "validationFailureCount": 1
}
},
{
"name": "invalid lower bound - dayOfMonth",
"validitySchedules": [
{
"startTime": "2018/01/12 14:32:00", "endTime": "2050/01/23 06:30:00",
"recurrences": [
{
"schedule": { "minute": "0", "hour": "0", "dayOfMonth": "0", "dayOfWeek": "1", "month": "1", "year": "2018" },
"interval": { "minutes": 10 }
}
]
}
],
"accessTime": "20180112-18:32:27.000-0800",
"result": { "isValid": false, "isApplicable": false, "validationFailureCount": 1
}
},
{
"name": "invalid upper bound - dayOfMonth",
"validitySchedules": [
{
"startTime": "2018/01/12 14:32:00", "endTime": "2050/01/23 06:30:00",
"recurrences": [
{
"schedule": { "minute": "0", "hour": "0", "dayOfMonth": "32", "dayOfWeek": "1", "month": "1", "year": "2018" },
"interval": { "minutes": 10 }
}
]
}
],
"accessTime": "20180112-18:32:27.000-0800",
"result": { "isValid": false, "isApplicable": false, "validationFailureCount": 1
}
},
{
"name": "invalid lower bound - dayOfWeek",
"validitySchedules": [
{
"startTime": "2018/01/12 14:32:00", "endTime": "2050/01/23 06:30:00",
"recurrences": [
{
"schedule": { "minute": "0", "hour": "0", "dayOfMonth": "1", "dayOfWeek": "0", "month": "1", "year": "2018" },
"interval": { "minutes": 10 }
}
]
}
],
"accessTime": "20180112-18:32:27.000-0800",
"result": { "isValid": false, "isApplicable": false, "validationFailureCount": 1
}
},
{
"name": "invalid upper bound - dayOfWeek",
"validitySchedules": [
{
"startTime": "2018/01/12 14:32:00", "endTime": "2050/01/23 06:30:00",
"recurrences": [
{
"schedule": { "minute": "0", "hour": "0", "dayOfMonth": "1", "dayOfWeek": "8", "month": "1", "year": "2018" },
"interval": { "minutes": 10 }
}
]
}
],
"accessTime": "20180112-18:32:27.000-0800",
"result": { "isValid": false, "isApplicable": false, "validationFailureCount": 1
}
},
{
"name": "invalid lower bound - month",
"validitySchedules": [
{
"startTime": "2018/01/12 14:32:00", "endTime": "2050/01/23 06:30:00",
"recurrences": [
{
"schedule": { "minute": "0", "hour": "0", "dayOfMonth": "1", "dayOfWeek": "1", "month": "0", "year": "2018" },
"interval": { "minutes": 10 }
}
]
}
],
"accessTime": "20180112-18:32:27.000-0800",
"result": { "isValid": false, "isApplicable": false, "validationFailureCount": 1
}
},
{
"name": "invalid upper bound - month",
"validitySchedules": [
{
"startTime": "2018/01/12 14:32:00", "endTime": "2050/01/23 06:30:00",
"recurrences": [
{
"schedule": { "minute": "0", "hour": "0", "dayOfMonth": "1", "dayOfWeek": "1", "month": "13", "year": "2018" },
"interval": { "minutes": 10 }
}
]
}
],
"accessTime": "20180112-18:32:27.000-0800",
"result": { "isValid": false, "isApplicable": false, "validationFailureCount": 1
}
},
{
"name": "invalid lower bound - year",
"validitySchedules": [
{
"startTime": "2018/01/12 14:32:00", "endTime": "2050/01/23 06:30:00",
"recurrences": [
{
"schedule": { "minute": "0", "hour": "0", "dayOfMonth": "1", "dayOfWeek": "1", "month": 1, "year": "2016" },
"interval": { "minutes": 10 }
}
]
}
],
"accessTime": "20180112-18:32:27.000-0800",
"result": { "isValid": false, "isApplicable": false, "validationFailureCount": 1
}
},
{
"name": "invalid upper bound - year",
"validitySchedules": [
{
"startTime": "2018/01/12 14:32:00", "endTime": "2050/01/23 06:30:00",
"recurrences": [
{
"schedule": { "minute": "0", "hour": "0", "dayOfMonth": "1", "dayOfWeek": "1", "month": "1", "year": "2101" },
"interval": { "minutes": 10 }
}
]
}
],
"accessTime": "20180112-18:32:27.000-0800",
"result": { "isValid": false, "isApplicable": false, "validationFailureCount": 1
}
}
]

0 comments on commit b6e4324

Please sign in to comment.