Fixed copy()

.idea/workspace.xml

@@ -2,13 +2,7 @@
 <project version="4">
   <component name="ChangeListManager">
     <list default="true" id="4cada9f2-6d0f-46eb-ad0f-05b3102352a5" name="Default Changelist" comment="">
-      <change beforePath="$PROJECT_DIR$/.idea/compiler.xml" beforeDir="false" afterPath="$PROJECT_DIR$/.idea/compiler.xml" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/.idea/misc.xml" beforeDir="false" afterPath="$PROJECT_DIR$/.idea/misc.xml" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/.idea/modules.xml" beforeDir="false" afterPath="$PROJECT_DIR$/.idea/modules.xml" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/.idea/vcs.xml" beforeDir="false" afterPath="$PROJECT_DIR$/.idea/vcs.xml" afterDir="false" />
       <change beforePath="$PROJECT_DIR$/.idea/workspace.xml" beforeDir="false" afterPath="$PROJECT_DIR$/.idea/workspace.xml" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/pom.xml" beforeDir="false" afterPath="$PROJECT_DIR$/pom.xml" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/src/main/java/com/zipcodewilmington/singlylinkedlist/MainApplication.java" beforeDir="false" afterPath="$PROJECT_DIR$/src/main/java/com/zipcodewilmington/singlylinkedlist/MainApplication.java" afterDir="false" />
       <change beforePath="$PROJECT_DIR$/src/main/java/com/zipcodewilmington/singlylinkedlist/SinglyLinkedList.java" beforeDir="false" afterPath="$PROJECT_DIR$/src/main/java/com/zipcodewilmington/singlylinkedlist/SinglyLinkedList.java" afterDir="false" />
       <change beforePath="$PROJECT_DIR$/src/test/java/com/zipcodewilmington/singlylinkedlist/SinglyLinkedListTest.java" beforeDir="false" afterPath="$PROJECT_DIR$/src/test/java/com/zipcodewilmington/singlylinkedlist/SinglyLinkedListTest.java" afterDir="false" />
     </list>
@@ -23,12 +17,12 @@
     <session id="-472648142">
       <usages-collector id="statistics.lifecycle.project">
         <counts>
-          <entry key="project.closed" value="4" />
+          <entry key="project.closed" value="6" />
           <entry key="project.open.time.0" value="1" />
-          <entry key="project.open.time.1" value="2" />
+          <entry key="project.open.time.1" value="4" />
           <entry key="project.open.time.12" value="1" />
           <entry key="project.open.time.6" value="1" />
-          <entry key="project.opened" value="5" />
+          <entry key="project.opened" value="7" />
         </counts>
       </usages-collector>
       <usages-collector id="statistics.file.extensions.open">
@@ -43,18 +37,18 @@
       </usages-collector>
       <usages-collector id="statistics.file.extensions.edit">
         <counts>
-          <entry key="java" value="3825" />
+          <entry key="java" value="4009" />
         </counts>
       </usages-collector>
       <usages-collector id="statistics.file.types.edit">
         <counts>
-          <entry key="JAVA" value="3825" />
+          <entry key="JAVA" value="4009" />
         </counts>
       </usages-collector>
     </session>
   </component>
   <component name="FileEditorManager">
-    <splitter split-orientation="horizontal" split-proportion="0.43593314">
+    <splitter split-orientation="horizontal" split-proportion="0.45851216">
       <split-first>
         <leaf SIDE_TABS_SIZE_LIMIT_KEY="300">
           <file pinned="false" current-in-tab="false">
@@ -69,14 +63,8 @@
           <file pinned="false" current-in-tab="true">
             <entry file="file://$PROJECT_DIR$/src/main/java/com/zipcodewilmington/singlylinkedlist/SinglyLinkedList.java">
               <provider selected="true" editor-type-id="text-editor">
