Skip to content
Snippets Groups Projects
Commit 1d06ee5c authored by buczekp's avatar buczekp
Browse files

[LMS-2282] minor: changed error message

SVN: 21495
parent 9c5d76e8
No related branches found
No related tags found
No related merge requests found
...@@ -230,7 +230,7 @@ public class DataBO extends AbstractDataSetBusinessObject implements IDataBO ...@@ -230,7 +230,7 @@ public class DataBO extends AbstractDataSetBusinessObject implements IDataBO
{ {
final DataPE contained = getOrCreateData(containedCode, experiment); final DataPE contained = getOrCreateData(containedCode, experiment);
data.addComponent(contained); data.addComponent(contained);
checkSameSpace(data, contained); // needed for already existing data sets checkSameSpace(data, contained);
} }
} }
} }
...@@ -508,9 +508,9 @@ public class DataBO extends AbstractDataSetBusinessObject implements IDataBO ...@@ -508,9 +508,9 @@ public class DataBO extends AbstractDataSetBusinessObject implements IDataBO
{ {
return; return;
} }
throw UserFailureException.fromTemplate("Data set's '%s' space ('%s') needs to be the same" throw UserFailureException.fromTemplate(
+ " as its container's '%s' space ('%s').", component.getCode(), component "Data set '%s' must be in the same space ('%s') as its container.",
.getSpace().getCode(), container.getCode(), container.getSpace().getCode()); component.getCode(), container.getSpace().getCode());
} }
private void checkSameSpace(DataPE container, List<DataPE> components) private void checkSameSpace(DataPE container, List<DataPE> components)
......
...@@ -515,8 +515,8 @@ public class DataBOTest extends AbstractBOTest ...@@ -515,8 +515,8 @@ public class DataBOTest extends AbstractBOTest
fail("Expected UserFailureException"); fail("Expected UserFailureException");
} catch (UserFailureException ex) } catch (UserFailureException ex)
{ {
assertEquals("Data set's '" + COMPONENT_CODE + "' space ('S2') needs to be the same" assertEquals("Data set '" + COMPONENT_CODE
+ " as its container's 'DS1' space ('S1').", ex.getMessage()); + "' must be in the same space ('S1') as its container.", ex.getMessage());
} }
context.assertIsSatisfied(); context.assertIsSatisfied();
...@@ -644,7 +644,7 @@ public class DataBOTest extends AbstractBOTest ...@@ -644,7 +644,7 @@ public class DataBOTest extends AbstractBOTest
createDataSetUpdates(dataSet, null, EXPERIMENT_IDENTIFIER); createDataSetUpdates(dataSet, null, EXPERIMENT_IDENTIFIER);
String[] componentCodes = String[] componentCodes =
{ COMPONENT_CODE }; { COMPONENT_CODE };
dataSetUpdatesDTO.setModifiedParentDatasetCodesOrNull(componentCodes); dataSetUpdatesDTO.setModifiedContainedDatasetCodesOrNull(componentCodes);
prepareForUpdate(dataSet, experiment); prepareForUpdate(dataSet, experiment);
context.checking(new Expectations() context.checking(new Expectations()
{ {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment