RenJuan/TASKPM
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

ItEr34S08ValidacionEProbasFuncionaisItEr33S08: Providing the type is always necessary

Browse source
This commit is contained in:
Óscar González Fernández 2009-11-15 20:12:40 +01:00
parent 112d941474
commit c9164d982b
3 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
navalplanner-business/src/main/java/org/navalplanner/business/orders/entities/OrderElement.java
View file

@ -98,7 +98,7 @@ public abstract class OrderElement extends BaseEntity {
List<SchedulingState> childrenStates = getChildrenStates();
SchedulingState result = childrenStates.isEmpty() ? new SchedulingState(
getSchedulingStateType())
: new SchedulingState(childrenStates);
: new SchedulingState(schedulingStateType, childrenStates);
result.addTypeChangeListener(new ITypeChangedListener() {
@Override
public void typeChanged(Type newType) {

3
navalplanner-business/src/main/java/org/navalplanner/business/orders/entities/SchedulingState.java
View file

@ -138,7 +138,8 @@ public class SchedulingState {
public SchedulingState() {
}
public SchedulingState(List<SchedulingState> children) {
public SchedulingState(Type type, List<SchedulingState> children) {
this(type);
for (SchedulingState each : children) {
if (!each.isRoot()) {
throw new IllegalArgumentException(each

2
navalplanner-business/src/test/java/org/navalplanner/business/test/orders/entities/SchedulingStateTest.java
View file

@ -91,7 +91,7 @@ public class SchedulingStateTest {
@Test(expected = IllegalArgumentException.class)
public void cannotCreateASchedulingStateWithChildrenAlreadyAssigned() {
new SchedulingState(Arrays.asList(childA));
new SchedulingState(Type.NO_SCHEDULED, Arrays.asList(childA));
}
@Test