diff --git a/model-command/src/test/java/lcsb/mapviewer/commands/CreateHierarchyCommandTest.java b/model-command/src/test/java/lcsb/mapviewer/commands/CreateHierarchyCommandTest.java index 940e93533368870a1cc083ffcf7172a34dae717c..4d733b555e86dd03627f2cc685471b0855d642aa 100644 --- a/model-command/src/test/java/lcsb/mapviewer/commands/CreateHierarchyCommandTest.java +++ b/model-command/src/test/java/lcsb/mapviewer/commands/CreateHierarchyCommandTest.java @@ -23,7 +23,6 @@ import lcsb.mapviewer.converter.model.celldesigner.CellDesignerXmlParser; import lcsb.mapviewer.model.map.compartment.Compartment; import lcsb.mapviewer.model.map.compartment.PathwayCompartment; import lcsb.mapviewer.model.map.model.Model; -import lcsb.mapviewer.model.map.species.Complex; import lcsb.mapviewer.model.map.species.Element; import lcsb.mapviewer.model.map.species.Species; @@ -131,7 +130,7 @@ public class CreateHierarchyCommandTest extends CommandTestFunctions { Model model = getModelForFile("testFiles/artifitial_compartments.xml", false); new CreateHierarchyCommand(model, 2, 2).execute(); - + assertFalse(model.getElementByElementId("sa1").getCompartment() instanceof PathwayCompartment); } catch (Exception e) { e.printStackTrace(); diff --git a/model/src/test/java/lcsb/mapviewer/model/map/compartment/CompartmentTest.java b/model/src/test/java/lcsb/mapviewer/model/map/compartment/CompartmentTest.java index 4d48aa7f1401bc4d70cebcd2cc0e43d716cc194c..bbe1ad98aa03eb51ff73b7b1d15a9f3a4802a5ca 100644 --- a/model/src/test/java/lcsb/mapviewer/model/map/compartment/CompartmentTest.java +++ b/model/src/test/java/lcsb/mapviewer/model/map/compartment/CompartmentTest.java @@ -152,6 +152,26 @@ public class CompartmentTest { } } + @Test + public void testDoesntContainsIdenticalSpecies() throws Exception { + try { + Compartment compartment = new Compartment(); + + compartment.setName("AS"); + + Species species = new GenericProtein("idd"); + species.setName("not identical name"); + + compartment.addElement(species); + + assertFalse(compartment.containsIdenticalSpecies()); + + } catch (Exception e) { + e.printStackTrace(); + throw e; + } + } + private Model createCrossModel() { Model model = new ModelFullIndexed(null); model.addElement(new Compartment("default")); diff --git a/model/src/test/java/lcsb/mapviewer/model/map/reaction/ReactionTest.java b/model/src/test/java/lcsb/mapviewer/model/map/reaction/ReactionTest.java index 94cd14d33a4e749f1fe4929de4a475e7b93715d6..7ea08b523f44e85ed38eec4b8868694849b6e19e 100644 --- a/model/src/test/java/lcsb/mapviewer/model/map/reaction/ReactionTest.java +++ b/model/src/test/java/lcsb/mapviewer/model/map/reaction/ReactionTest.java @@ -58,6 +58,7 @@ public class ReactionTest extends ModelTestFunctions { Modifier modifier = new Catalysis(species); reaction.addModifier(modifier); assertTrue(reaction.containsElement(species)); + assertFalse(reaction.containsElement(new GenericProtein("id"))); } catch (Exception e) { e.printStackTrace(); throw e; diff --git a/model/src/test/java/lcsb/mapviewer/model/map/species/ComplexTest.java b/model/src/test/java/lcsb/mapviewer/model/map/species/ComplexTest.java index 3990f8888ead163ee27f04407bf426f4c3ecda70..6e10c57191e1ddccfae1edbe6b65c013e8e34e8b 100644 --- a/model/src/test/java/lcsb/mapviewer/model/map/species/ComplexTest.java +++ b/model/src/test/java/lcsb/mapviewer/model/map/species/ComplexTest.java @@ -10,9 +10,10 @@ import org.junit.Before; import org.junit.Test; import org.mockito.Mockito; +import lcsb.mapviewer.ModelTestFunctions; import lcsb.mapviewer.common.exception.NotImplementedException; -public class ComplexTest { +public class ComplexTest extends ModelTestFunctions { @Before public void setUp() throws Exception { @@ -107,6 +108,24 @@ public class ComplexTest { } } + @Test + public void testRemoveNonExistingElement() { + try { + Complex complex = new Complex(); + Complex complex2 = new Complex(); + + Species protein = new GenericProtein("1"); + protein.setComplex(complex2); + complex.removeElement(protein); + + assertEquals(1, super.getWarnings().size()); + + } catch (Exception e) { + e.printStackTrace(); + throw e; + } + } + @Test public void testCopy() { try {