Skip to content

Commit

Permalink
Merge pull request #19 from sebastian-toepfer/keyword_multipleOf
Browse files Browse the repository at this point in the history
Keyword multiple of
  • Loading branch information
sebastian-toepfer authored Oct 3, 2023
2 parents be7de07 + f30297a commit 28540c3
Show file tree
Hide file tree
Showing 7 changed files with 196 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,12 @@ public boolean isInstance(final JsonValue value) {
}

private boolean isIntegral(final BigDecimal decimal) {
return BigDecimal.ZERO.equals(decimal) || decimal.stripTrailingZeros().scale() == 0;
return (
BigDecimal.ZERO.equals(decimal) ||
BigDecimal.ONE.equals(decimal) ||
BigDecimal.TEN.equals(decimal) ||
decimal.stripTrailingZeros().scale() == 0
);
}
},
STRING(JsonValue.ValueType.STRING);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@ static Stream<? extends Arguments> provideWithValidCombinations() {
Arguments.of(InstanceType.ARRAY, JsonValue.EMPTY_JSON_ARRAY),
Arguments.of(InstanceType.NUMBER, Json.createValue(23L)),
Arguments.of(InstanceType.INTEGER, Json.createValue(23L)),
Arguments.of(InstanceType.INTEGER, Json.createValue(10L)),
Arguments.of(InstanceType.INTEGER, Json.createValue(0.0))
);
}
Expand Down
7 changes: 7 additions & 0 deletions core/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -160,6 +160,13 @@
<inculde>**/tests/draft2019-09/exclusiveMaximum.json</inculde>
<inculde>**/tests/draft2020-12/exclusiveMaximum.json</inculde>
<inculde>**/tests/draft-next/exclusiveMaximum.json</inculde>

<inculde>**/tests/draft4/multipleOf.json</inculde>
<inculde>**/tests/draft6/multipleOf.json</inculde>
<inculde>**/tests/draft7/multipleOf.json</inculde>
<inculde>**/tests/draft2019-09/multipleOf.json</inculde>
<inculde>**/tests/draft2020-12/multipleOf.json</inculde>
<inculde>**/tests/draft-next/multipleOf.json</inculde>
</includes>
</resource>
</resources>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,8 @@ public BasicVocabulary() {
new MinimumKeywordType(),
new ExclusiveMinimumKeywordType(),
new MaximumKeywordType(),
new ExclusiveMaximumKeywordType()
new ExclusiveMaximumKeywordType(),
new MultipleOfKeywordType()
);
}

Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,84 @@
/*
* The MIT License
*
* Copyright 2023 sebastian.
*
* Permission is hereby granted, free of charge, to any person obtaining a copy
* of this software and associated documentation files (the "Software"), to deal
* in the Software without restriction, including without limitation the rights
* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
* copies of the Software, and to permit persons to whom the Software is
* furnished to do so, subject to the following conditions:
*
* The above copyright notice and this permission notice shall be included in
* all copies or substantial portions of the Software.
*
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
* THE SOFTWARE.
*/
package io.github.sebastiantoepfer.jsonschema.core.vocab.basic;

import io.github.sebastiantoepfer.jsonschema.ConstraintViolation;
import io.github.sebastiantoepfer.jsonschema.InstanceType;
import io.github.sebastiantoepfer.jsonschema.core.vocab.ConstraintAssertion;
import io.github.sebastiantoepfer.jsonschema.keyword.Keyword;
import io.github.sebastiantoepfer.jsonschema.keyword.KeywordType;
import jakarta.json.JsonNumber;
import jakarta.json.JsonValue;
import java.math.BigDecimal;
import java.util.Collection;
import java.util.Collections;
import java.util.List;
import java.util.Objects;

