diff --git a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Service.java b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Service.java index 38a496b6..11be0994 100644 --- a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Service.java +++ b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Service.java @@ -212,15 +212,16 @@ public class Service extends AbstractJob { } /** - * Set priority to service. Only 1 = high priority, 2 = medium and 3 = low are allowed. + * Set priority to service. Only 1 (very high) to 10 (very low) are allowed. *
- * Default is 2 = medium.
+ * Default is 2.
*
* @param priority
* @return builder
*/
public Builder
* Default is 2 = medium.
*
@@ -277,7 +277,8 @@ public class Shipment extends AbstractJob {
* @return builder
*/
public Builder setPriority(int priority) {
- if(priority < 1 || priority > 3) throw new IllegalArgumentException("incorrect priority. only 1 = high, 2 = medium and 3 = low is allowed");
+ if (priority < 1 || priority > 10)
+ throw new IllegalArgumentException("incorrect priority. only 1 (very high) to 10 (very low) are allowed");
this.priority = priority;
return this;
}
diff --git a/jsprit-core/src/test/java/com/graphhopper/jsprit/core/problem/job/ServiceTest.java b/jsprit-core/src/test/java/com/graphhopper/jsprit/core/problem/job/ServiceTest.java
index bec514e2..f1efce37 100644
--- a/jsprit-core/src/test/java/com/graphhopper/jsprit/core/problem/job/ServiceTest.java
+++ b/jsprit-core/src/test/java/com/graphhopper/jsprit/core/problem/job/ServiceTest.java
@@ -250,6 +250,13 @@ public class ServiceTest {
Assert.assertEquals(3, s.getPriority());
}
+ @Test
+ public void whenSettingPriorities_itShouldBeSetCorrectly3() {
+ Service s = Service.Builder.newInstance("s").setLocation(Location.newInstance("loc"))
+ .setPriority(10).build();
+ Assert.assertEquals(10, s.getPriority());
+ }
+
@Test
public void whenNotSettingPriorities_defaultShouldBe2(){
Service s = Service.Builder.newInstance("s").setLocation(Location.newInstance("loc"))
diff --git a/jsprit-core/src/test/java/com/graphhopper/jsprit/core/problem/job/ShipmentTest.java b/jsprit-core/src/test/java/com/graphhopper/jsprit/core/problem/job/ShipmentTest.java
index 6c463d04..cbfa1f26 100644
--- a/jsprit-core/src/test/java/com/graphhopper/jsprit/core/problem/job/ShipmentTest.java
+++ b/jsprit-core/src/test/java/com/graphhopper/jsprit/core/problem/job/ShipmentTest.java
@@ -401,6 +401,14 @@ public class ShipmentTest {
Assert.assertEquals(3, s.getPriority());
}
+ @Test
+ public void whenSettingPriorities_itShouldBeSetCorrectly3() {
+ Shipment s = Shipment.Builder.newInstance("s").setPickupLocation(Location.newInstance("loc"))
+ .setDeliveryLocation(Location.newInstance("loc"))
+ .setPriority(10).build();
+ Assert.assertEquals(10, s.getPriority());
+ }
+
@Test
public void whenNotSettingPriorities_defaultShouldBe2(){
Shipment s = Shipment.Builder.newInstance("s").setPickupLocation(Location.newInstance("loc"))