checkbook tests finished

.idea/workspace.xml

@@ -2,6 +2,8 @@
 <project version="4">
   <component name="ChangeListManager">
     <list default="true" id="926ad23d-2ddc-41ae-980d-780dcc1811fa" name="Default" comment="">
+      <change beforePath="$PROJECT_DIR$/Checkbook/src/main/java/Account.java" beforeDir="false" afterPath="$PROJECT_DIR$/Checkbook/src/main/java/Account.java" afterDir="false" />
+      <change beforePath="$PROJECT_DIR$/Checkbook/src/main/java/Checkbook.java" beforeDir="false" afterPath="$PROJECT_DIR$/Checkbook/src/main/java/Checkbook.java" afterDir="false" />
       <change beforePath="$PROJECT_DIR$/Checkbook/src/test/java/CheckbookTest.java" beforeDir="false" afterPath="$PROJECT_DIR$/Checkbook/src/test/java/CheckbookTest.java" afterDir="false" />
     </list>
     <ignored path="$PROJECT_DIR$/Checkbook/target/" />
@@ -22,13 +24,13 @@
       <usages-collector id="statistics.file.types.open">
         <counts>
           <entry key="CLASS" value="6" />
-          <entry key="JAVA" value="44" />
+          <entry key="JAVA" value="45" />
           <entry key="Markdown" value="1" />
         </counts>
       </usages-collector>
       <usages-collector id="statistics.file.types.edit">
         <counts>
-          <entry key="JAVA" value="9750" />
+          <entry key="JAVA" value="11342" />
           <entry key="Markdown" value="11" />
         </counts>
       </usages-collector>
@@ -40,8 +42,8 @@
         <entry file="file://$PROJECT_DIR$/README.md">
           <provider selected="true" editor-type-id="split-provider[text-editor;markdown-preview-editor]">
             <state split_layout="FIRST">
-              <first_editor relative-caret-position="283">
-                <caret line="48" lean-forward="true" selection-start-line="48" selection-end-line="48" />
+              <first_editor relative-caret-position="214">
+                <caret line="45" column="14" lean-forward="true" selection-start-line="45" selection-start-column="14" selection-end-line="45" selection-end-column="14" />
               </first_editor>
               <second_editor />
             </state>
@@ -51,7 +53,7 @@
       <file pinned="false" current-in-tab="false">
         <entry file="file://$PROJECT_DIR$/Checkbook/src/main/java/Transaction.java">
           <provider selected="true" editor-type-id="text-editor">
-            <state relative-caret-position="-279">
+            <state relative-caret-position="-438">
               <caret line="12" column="26" lean-forward="true" selection-start-line="12" selection-start-column="26" selection-end-line="12" selection-end-column="26" />
               <folding>
                 <element signature="imports" expanded="true" />
@@ -63,13 +65,12 @@
       <file pinned="false" current-in-tab="false">
         <entry file="file://$PROJECT_DIR$/Checkbook/src/main/java/Checkbook.java">
           <provider selected="true" editor-type-id="text-editor">
-            <state relative-caret-position="-1294">
-              <caret line="45" lean-forward="true" selection-start-line="45" selection-end-line="45" />
+            <state relative-caret-position="169">
+              <caret line="47" column="45" lean-forward="true" selection-start-line="47" selection-start-column="45" selection-end-line="47" selection-end-column="45" />
               <folding>
                 <element signature="imports" expanded="true" />
                 <element signature="e#1262#1263#0" expanded="true" />
                 <element signature="e#1316#1317#0" expanded="true" />
-                <element signature="e#1584#1585#0" expanded="true" />
                 <element signature="e#1641#1642#0" expanded="true" />
                 <element signature="e#2881#2882#0" expanded="true" />
                 <element signature="e#2928#2929#0" expanded="true" />
@@ -79,6 +80,10 @@
                 <element signature="e#3116#3117#0" expanded="true" />
                 <element signature="e#3136#3137#0" expanded="true" />
                 <element signature="e#3169#3170#0" expanded="true" />
+                <element signature="e#2396#2397#0" expanded="true" />
+                <element signature="e#2443#2444#0" expanded="true" />
+                <element signature="e#2528#2529#0" expanded="true" />
+                <element signature="e#2585#2586#0" expanded="true" />
                 <element signature="e#3541#3542#0" expanded="true" />
                 <element signature="e#3610#3611#0" expanded="true" />
                 <element signature="e#3665#3666#0" expanded="true" />
@@ -91,8 +96,8 @@
       <file pinned="false" current-in-tab="true">
         <entry file="file://$PROJECT_DIR$/Checkbook/src/test/java/CheckbookTest.java">
           <provider selected="true" editor-type-id="text-editor">