-                <state relative-caret-position="2235">
-                  <caret line="149" selection-start-line="149" selection-end-line="149" />
-                  <folding>
-                    <element signature="e#3864#3865#0" expanded="true" />
-                    <element signature="e#3896#3897#0" expanded="true" />
-                    <element signature="e#4115#4116#0" expanded="true" />
-                    <element signature="e#4155#4156#0" expanded="true" />
-                  </folding>
+                <state relative-caret-position="245">
+                  <caret line="124" column="29" lean-forward="true" selection-start-line="124" selection-start-column="29" selection-end-line="124" selection-end-column="35" />
                 </state>
               </provider>
             </entry>
@@ -93,8 +81,8 @@
           <file pinned="false" current-in-tab="false">
             <entry file="file://$PROJECT_DIR$/src/test/java/com/zipcodewilmington/singlylinkedlist/SinglyLinkedListTest.java">
               <provider selected="true" editor-type-id="text-editor">
-                <state relative-caret-position="165">
-                  <caret line="13" column="8" selection-start-line="13" selection-start-column="8" selection-end-line="13" selection-end-column="8" />
+                <state relative-caret-position="180">
+                  <caret line="12" column="8" selection-start-line="12" selection-start-column="8" selection-end-line="12" selection-end-column="8" />
                   <folding>
                     <element signature="imports" expanded="true" />
                   </folding>
@@ -109,8 +97,8 @@
           <file pinned="false" current-in-tab="true">
             <entry file="file://$PROJECT_DIR$/src/test/java/com/zipcodewilmington/singlylinkedlist/SinglyLinkedListTest.java">
               <provider selected="true" editor-type-id="text-editor">
-                <state relative-caret-position="299">
-                  <caret line="21" column="22" selection-start-line="21" selection-start-column="22" selection-end-line="21" selection-end-column="22" />
+                <state relative-caret-position="269">
+                  <caret line="193" column="9" lean-forward="true" selection-start-line="193" selection-start-column="9" selection-end-line="193" selection-end-column="9" />
                   <folding>
                     <element signature="imports" expanded="true" />
                   </folding>
@@ -121,7 +109,7 @@
           <file pinned="false" current-in-tab="false">
             <entry file="jar:///Library/Java/JavaVirtualMachines/jdk1.8.0_191.jdk/Contents/Home/src.zip!/java/lang/Comparable.java">
               <provider selected="true" editor-type-id="text-editor">
-                <state relative-caret-position="-216">
+                <state relative-caret-position="1080">
                   <caret line="95" column="17" selection-start-line="95" selection-start-column="17" selection-end-line="95" selection-end-column="17" />
                 </state>
               </provider>
@@ -231,7 +219,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="1541259452760" />
+    <property name="com.android.tools.idea.instantapp.provision.ProvisionBeforeRunTaskProvider.myTimeStamp" value="1541262400955" />
     <property name="nodejs_interpreter_path.stuck_in_default_project" value="undefined stuck path" />
     <property name="nodejs_npm_path_reset_for_default_project" value="true" />
   </component>
@@ -247,7 +235,7 @@
       </list>
     </option>
   </component>
-  <component name="RunManager" selected="JUnit.SinglyLinkedListTest">
+  <component name="RunManager" selected="JUnit.SinglyLinkedListTest.SinglyLinkedListTest11">
     <configuration name="SinglyLinkedListTest" type="JUnit" factoryName="JUnit" temporary="true" nameIsGenerated="true">
       <module name="singlylinkedlist" />
       <extension name="coverage">
@@ -335,9 +323,9 @@
     </list>
     <recent_temporary>
       <list>
+        <item itemvalue="JUnit.SinglyLinkedListTest.SinglyLinkedListTest11" />
         <item itemvalue="JUnit.SinglyLinkedListTest" />
         <item itemvalue="JUnit.SinglyLinkedListTest.SinglyLinkedListTest12" />
-        <item itemvalue="JUnit.SinglyLinkedListTest.SinglyLinkedListTest11" />
         <item itemvalue="JUnit.SinglyLinkedListTest.SinglyLinkedListTest10" />
         <item itemvalue="JUnit.SinglyLinkedListTest.SinglyLinkedListTest9" />
       </list>
@@ -357,29 +345,13 @@
       <workItem from="1541027519485" duration="36000" />
       <workItem from="1541092107678" duration="2198000" />
       <workItem from="1541196317502" duration="5339000" />
