In an effort to get more familiar with the codebase, I have started to
write some unit tests for certain areas of the code. The first two
tests I've written are below and test the ToDoItem and ToDoItemList
classes. These classes are related to issue 4979 which I am still
stuck on per my earlier email
(http://argouml.tigris.org/issues/show_bug.cgi?id=4979).
Would someone with commit access be able to add these files the build?
If needed, I can provide them in a different format or attach them to
a new issue in the bug tracker.
Let me know if you want me to write tests for certain areas of the code.
Thanks,
Scott
New unit tests patch:
-------------------------------
Index: tests/org/argouml/cognitive/TestToDoItemList.java
===================================================================
--- tests/org/argouml/cognitive/TestToDoItemList.java (revision 0)
+++ tests/org/argouml/cognitive/TestToDoItemList.java (revision 0)
@@ -0,0 +1,105 @@
+// $Id
+// Copyright (c) 2008 The Regents of the University of California. All
+// Rights Reserved. Permission to use, copy, modify, and distribute this
+// software and its documentation without fee, and without a written
+// agreement is hereby granted, provided that the above copyright notice
+// and this paragraph appear in all copies. This software program and
+// documentation are copyrighted by The Regents of the University of
+// California. The software program and documentation are supplied "AS
+// IS", without any accompanying services from The Regents. The Regents
+// does not warrant that the operation of the program will be
+// uninterrupted or error-free. The end-user understands that the program
+// was developed for research purposes and is advised not to rely
+// exclusively on the program for any reason. IN NO EVENT SHALL THE
+// UNIVERSITY OF CALIFORNIA BE LIABLE TO ANY PARTY FOR DIRECT, INDIRECT,
+// SPECIAL, INCIDENTAL, OR CONSEQUENTIAL DAMAGES, INCLUDING LOST PROFITS,
+// ARISING OUT OF THE USE OF THIS SOFTWARE AND ITS DOCUMENTATION, EVEN IF
+// THE UNIVERSITY OF CALIFORNIA HAS BEEN ADVISED OF THE POSSIBILITY OF
+// SUCH DAMAGE. THE UNIVERSITY OF CALIFORNIA SPECIFICALLY DISCLAIMS ANY
+// WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF
+// MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE. THE SOFTWARE
+// PROVIDED HEREUNDER IS ON AN "AS IS" BASIS, AND THE UNIVERSITY OF
+// CALIFORNIA HAS NO OBLIGATIONS TO PROVIDE MAINTENANCE, SUPPORT,
+// UPDATES, ENHANCEMENTS, OR MODIFICATIONS.
+
+package org.argouml.cognitive;
+
+import junit.framework.TestCase;
+
+import org.argouml.model.InitializeModel;
+import org.argouml.cognitive.ToDoItem;
+import org.argouml.cognitive.ToDoList;
+
+/**
+ * Testing the creation of a ToDoItemList.
+ */
+public class TestToDoItemList extends TestCase {
+
+ /**
+ * The constructor.
+ *
+ * @param name the name of the test.
+ */
+ public TestToDoItemList(String arg0) {
+ super(arg0);
+ }
+
+ /*
+ * @see junit.framework.TestCase#setUp()
+ */
+ @Override
+ public void setUp() throws Exception {
+ super.setUp();
+
+ // This test calls ToDoList.addElement which requires the
+ // Model. We are able to get away with using the Mock model.
+ InitializeModel.initializeMock();
+ }
+
+
+ /**
+ * Test constructor and some basic methods.
+ */
+ public void testBasics() {
+
+ // initialize test data
+ int priority = ToDoItem.HIGH_PRIORITY;
+ String headline = "Test Headline";
+ String description = "Test Description";
+ String moreInfo = "http://argouml.tigris.org/test";;
+ Critic critic = new Critic();
+ critic.setHeadline(headline);
+ ToDoItem item = new ToDoItem(critic, headline, priority,
description, moreInfo);
+ ListSet<String> offenders = new ListSet<String>();
+ offenders.add("Offender # 1");
+ offenders.add("Offender # 2");
+ item.setOffenders(offenders);
+
+ // initialize the ToDoList
+ ToDoList list = new ToDoList();
+
+ // add a ToDoItem
+ list.addElement(item);
+
+ // redundantly add item again which should be ignored
+ list.addElement(item);
+
+ // size should be 1 since addition of second item would have
been ignored
+ assertTrue("ToDoList.size() is incorrect", list.size() == 1);
+
+ // size of offender list should be 2
+ assertTrue("ToDoList.getOffenders().size() is incorrect",
list.getOffenders().size() == 2);
+
+ // retrieve item
+ assertTrue("ToDoItem.get(0) did not return correct item",
list.size() > 0 && list.get(0) == item);
+
+ // remove item
+ assertTrue("ToDoItem.removeElement(item) failed to remove item",
list.removeElement(item));
+
+ // size should be 0 since item was removed
+ assertTrue("ToDoList.size() is incorrect", list.size() == 0);
+
+ // size of offender list should be 0 since item was removed
+ assertTrue("ToDoList.getOffenders().size() is incorrect",
list.getOffenders().size() == 0);
+ }
+}
Index: tests/org/argouml/cognitive/TestToDoItem.java
===================================================================
--- tests/org/argouml/cognitive/TestToDoItem.java (revision 0)
+++ tests/org/argouml/cognitive/TestToDoItem.java (revision 0)
@@ -0,0 +1,96 @@
+// $Id
+// Copyright (c) 2008 The Regents of the University of California. All
+// Rights Reserved. Permission to use, copy, modify, and distribute this
+// software and its documentation without fee, and without a written
+// agreement is hereby granted, provided that the above copyright notice
+// and this paragraph appear in all copies. This software program and
+// documentation are copyrighted by The Regents of the University of
+// California. The software program and documentation are supplied "AS
+// IS", without any accompanying services from The Regents. The Regents
+// does not warrant that the operation of the program will be
+// uninterrupted or error-free. The end-user understands that the program
+// was developed for research purposes and is advised not to rely
+// exclusively on the program for any reason. IN NO EVENT SHALL THE
+// UNIVERSITY OF CALIFORNIA BE LIABLE TO ANY PARTY FOR DIRECT, INDIRECT,
+// SPECIAL, INCIDENTAL, OR CONSEQUENTIAL DAMAGES, INCLUDING LOST PROFITS,
+// ARISING OUT OF THE USE OF THIS SOFTWARE AND ITS DOCUMENTATION, EVEN IF
+// THE UNIVERSITY OF CALIFORNIA HAS BEEN ADVISED OF THE POSSIBILITY OF
+// SUCH DAMAGE. THE UNIVERSITY OF CALIFORNIA SPECIFICALLY DISCLAIMS ANY
+// WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF
+// MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE. THE SOFTWARE
+// PROVIDED HEREUNDER IS ON AN "AS IS" BASIS, AND THE UNIVERSITY OF
+// CALIFORNIA HAS NO OBLIGATIONS TO PROVIDE MAINTENANCE, SUPPORT,
+// UPDATES, ENHANCEMENTS, OR MODIFICATIONS.
+
+package org.argouml.cognitive;
+
+import junit.framework.TestCase;
+
+
+import org.argouml.cognitive.ToDoItem;
+import org.argouml.cognitive.Critic;
+
+/**
+ * Testing the creation of a ToDoItem.
+ */
+public class TestToDoItem extends TestCase {
+
+ /**
+ * The constructor.
+ *
+ * @param name the name of the test.
+ */
+ public TestToDoItem(String name) {
+ super(name);
+ }
+
+ /**
+ * Test constructor and a some basic methods.
+ */
+ public void testBasics() {
+ // initialize test data
+ int priority = ToDoItem.HIGH_PRIORITY;
+ String headline = "Test Headline";
+ String description = "Test Description";
+ String moreInfo = "http://argouml.tigris.org/test";;
+ Critic critic = new Critic();
+ critic.setHeadline(headline);
+
+ // initialize new ToDoItem
+ ToDoItem item = new ToDoItem(critic, headline, priority,
description, moreInfo);
+
+ // test that properties were correctly initialized by the
constructor
+ assertTrue("ToDoItem.getHeadline() incorrect after initialized
constructor",
+ headline == item.getHeadline());
+ assertTrue("ToDoItem.getDescription() incorrect after
initialized
by constructor",
+ description == item.getDescription());
+ assertTrue("ToDoItem.getPriority() incorrect after initialized
by
constructor",
+ priority == item.getPriority());
+ assertTrue("ToDoItem.getMoreInfoURL() incorrect after
initialized
by constructor",
+ moreInfo == item.getMoreInfoURL());
+ assertTrue("((Critic)ToDoItem.getPoster()).getHeadline()
incorrect
after initialized by constructor",
+ headline ==
((Critic)item.getPoster()).getHeadline());
+
+ // reset the values to a different value
+ priority = ToDoItem.LOW_PRIORITY;
+ headline += " - Set Test";
+ description += " - Set Test";
+ moreInfo += "/settest.htm";
+
+ // update the properties of the ToDoItem
+ item.setHeadline(headline);
+ item.setPriority(priority);
+ item.setDescription(description);
+ item.setMoreInfoURL(moreInfo);
+
+ // test that properties were correctly set by the setters
+ assertTrue("ToDoItem.getHeadline() incorrect after
ToDoItem.setHeadline()",
+ headline == item.getHeadline());
+ assertTrue("ToDoItem.getDescription() incorrect after
ToDoItem.setDescription()",
+ description == item.getDescription());
+ assertTrue("ToDoItem.getPriority() incorrect after
ToDoItem.setPriority()",
+ priority == item.getPriority());
+ assertTrue("ToDoItem.getMoreInfoURL() incorrect after
ToDoItem.setMoreInfoURL()",
+ moreInfo == item.getMoreInfoURL());
+ }
+}
\ No newline at end of file
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