-            <state relative-caret-position="362">
-              <caret line="354" column="94" selection-start-line="354" selection-start-column="94" selection-end-line="354" selection-end-column="94" />
+            <state relative-caret-position="238">
+              <caret line="365" column="48" selection-start-line="365" selection-start-column="48" selection-end-line="365" selection-end-column="48" />
               <folding>
                 <element signature="imports" expanded="true" />
               </folding>
@@ -101,6 +106,15 @@
         </entry>
       </file>
       <file pinned="false" current-in-tab="false">
+        <entry file="file://$PROJECT_DIR$/Checkbook/src/main/java/Account.java">
+          <provider selected="true" editor-type-id="text-editor">
+            <state relative-caret-position="23">
+              <caret line="1" column="1" lean-forward="true" selection-start-line="1" selection-start-column="1" selection-end-line="1" selection-end-column="1" />
+            </state>
+          </provider>
+        </entry>
+      </file>
+      <file pinned="false" current-in-tab="false">
         <entry file="file://$PROJECT_DIR$/Checkbook/src/main/java/Payee.java">
           <provider selected="true" editor-type-id="text-editor">
             <state relative-caret-position="241">
@@ -148,9 +162,9 @@
         <option value="$PROJECT_DIR$/Checkbook/src/main/java/TransactionType.java" />
         <option value="$PROJECT_DIR$/Checkbook/src/main/java/AccountBook.java" />
         <option value="$PROJECT_DIR$/README.md" />
-        <option value="$PROJECT_DIR$/Checkbook/src/main/java/Account.java" />
         <option value="$PROJECT_DIR$/Checkbook/src/main/java/Transaction.java" />
         <option value="$PROJECT_DIR$/Checkbook/src/main/java/Payee.java" />
+        <option value="$PROJECT_DIR$/Checkbook/src/main/java/Account.java" />
         <option value="$PROJECT_DIR$/Checkbook/src/main/java/Checkbook.java" />
         <option value="$PROJECT_DIR$/Checkbook/src/test/java/CheckbookTest.java" />
       </list>
@@ -238,7 +252,7 @@
   <component name="PropertiesComponent">
     <property name="WebServerToolWindowFactoryState" value="false" />
     <property name="aspect.path.notification.shown" value="true" />
-    <property name="com.android.tools.idea.instantapp.provision.ProvisionBeforeRunTaskProvider.myTimeStamp" value="1544135882851" />
+    <property name="com.android.tools.idea.instantapp.provision.ProvisionBeforeRunTaskProvider.myTimeStamp" value="1544138934248" />
     <property name="com.intellij.testIntegration.createTest.CreateTestDialog.defaultLibrary" value="JUnit4" />
     <property name="extract.method.default.visibility" value="private" />
     <property name="jdk.selected.JAVA_MODULE" value="1.8" />
@@ -269,7 +283,7 @@
       </list>
     </option>
   </component>
-  <component name="RunManager" selected="JUnit.CheckbookTest.testGetCreditsafterAdding">
+  <component name="RunManager" selected="JUnit.CheckbookTest.testRemoveAllAddNewTransactions">
     <configuration default="true" type="Applet" factoryName="Applet">
       <option name="POLICY_FILE" value="$APPLICATION_HOME_DIR$/bin/appletviewer.policy" />
       <method v="2">
@@ -282,55 +296,55 @@
         <option name="Make" enabled="true" />
       </method>
     </configuration>
-    <configuration name="CheckbookTest.testGetCredits" type="JUnit" factoryName="JUnit" temporary="true" nameIsGenerated="true">
+    <configuration name="CheckbookTest.testClearAddAndRemove" type="JUnit" factoryName="JUnit" temporary="true" nameIsGenerated="true">
       <module name="Checkbook" />
       <option name="PACKAGE_NAME" value="" />
       <option name="MAIN_CLASS_NAME" value="CheckbookTest" />
-      <option name="METHOD_NAME" value="testGetCredits" />
+      <option name="METHOD_NAME" value="testClearAddAndRemove" />
       <option name="TEST_OBJECT" value="method" />
       <option name="WORKING_DIRECTORY" value="%MODULE_WORKING_DIR%" />
       <method v="2">
         <option name="Make" enabled="true" />
       </method>
     </configuration>
-    <configuration name="CheckbookTest.testGetCreditsNone" type="JUnit" factoryName="JUnit" temporary="true" nameIsGenerated="true">
+    <configuration name="CheckbookTest.testClearEmpty" type="JUnit" factoryName="JUnit" temporary="true" nameIsGenerated="true">
       <module name="Checkbook" />
       <option name="PACKAGE_NAME" value="" />
       <option name="MAIN_CLASS_NAME" value="CheckbookTest" />