-      <workItem from="1541234299179" duration="5970000" />
+      <workItem from="1541234299179" duration="6045000" />
+      <workItem from="1541259898121" duration="48000" />
+      <workItem from="1541261768122" duration="1211000" />
     </task>
     <servers />
   </component>
   <component name="TestHistory">
-    <history-entry file="SinglyLinkedListTest_SinglyLinkedListTest12 - 2018.11.03 at 11h 14m 18s.xml">
-      <configuration name="SinglyLinkedListTest.SinglyLinkedListTest12" configurationId="JUnit" />
-    </history-entry>
-    <history-entry file="SinglyLinkedListTest_SinglyLinkedListTest12 - 2018.11.03 at 11h 18m 27s.xml">
-      <configuration name="SinglyLinkedListTest.SinglyLinkedListTest12" configurationId="JUnit" />
-    </history-entry>
-    <history-entry file="SinglyLinkedListTest_SinglyLinkedListTest12 - 2018.11.03 at 11h 19m 59s.xml">
-      <configuration name="SinglyLinkedListTest.SinglyLinkedListTest12" configurationId="JUnit" />
-    </history-entry>
-    <history-entry file="SinglyLinkedListTest_SinglyLinkedListTest12 - 2018.11.03 at 11h 20m 43s.xml">
-      <configuration name="SinglyLinkedListTest.SinglyLinkedListTest12" configurationId="JUnit" />
-    </history-entry>
-    <history-entry file="SinglyLinkedListTest_SinglyLinkedListTest12 - 2018.11.03 at 11h 20m 55s.xml">
-      <configuration name="SinglyLinkedListTest.SinglyLinkedListTest12" configurationId="JUnit" />
-    </history-entry>
-    <history-entry file="SinglyLinkedListTest - 2018.11.03 at 11h 28m 21s.xml">
-      <configuration name="SinglyLinkedListTest" configurationId="JUnit" />
-    </history-entry>
     <history-entry file="SinglyLinkedListTest - 2018.11.03 at 11h 28m 39s.xml">
       <configuration name="SinglyLinkedListTest" configurationId="JUnit" />
     </history-entry>
@@ -392,9 +364,27 @@
     <history-entry file="SinglyLinkedListTest - 2018.11.03 at 11h 29m 51s.xml">
       <configuration name="SinglyLinkedListTest" configurationId="JUnit" />
     </history-entry>
+    <history-entry file="SinglyLinkedListTest_SinglyLinkedListTest11 - 2018.11.03 at 12h 21m 35s.xml">
+      <configuration name="SinglyLinkedListTest.SinglyLinkedListTest11" configurationId="JUnit" />
+    </history-entry>
+    <history-entry file="SinglyLinkedListTest_SinglyLinkedListTest11 - 2018.11.03 at 12h 23m 01s.xml">
+      <configuration name="SinglyLinkedListTest.SinglyLinkedListTest11" configurationId="JUnit" />
+    </history-entry>
+    <history-entry file="SinglyLinkedListTest_SinglyLinkedListTest11 - 2018.11.03 at 12h 23m 30s.xml">
+      <configuration name="SinglyLinkedListTest.SinglyLinkedListTest11" configurationId="JUnit" />
+    </history-entry>
+    <history-entry file="SinglyLinkedListTest_SinglyLinkedListTest11 - 2018.11.03 at 12h 24m 06s.xml">
+      <configuration name="SinglyLinkedListTest.SinglyLinkedListTest11" configurationId="JUnit" />
+    </history-entry>
+    <history-entry file="SinglyLinkedListTest_SinglyLinkedListTest11 - 2018.11.03 at 12h 24m 28s.xml">
+      <configuration name="SinglyLinkedListTest.SinglyLinkedListTest11" configurationId="JUnit" />
+    </history-entry>
+    <history-entry file="SinglyLinkedListTest_SinglyLinkedListTest11 - 2018.11.03 at 12h 24m 56s.xml">
+      <configuration name="SinglyLinkedListTest.SinglyLinkedListTest11" configurationId="JUnit" />
+    </history-entry>
   </component>
   <component name="TimeTrackingManager">
