Apache
/
lucene
mirror of https://github.com/apache/lucene.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

SOLR-13207: Fix tests

This commit is contained in:
Tomas Fernandez Lobbe 2019-11-01 15:53:38 -07:00
parent 3cf131de52
commit 5c6a299eff
2 changed files with 20 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
solr/core/src/java/org/apache/solr/util/SolrPluginUtils.java
View File

@ -678,11 +678,11 @@ public class SolrPluginUtils {
spec = spaceAroundLessThanPattern.matcher(spec).replaceAll("<");
for (String s : spacePattern.split(spec)) {
String[] parts = lessThanPattern.split(s,0);
if (parts.length == 0) {
if (parts.length < 2) {
throw new SolrException(SolrException.ErrorCode.BAD_REQUEST,
"Operator < must be followed by a number");
"Invalid 'mm' spec: '" + s + "'. Expecting values before and after '<'");
}
int upperBound = checkedParseInt(parts[0], "Operator < must be followed by a number");
int upperBound = checkedParseInt(parts[0], "Invalid 'mm' spec. Expecting an integer.");
if (optionalClauseCount <= upperBound) {
return result;
} else {
@ -699,11 +699,11 @@ public class SolrPluginUtils {
/* percentage - assume the % was the last char. If not, let Integer.parseInt fail. */
spec = spec.substring(0,spec.length()-1);
int percent = checkedParseInt(spec,
"% must be preceded by a number and not combined with other operators");
"Invalid 'mm' spec. Expecting an integer.");
float calc = (result * percent) * (1/100f);
result = calc < 0 ? result + (int)calc : (int)calc;
} else {
int calc = checkedParseInt(spec, "Input should be a number");
int calc = checkedParseInt(spec, "Invalid 'mm' spec. Expecting an integer.");
result = calc < 0 ? result + calc : calc;
}

20
solr/core/src/test/org/apache/solr/util/SolrPluginUtilsTest.java
View File

@ -341,11 +341,21 @@ public class SolrPluginUtilsTest extends SolrTestCaseJ4 {
@Test
public void testMinShouldMatchBadQueries() {
expectThrows(SolrException.class, () -> calcMSM(1, "1<"));
expectThrows(SolrException.class, () -> calcMSM(1, "1<x"));
expectThrows(SolrException.class, () -> calcMSM(1, "x%"));
expectThrows(SolrException.class, () -> calcMSM(1, "%%"));
expectThrows(SolrException.class, () -> calcMSM(1, "x"));
Exception e = expectThrows(SolrException.class, () -> calcMSM(2, "1<"));
assertEquals("Invalid 'mm' spec: '1<'. Expecting values before and after '<'" , e.getMessage());
e = expectThrows(SolrException.class, () -> calcMSM(2, "1<x"));
assertEquals("Invalid 'mm' spec. Expecting an integer.", e.getMessage());
e = expectThrows(SolrException.class, () -> calcMSM(1, "x%"));
assertEquals("Invalid 'mm' spec. Expecting an integer.", e.getMessage());
e = expectThrows(SolrException.class, () -> calcMSM(1, "%%"));
assertEquals("Invalid 'mm' spec. Expecting an integer.", e.getMessage());
e = expectThrows(SolrException.class, () -> calcMSM(1, "x"));
assertEquals("Invalid 'mm' spec. Expecting an integer.", e.getMessage());
e = expectThrows(SolrException.class, () -> calcMSM(10, "2<-25% 9<X"));
assertEquals("Invalid 'mm' spec. Expecting an integer." , e.getMessage());
e = expectThrows(SolrException.class, () -> calcMSM(10, "2<-25% 9<"));
assertEquals("Invalid 'mm' spec: '9<'. Expecting values before and after '<'" , e.getMessage());
}
@Test