-      <option name="METHOD_NAME" value="testGetCreditsNone" />
+      <option name="METHOD_NAME" value="testClearEmpty" />
       <option name="TEST_OBJECT" value="method" />
       <option name="WORKING_DIRECTORY" value="%MODULE_WORKING_DIR%" />
       <method v="2">
         <option name="Make" enabled="true" />
       </method>
     </configuration>
-    <configuration name="CheckbookTest.testGetCreditsafterAdding" type="JUnit" factoryName="JUnit" temporary="true" nameIsGenerated="true">
+    <configuration name="CheckbookTest.testRemoveAll" type="JUnit" factoryName="JUnit" temporary="true" nameIsGenerated="true">
       <module name="Checkbook" />
       <option name="PACKAGE_NAME" value="" />
       <option name="MAIN_CLASS_NAME" value="CheckbookTest" />
-      <option name="METHOD_NAME" value="testGetCreditsafterAdding" />
+      <option name="METHOD_NAME" value="testRemoveAll" />
       <option name="TEST_OBJECT" value="method" />
       <option name="WORKING_DIRECTORY" value="%MODULE_WORKING_DIR%" />
       <method v="2">
         <option name="Make" enabled="true" />
       </method>
     </configuration>
-    <configuration name="CheckbookTest.testGetDebitsManipulatedCheckbook" type="JUnit" factoryName="JUnit" temporary="true" nameIsGenerated="true">
+    <configuration name="CheckbookTest.testRemoveAllAddNewTransactions" type="JUnit" factoryName="JUnit" temporary="true" nameIsGenerated="true">
       <module name="Checkbook" />
       <option name="PACKAGE_NAME" value="" />
       <option name="MAIN_CLASS_NAME" value="CheckbookTest" />
-      <option name="METHOD_NAME" value="testGetDebitsManipulatedCheckbook" />
+      <option name="METHOD_NAME" value="testRemoveAllAddNewTransactions" />
       <option name="TEST_OBJECT" value="method" />
       <option name="WORKING_DIRECTORY" value="%MODULE_WORKING_DIR%" />
       <method v="2">
         <option name="Make" enabled="true" />
       </method>
     </configuration>
-    <configuration name="CheckbookTest.testToArrayFailV2" type="JUnit" factoryName="JUnit" temporary="true" nameIsGenerated="true">
+    <configuration name="CheckbookTest.testRemoveAllNone" type="JUnit" factoryName="JUnit" temporary="true" nameIsGenerated="true">
       <module name="Checkbook" />
       <option name="PACKAGE_NAME" value="" />
       <option name="MAIN_CLASS_NAME" value="CheckbookTest" />
-      <option name="METHOD_NAME" value="testToArrayFailV2" />
+      <option name="METHOD_NAME" value="testRemoveAllNone" />
       <option name="TEST_OBJECT" value="method" />
       <option name="WORKING_DIRECTORY" value="%MODULE_WORKING_DIR%" />
       <method v="2">
@@ -361,19 +375,19 @@
       </method>
     </configuration>
     <list>
-      <item itemvalue="JUnit.CheckbookTest.testGetDebitsManipulatedCheckbook" />
-      <item itemvalue="JUnit.CheckbookTest.testToArrayFailV2" />
-      <item itemvalue="JUnit.CheckbookTest.testGetCredits" />
-      <item itemvalue="JUnit.CheckbookTest.testGetCreditsNone" />
-      <item itemvalue="JUnit.CheckbookTest.testGetCreditsafterAdding" />
+      <item itemvalue="JUnit.CheckbookTest.testClearEmpty" />
+      <item itemvalue="JUnit.CheckbookTest.testClearAddAndRemove" />
+      <item itemvalue="JUnit.CheckbookTest.testRemoveAll" />
+      <item itemvalue="JUnit.CheckbookTest.testRemoveAllNone" />
+      <item itemvalue="JUnit.CheckbookTest.testRemoveAllAddNewTransactions" />
     </list>
     <recent_temporary>
       <list>
-        <item itemvalue="JUnit.CheckbookTest.testGetCreditsafterAdding" />
-        <item itemvalue="JUnit.CheckbookTest.testGetCreditsNone" />
-        <item itemvalue="JUnit.CheckbookTest.testGetCredits" />
-        <item itemvalue="JUnit.CheckbookTest.testToArrayFailV2" />
-        <item itemvalue="JUnit.CheckbookTest.testGetDebitsManipulatedCheckbook" />
+        <item itemvalue="JUnit.CheckbookTest.testRemoveAllAddNewTransactions" />
+        <item itemvalue="JUnit.CheckbookTest.testRemoveAllNone" />
+        <item itemvalue="JUnit.CheckbookTest.testRemoveAll" />
+        <item itemvalue="JUnit.CheckbookTest.testClearAddAndRemove" />
+        <item itemvalue="JUnit.CheckbookTest.testClearEmpty" />
       </list>
     </recent_temporary>
   </component>