-    <option name="totallyTimeSpent" value="15139000" />
+    <option name="totallyTimeSpent" value="16473000" />
   </component>
   <component name="ToolWindowManager">
     <frame x="0" y="0" width="1440" height="900" extended-state="6" />
@@ -454,28 +444,22 @@
     </entry>
     <entry file="jar:///Library/Java/JavaVirtualMachines/jdk1.8.0_191.jdk/Contents/Home/src.zip!/java/lang/Comparable.java">
       <provider selected="true" editor-type-id="text-editor">
-        <state relative-caret-position="-216">
+        <state relative-caret-position="1080">
           <caret line="95" column="17" selection-start-line="95" selection-start-column="17" selection-end-line="95" selection-end-column="17" />
         </state>
       </provider>
     </entry>
     <entry file="file://$PROJECT_DIR$/src/main/java/com/zipcodewilmington/singlylinkedlist/SinglyLinkedList.java">
       <provider selected="true" editor-type-id="text-editor">
-        <state relative-caret-position="2235">
-          <caret line="149" selection-start-line="149" selection-end-line="149" />
-          <folding>
-            <element signature="e#3864#3865#0" expanded="true" />
-            <element signature="e#3896#3897#0" expanded="true" />
-            <element signature="e#4115#4116#0" expanded="true" />
-            <element signature="e#4155#4156#0" expanded="true" />
-          </folding>
+        <state relative-caret-position="245">
+          <caret line="124" column="29" lean-forward="true" selection-start-line="124" selection-start-column="29" selection-end-line="124" selection-end-column="35" />
         </state>
       </provider>
     </entry>
     <entry file="file://$PROJECT_DIR$/src/test/java/com/zipcodewilmington/singlylinkedlist/SinglyLinkedListTest.java">
       <provider selected="true" editor-type-id="text-editor">
-        <state relative-caret-position="299">
-          <caret line="21" column="22" selection-start-line="21" selection-start-column="22" selection-end-line="21" selection-end-column="22" />
+        <state relative-caret-position="269">
+          <caret line="193" column="9" lean-forward="true" selection-start-line="193" selection-start-column="9" selection-end-line="193" selection-end-column="9" />
           <folding>
             <element signature="imports" expanded="true" />
           </folding>

src/main/java/com/zipcodewilmington/singlylinkedlist/SinglyLinkedList.java

@@ -117,10 +117,19 @@ public class SinglyLinkedList<T extends Comparable<T>> {
         return false;
     }
 
-    public SinglyLinkedList<T> copy(SinglyLinkedList<T> original) {
-        SinglyLinkedList<T> copyOfOriginal = new SinglyLinkedList<T>();
-        copyOfOriginal = original;
-        return copyOfOriginal;
+    public SinglyLinkedList<T> copy() {
+        SinglyLinkedList<T> copy = new SinglyLinkedList<T>();
+
+        Node node = head;
+
+        for (int i = 1; i <= size() && node.getNext() != null; i++) {
+            copy.add(node.getData());
+            node = node.getNext();
+        }
+
+        copy.add(tail.getData());
+
+        return copy;
     }
 
     public void sort() {

src/test/java/com/zipcodewilmington/singlylinkedlist/SinglyLinkedListTest.java

@@ -1,7 +1,6 @@
 package com.zipcodewilmington.singlylinkedlist;
 
 import org.junit.Assert;
-import org.junit.Before;
 import org.junit.Test;
 
 /**
@@ -182,9 +181,22 @@ public class SinglyLinkedListTest {
         linkedList.add("c");
 
         SinglyLinkedList actual = linkedList;
-        SinglyLinkedList expected = linkedList.copy(linkedList);
+        SinglyLinkedList expected = linkedList.copy();
+
+        for (int i = 1; i <= expected.size(); i++) {
+            System.out.println(expected.get(i));
+        }
+
+        expected.remove();
+
+        for (int i = 1; i <= expected.size(); i++) {
+            System.out.println(expected.get(i));
+        }
+
+        for (int i = 1; i < linkedList.size()+1; i++) {
+            System.out.println(linkedList.get(i));
+        }
 
-        Assert.assertTrue(actual.equals(expected));
     }
 
     @Test