final class MultipleOfKeywordType implements KeywordType {

@Override
public String name() {
return "multipleOf";
}

@Override
public Keyword createKeyword(final JsonValue value) {
if (InstanceType.NUMBER.isInstance(value)) {
return new MultipleOfKeyword((JsonNumber) value);
} else {
throw new IllegalArgumentException("value must be a postiv number");
}
}

private class MultipleOfKeyword implements ConstraintAssertion {

private final BigDecimal multipleOf;

public MultipleOfKeyword(final JsonNumber multipleOf) {
this.multipleOf = multipleOf.bigDecimalValue();
}

@Override
public boolean hasName(final String name) {
return Objects.equals(name(), name);
}

@Override
public Collection<ConstraintViolation> violationsBy(final JsonValue value) {
final Collection<ConstraintViolation> result;
if (
!InstanceType.NUMBER.isInstance(value) ||
BigDecimal.ZERO.equals(
((JsonNumber) value).bigDecimalValue().remainder(multipleOf).stripTrailingZeros()
)
) {
result = Collections.emptyList();
} else {
result = List.of(new ConstraintViolation());
}
return result;
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,6 @@ class JsonTestSuiteIT {
@ParameterizedTest(name = "{0}")
@ArgumentsSource(JsonTestSuiteTestCaseProvider.class)
void runJsonTestSuite(final JsonTestSuiteTestCase testcase) {
assertTrue(testcase.isValid());
assertTrue(testcase.isValid(), testcase.toString());
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,95 @@
/*
* The MIT License
*
* Copyright 2023 sebastian.
*
* Permission is hereby granted, free of charge, to any person obtaining a copy
* of this software and associated documentation files (the "Software"), to deal
* in the Software without restriction, including without limitation the rights
* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
* copies of the Software, and to permit persons to whom the Software is
* furnished to do so, subject to the following conditions:
*
* The above copyright notice and this permission notice shall be included in
* all copies or substantial portions of the Software.
*
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
* THE SOFTWARE.
*/
package io.github.sebastiantoepfer.jsonschema.core.vocab.basic;

import static org.hamcrest.MatcherAssert.assertThat;
import static org.hamcrest.Matchers.is;
import static org.junit.jupiter.api.Assertions.assertThrows;

import io.github.sebastiantoepfer.jsonschema.keyword.Keyword;
import jakarta.json.Json;
import jakarta.json.JsonValue;
import java.math.BigDecimal;
import org.junit.jupiter.api.Test;

class MultipleOfKeywordTypeTest {

@Test
void should_know_his_name() {
final Keyword multipleOf = new MultipleOfKeywordType().createKeyword(Json.createValue(10));

assertThat(multipleOf.hasName("multipleOf"), is(true));
assertThat(multipleOf.hasName("test"), is(false));
}

@Test
void should_not_be_creatable_with_non_integer_value() {
final MultipleOfKeywordType keywordType = new MultipleOfKeywordType();
assertThrows(IllegalArgumentException.class, () -> keywordType.createKeyword(JsonValue.FALSE));
}

@Test
void should_be_valid_for_non_number_values() {
assertThat(
new MultipleOfKeywordType()
.createKeyword(Json.createValue(10))
.asAssertion()
.isValidFor(JsonValue.EMPTY_JSON_OBJECT),
is(true)
);
}

@Test
void should_be_valid_for_a_multipleOf() {
assertThat(
new MultipleOfKeywordType()
.createKeyword(Json.createValue(1.5))
.asAssertion()
.isValidFor(Json.createValue(4.5)),
is(true)
);
}

@Test
void should_be_invalid_for_non_multipleOf() {
assertThat(
new MultipleOfKeywordType()
.createKeyword(Json.createValue(2))
.asAssertion()
.isValidFor(Json.createValue(7)),
is(false)
);
}

@Test
void should_be_valid_for_any_int_if_multipleOf_is_1en8() {
assertThat(
new MultipleOfKeywordType()
.createKeyword(Json.createValue(new BigDecimal("1e-8")))
.asAssertion()
.isValidFor(Json.createValue(12391239123L)),
is(true)
);
}
}

0 comments on commit 28540c3

Please sign in to comment.