@@ -388,44 +402,44 @@
       <option name="presentableId" value="Default" />
       <updated>1543958631974</updated>
       <workItem from="1543958633447" duration="3259000" />
-      <workItem from="1544046850600" duration="24772000" />
+      <workItem from="1544046850600" duration="27867000" />
     </task>
     <servers />
   </component>
   <component name="TestHistory">
-    <history-entry file="CheckbookTest_getTransactionsForPayeeExistsAfterAdd - 2018.12.06 at 14h 24m 20s.xml">
-      <configuration name="CheckbookTest.getTransactionsForPayeeExistsAfterAdd" configurationId="JUnit" />
+    <history-entry file="CheckbookTest_testContainsNone - 2018.12.06 at 17h 51m 55s.xml">
+      <configuration name="CheckbookTest.testContainsNone" configurationId="JUnit" />
     </history-entry>
-    <history-entry file="CheckbookTest_getTransactionsForPayeeExistsAfterAdd - 2018.12.06 at 14h 24m 50s.xml">
-      <configuration name="CheckbookTest.getTransactionsForPayeeExistsAfterAdd" configurationId="JUnit" />
+    <history-entry file="CheckbookTest_testContainsAllAddThenCheck - 2018.12.06 at 17h 53m 50s.xml">
+      <configuration name="CheckbookTest.testContainsAllAddThenCheck" configurationId="JUnit" />
     </history-entry>
-    <history-entry file="CheckbookTest_getTransactionAddingNewTForDateA - 2018.12.06 at 14h 27m 07s.xml">
-      <configuration name="CheckbookTest.getTransactionAddingNewTForDateA" configurationId="JUnit" />
+    <history-entry file="CheckbookTest_testAddAll - 2018.12.06 at 17h 58m 31s.xml">
+      <configuration name="CheckbookTest.testAddAll" configurationId="JUnit" />
     </history-entry>
-    <history-entry file="CheckbookTest_getTransactionsForPayeeExists - 2018.12.06 at 14h 31m 23s.xml">
-      <configuration name="CheckbookTest.getTransactionsForPayeeExists" configurationId="JUnit" />
+    <history-entry file="CheckbookTest_testAddAllNone - 2018.12.06 at 17h 59m 27s.xml">
+      <configuration name="CheckbookTest.testAddAllNone" configurationId="JUnit" />
     </history-entry>
-    <history-entry file="CheckbookTest_testGetDebits - 2018.12.06 at 17h 31m 07s.xml">
-      <configuration name="CheckbookTest.testGetDebits" configurationId="JUnit" />
+    <history-entry file="CheckbookTest_testAddAllOtherCollection - 2018.12.06 at 18h 03m 32s.xml">
+      <configuration name="CheckbookTest.testAddAllOtherCollection" configurationId="JUnit" />
     </history-entry>
-    <history-entry file="CheckbookTest_testGetDebits - 2018.12.06 at 17h 31m 20s.xml">
-      <configuration name="CheckbookTest.testGetDebits" configurationId="JUnit" />
+    <history-entry file="CheckbookTest_testClear - 2018.12.06 at 18h 09m 32s.xml">
+      <configuration name="CheckbookTest.testClear" configurationId="JUnit" />
     </history-entry>
-    <history-entry file="CheckbookTest_testGetDebits - 2018.12.06 at 17h 32m 48s.xml">
-      <configuration name="CheckbookTest.testGetDebits" configurationId="JUnit" />
+    <history-entry file="CheckbookTest_testClearEmpty - 2018.12.06 at 18h 10m 23s.xml">
+      <configuration name="CheckbookTest.testClearEmpty" configurationId="JUnit" />
     </history-entry>
-    <history-entry file="CheckbookTest_testGetDebitsNone - 2018.12.06 at 17h 34m 21s.xml">
-      <configuration name="CheckbookTest.testGetDebitsNone" configurationId="JUnit" />
+    <history-entry file="CheckbookTest_testClearAddAndRemove - 2018.12.06 at 18h 12m 07s.xml">
+      <configuration name="CheckbookTest.testClearAddAndRemove" configurationId="JUnit" />
     </history-entry>
-    <history-entry file="CheckbookTest_testGetDebitsManipulatedCheckbook - 2018.12.06 at 17h 35m 23s.xml">
-      <configuration name="CheckbookTest.testGetDebitsManipulatedCheckbook" configurationId="JUnit" />
+    <history-entry file="CheckbookTest_testRemoveAll - 2018.12.06 at 18h 14m 56s.xml">
+      <configuration name="CheckbookTest.testRemoveAll" configurationId="JUnit" />
     </history-entry>
-    <history-entry file="CheckbookTest_testToArrayFailV2 - 2018.12.06 at 17h 36m 26s.xml">
-      <configuration name="CheckbookTest.testToArrayFailV2" configurationId="JUnit" />
+    <history-entry file="CheckbookTest_testRemoveAllNone - 2018.12.06 at 18h 25m 04s.xml">
+      <configuration name="CheckbookTest.testRemoveAllNone" configurationId="JUnit" />
     </history-entry>
   </component>
   <component name="TimeTrackingManager">
-    <option name="totallyTimeSpent" value="28031000" />
+    <option name="totallyTimeSpent" value="31126000" />
   </component>
   <component name="ToolWindowManager">
     <frame x="0" y="23" width="1440" height="877" extended-state="0" />
@@ -553,13 +567,6 @@
         </state>
       </provider>
     </entry>
-    <entry file="file://$PROJECT_DIR$/Checkbook/src/main/java/Account.java">
-      <provider selected="true" editor-type-id="text-editor">
-        <state relative-caret-position="46">
-          <caret line="2" lean-forward="true" selection-start-line="2" selection-end-line="2" />
-        </state>
-      </provider>
-    </entry>
     <entry file="file://$PROJECT_DIR$/Checkbook/src/main/java/AccountBook.java">
       <provider selected="true" editor-type-id="text-editor">
         <state relative-caret-position="46">
@@ -601,7 +608,7 @@
     </entry>
     <entry file="file://$PROJECT_DIR$/Checkbook/src/main/java/Transaction.java">
       <provider selected="true" editor-type-id="text-editor">
-        <state relative-caret-position="-279">
+        <state relative-caret-position="-438">
           <caret line="12" column="26" lean-forward="true" selection-start-line="12" selection-start-column="26" selection-end-line="12" selection-end-column="26" />
           <folding>
             <element signature="imports" expanded="true" />
@@ -612,22 +619,28 @@
     <entry file="file://$PROJECT_DIR$/README.md">
       <provider selected="true" editor-type-id="split-provider[text-editor;markdown-preview-editor]">
         <state split_layout="FIRST">
-          <first_editor relative-caret-position="283">
-            <caret line="48" lean-forward="true" selection-start-line="48" selection-end-line="48" />
+          <first_editor relative-caret-position="214">
+            <caret line="45" column="14" lean-forward="true" selection-start-line="45" selection-start-column="14" selection-end-line="45" selection-end-column="14" />
           </first_editor>
           <second_editor />
         </state>
       </provider>
     </entry>
+    <entry file="file://$PROJECT_DIR$/Checkbook/src/main/java/Account.java">
+      <provider selected="true" editor-type-id="text-editor">
+        <state relative-caret-position="23">
+          <caret line="1" column="1" lean-forward="true" selection-start-line="1" selection-start-column="1" selection-end-line="1" selection-end-column="1" />
+        </state>
+      </provider>
+    </entry>
     <entry file="file://$PROJECT_DIR$/Checkbook/src/main/java/Checkbook.java">
       <provider selected="true" editor-type-id="text-editor">
-        <state relative-caret-position="-1294">
-          <caret line="45" lean-forward="true" selection-start-line="45" selection-end-line="45" />
+        <state relative-caret-position="169">
+          <caret line="47" column="45" lean-forward="true" selection-start-line="47" selection-start-column="45" selection-end-line="47" selection-end-column="45" />
           <folding>
             <element signature="imports" expanded="true" />
             <element signature="e#1262#1263#0" expanded="true" />
             <element signature="e#1316#1317#0" expanded="true" />
-            <element signature="e#1584#1585#0" expanded="true" />
             <element signature="e#1641#1642#0" expanded="true" />
             <element signature="e#2881#2882#0" expanded="true" />
             <element signature="e#2928#2929#0" expanded="true" />
@@ -637,6 +650,10 @@
             <element signature="e#3116#3117#0" expanded="true" />
             <element signature="e#3136#3137#0" expanded="true" />
             <element signature="e#3169#3170#0" expanded="true" />
+            <element signature="e#2396#2397#0" expanded="true" />
+            <element signature="e#2443#2444#0" expanded="true" />
+            <element signature="e#2528#2529#0" expanded="true" />
+            <element signature="e#2585#2586#0" expanded="true" />
             <element signature="e#3541#3542#0" expanded="true" />
             <element signature="e#3610#3611#0" expanded="true" />
             <element signature="e#3665#3666#0" expanded="true" />
@@ -647,8 +664,8 @@
     </entry>
     <entry file="file://$PROJECT_DIR$/Checkbook/src/test/java/CheckbookTest.java">
       <provider selected="true" editor-type-id="text-editor">
-        <state relative-caret-position="362">
-          <caret line="354" column="94" selection-start-line="354" selection-start-column="94" selection-end-line="354" selection-end-column="94" />
+        <state relative-caret-position="238">
+          <caret line="365" column="48" selection-start-line="365" selection-start-column="48" selection-end-line="365" selection-end-column="48" />
           <folding>
             <element signature="imports" expanded="true" />
           </folding>

Checkbook/src/main/java/Account.java

@@ -1,2 +1,2 @@
-public class Account<T extends AccountBook> {
+public class Account {
 }

Checkbook/src/main/java/Checkbook.java

@@ -2,6 +2,7 @@
 import java.lang.reflect.Field;
 import java.time.LocalDate;
 import java.util.*;
+import java.util.regex.Pattern;
 
 public class Checkbook {
     // Checkbook is a list of Transactions
@@ -28,43 +29,43 @@ public class Checkbook {
 
     //blind us with your Java eloquence....
 
-    public int size() {
+    int size() {
         return this.checkbook.size();
     }
 
-    public boolean isEmpty() {
+    boolean isEmpty() {
         return this.checkbook.isEmpty();
     }
 
-    public boolean contains(Transaction transaction) {
+    boolean contains(Transaction transaction) {
         return this.checkbook.contains(transaction);
     }
 
-    public boolean add(Transaction transaction) {
+    boolean add(Transaction transaction) {
         return this.checkbook.add(transaction);
     }
 
-    public boolean remove(Transaction transaction) {
+    boolean remove(Transaction transaction) {
         return this.checkbook.remove(transaction);
     }
 
-    public Iterator<Transaction> iterator() {
+    Iterator<Transaction> iterator() {
         return this.checkbook.iterator();
     }
 
-    public Transaction[] toArray() {
+    Transaction[] toArray() {
         return checkbook.toArray(new Transaction[checkbook.size()]);
     }
 
-    public Transaction[] toArray(Transaction[] a) {
+    Transaction[] toArray(Transaction[] a) {
         return this.checkbook.toArray(a);
     }
 
-    public Transaction[] toArrayFromList(List<Transaction> list) {
+    Transaction[] toArrayFromList(List<Transaction> list) {
         return list.toArray(new Transaction[list.size()]);
     }
 
-    public double checkbookValue() {
+    double checkbookValue() {
         double sum = 0.0;
         for (Transaction t: this.checkbook) {
             sum += t.getAmount();
@@ -72,11 +73,43 @@ public class Checkbook {
         return sum;
     }
 
-    public List<Transaction> getCheckbook() {
+    List<Transaction> getCheckbook() {
         return checkbook;
     }
 
-    public Transaction[] getSpecificTransactions(String fieldName, Object value) {
+    boolean containsAll(Collection<Transaction> c) {
+        return checkbook.containsAll(c);
+    }
+
+    boolean addAll(Collection<Transaction> c) {
+        return checkbook.addAll(c);
+    }
+
+    boolean removeAll(Collection<Transaction> c) {
+        return checkbook.removeAll(c);
+    }
+
+    void clear() {
+        checkbook.clear();
+    }
+
+     Transaction[] getAllTransactionsForDate(LocalDate a) {
+        return getSpecificTransactions("date", a);
+    }
+
+    Transaction[] getAllTransactionsForPayee(Payee a) {
+        return getSpecificTransactions("payee", a);
+    }
+
+    Transaction[] getAllDebitTransactions() {
+        return getSpecificTransactions("typee", TransactionType.DEBIT);
+    }
+
+    Transaction[] getAllCreditTransactions() {
+        return getSpecificTransactions("typee", TransactionType.CREDIT);
+    }
+
+    private Transaction[] getSpecificTransactions(String fieldName, Object value) {
 
         List<Transaction> arr = new ArrayList<>();
         for (Transaction ts : this.checkbook) {
@@ -99,28 +132,4 @@ public class Checkbook {
         }
         return null;
     }
-
-    public boolean containsAll(Collection<Transaction> c) {
-        return checkbook.containsAll(c);
-    }
-
-    boolean addAll(Collection<Transaction> c) {
-        return checkbook.addAll(c);
-    }
-
-    boolean removeAll(Collection<Transaction> c) {
-        return checkbook.removeAll(c);
-    }
-
-    void clear() {
-        checkbook.clear();
-    }
-
-    public Transaction[] getAllDebitTransactions() {
-        return getSpecificTransactions("typee", TransactionType.DEBIT);
-    }
-
-    public Transaction[] getAllCreditTransactions() {
-        return getSpecificTransactions("typee", TransactionType.CREDIT);
-    }
 }

Checkbook/src/test/java/CheckbookTest.java

@@ -1,16 +1,16 @@
-import org.junit.Assert;
 import org.junit.Test;
 
 import java.time.LocalDate;
-import java.util.Arrays;
-import java.util.NoSuchElementException;
+import java.util.*;
 
 import static org.junit.Assert.*;
 
 public class CheckbookTest {
     private Checkbook checkbook, checkbookWithDefault, checkbookWithOwner;
     private Payee p1, p2, p3;
-    private Transaction t1, t2, t3;
+    private Transaction t1, t2, t3, t4;
+    private ArrayList<Transaction> outsideTransactions;
+
     private final double cashAmount = 500.00;
     private final double t2Amount = 100.00;
     private final double t3Amount = 50.00;
@@ -25,6 +25,9 @@ public class CheckbookTest {
         t1 = new Transaction("ZC Fund", p1, TransactionType.CREDIT, cashAmount);
         t2 = new Transaction("Allowance", p3, TransactionType.DEBIT, t2Amount);
         t3 = new Transaction("Food", p2, TransactionType.CREDIT, t3Amount);
+        t4 = new Transaction("meow", new Payee(), TransactionType.DEBIT, 10.00);
+        outsideTransactions = new ArrayList<>();
+
 
         checkbook = new Checkbook(t1, t2, t3);
         checkbookWithDefault = new Checkbook(t2Amount);
@@ -235,12 +238,21 @@ public class CheckbookTest {
 
         checkbookWithDefault.add(t);
 
-        for (Transaction ta: checkbookWithDefault.getSpecificTransactions("date", a)) {
+        for (Transaction ta: checkbookWithDefault.getAllTransactionsForDate(a)) {
             assertEquals(a, ta.getDate());
         }
     }
 
     @Test
+    public void getTransactionsForWrongDate() {
+        LocalDate a = LocalDate.of(2220, 10, 20);
+
+        for (Transaction t:checkbook.getAllTransactionsForDate(a)) {
+            assertNull(t);
+        }
+    }
+
+    @Test
     public void getTransactionsForPayeeExists() {
         Payee payee = new Payee();
         Transaction t = new Transaction("test", payee, TransactionType.DEBIT, 10.00);
@@ -250,69 +262,140 @@ public class CheckbookTest {
         checkbookWithDefault.add(t);
         checkbookWithOwner.add(t);
 
-        assertEquals(expArr, checkbook.getSpecificTransactions("payee", payee));
-        assertEquals(expArr, checkbookWithDefault.getSpecificTransactions("payee", payee));
-        assertEquals(expArr, checkbookWithOwner.getSpecificTransactions("payee", payee));
+        assertEquals(expArr, checkbook.getAllTransactionsForPayee(payee));
+        assertEquals(expArr, checkbookWithDefault.getAllTransactionsForPayee(payee));
+        assertEquals(expArr, checkbookWithOwner.getAllTransactionsForPayee(payee));
+    }
+
+    @Test
+    public void getTransactionsForPayeeNotExists() {
+        Payee p = new Payee();
+        Transaction[] arr = {};
+        assertEquals(arr, checkbook.getAllTransactionsForPayee(p));
     }
 
     @Test
     public void testContainsAll() {
+        ArrayList<Transaction> arr = new ArrayList<>();
+        arr.add(t1);
+        arr.add(t2);
 
+        assertTrue(checkbook.containsAll(arr));
     }
 
     @Test
     public void testContainsNone() {
+        outsideTransactions. add(t4);
 
+        assertFalse(checkbook.containsAll(outsideTransactions));
+        assertFalse(checkbookWithOwner.containsAll(outsideTransactions));
+        assertFalse(checkbookWithDefault.containsAll(outsideTransactions));
     }
 
     @Test
     public void testContainsAllAddThenCheck() {
 
+        outsideTransactions.add(t4);
+        checkbook.add(t4);
+        checkbookWithOwner.add(t4);
+        checkbookWithDefault.add(t4);
+
+        assertTrue(checkbook.containsAll(outsideTransactions));
+        assertTrue(checkbookWithOwner.containsAll(outsideTransactions));
+        assertTrue(checkbookWithDefault.containsAll(outsideTransactions));
     }
 
     @Test
     public void testAddAll() {
+        int expectedSize = 5;
+        outsideTransactions.add(t4);
+        outsideTransactions.add(t1);
+
+        checkbook.addAll(outsideTransactions);
 
+        assertEquals(expectedSize, checkbook.size());
     }
 
     @Test
     public void testAddAllNone() {
+        int expSize = 3;
 
+        outsideTransactions.clear();
+        checkbook.addAll(outsideTransactions);
+
+        assertEquals(expSize, checkbook.size());
     }
 
     @Test
-    public void testAddAllException() {
+    public void testAddAllOtherCollection() {
+        int expSize = 5;
+        Set<Transaction> set = new HashSet<>();
+        set.add(t1);
+        set.add(t4);
 
+        checkbook.addAll(set);
+        assertEquals(expSize, checkbook.size());
     }
 
     @Test
     public void testRemoveAll() {
+        List<Transaction> arrToRemove = new ArrayList<>();
+        arrToRemove.add(t1);
+        arrToRemove.add(t2);
+        arrToRemove.add(t3);
+
+        int expSize = 0;
 
+        checkbook.removeAll(arrToRemove);
+        assertEquals(expSize, checkbook.size());
     }
 
     @Test
     public void testRemoveAllNone() {
-
+        int size = 0;
+        Checkbook cb = new Checkbook();
+        cb.removeAll(new HashSet<>());
+        assertEquals(size, cb.size());
     }
 
     @Test
     public void testRemoveAllAddNewTransactions() {
-
+        int expSize = 2;
+        outsideTransactions.add(t1);
+        checkbook.removeAll(outsideTransactions);
+        assertEquals(expSize, checkbook.size());
     }
 
     @Test
     public void testClear() {
+        int expSize = 0;
+
+        checkbook.clear();
+        checkbookWithDefault.clear();
+        checkbookWithOwner.clear();
 
+        assertEquals(expSize, checkbook.size());
+        assertEquals(expSize, checkbookWithDefault.size());
+        assertEquals(expSize, checkbookWithOwner.size());
     }
 
     @Test
     public void testClearEmpty() {
+        int expSize = 0;
+        Transaction t = checkbookWithDefault.getCheckbook().get(0);
+        checkbookWithDefault.remove(t);
 
+        checkbookWithDefault.clear();
+        assertEquals(expSize, checkbookWithDefault.size());
     }
 
     @Test
     public void testClearAddAndRemove() {
+        int expSize = 0;
+        checkbookWithOwner.add(t4);
+        checkbookWithOwner.clear();
 
+        assertEquals(expSize, checkbookWithOwner.size());
     }
 
     @Test
@@ -322,11 +405,11 @@ public class CheckbookTest {
 
         Transaction[] expArr =  {d1, d2};
         checkbookWithDefault.add(d2);
-        assertEquals(expArr, checkbookWithDefault.getSpecificTransactions("typee", TransactionType.DEBIT));
+        assertEquals(expArr, checkbookWithDefault.getAllDebitTransactions());
 
         Transaction[] expArr2 = {d2, d2};
         checkbookWithOwner.add(d2);
-        assertEquals(expArr2, checkbookWithOwner.getSpecificTransactions("typee", TransactionType.DEBIT));
+        assertEquals(expArr2, checkbookWithOwner.getAllDebitTransactions());
     }
 
     @Test
@@ -335,7 +418,7 @@ public class CheckbookTest {
 
         checkbook.remove(t2);
 
-        assertEquals(arr, checkbook.getSpecificTransactions("typee", TransactionType.DEBIT));
+        assertEquals(arr, checkbook.getAllDebitTransactions());
     }
 
     @Test
@@ -345,20 +428,20 @@ public class CheckbookTest {
         checkbook.remove(t2);
         checkbook.add(t);
 
-        assertEquals(arr, checkbook.getSpecificTransactions("typee", TransactionType.DEBIT));
+        assertEquals(arr, checkbook.getAllDebitTransactions());
     }
 
     @Test
     public void testGetCredits() {
         Transaction[] expArr = {t1, t3};
-        assertEquals(expArr, checkbook.getSpecificTransactions("typee", TransactionType.CREDIT));
+        assertEquals(expArr, checkbook.getAllCreditTransactions());
     }
 
     @Test
     public void testGetCreditsNone() {
         Transaction[] expArr = {};
-        assertEquals(expArr, checkbookWithDefault.getSpecificTransactions("typee", TransactionType.CREDIT));
-        assertEquals(expArr, checkbookWithOwner.getSpecificTransactions("typee", TransactionType.CREDIT));
+        assertEquals(expArr, checkbookWithDefault.getAllCreditTransactions());
+        assertEquals(expArr, checkbookWithOwner.getAllCreditTransactions());
     }
 
     @Test
@@ -367,6 +450,6 @@ public class CheckbookTest {
         Transaction[] arr = {t};
 
         checkbookWithOwner.add(t);
-        assertEquals(arr, checkbookWithOwner.getSpecificTransactions("typee", TransactionType.CREDIT));
+        assertEquals(arr, checkbookWithOwner.getAllCreditTransactions());
     }
 }