From a9a9fe865bbb8e39f40221234b60fc3c7124a331 Mon Sep 17 00:00:00 2001
From: Zhaolin Deng <zhaolin.deng@student.unimelb.edu.au>
Date: Mon, 17 Sep 2018 01:20:41 +1000
Subject: [PATCH] Replace BoundaryTests.java

---
 test/swen90006/machine/BoundaryTests.java | 78 ++++++++++-------------
 1 file changed, 34 insertions(+), 44 deletions(-)

diff --git a/test/swen90006/machine/BoundaryTests.java b/test/swen90006/machine/BoundaryTests.java
index 53de679..5f156a7 100644
--- a/test/swen90006/machine/BoundaryTests.java
+++ b/test/swen90006/machine/BoundaryTests.java
@@ -25,19 +25,9 @@ public class BoundaryTests
   }
 
   //Any method annotation with "@Test" is executed as a test.
-
-  @Test public void ec2Test()
-  {
-    //the assertEquals method used to check whether two values are
-    //equal, using the equals method
-	List<String> list = new ArrayList<String>();
-    list.add("jmp 0");
-    list.add("ret R0");	
-    Machine m = new Machine();
-    assertEquals(m.execute(list), "No result");
-  }
   
-  @Test public void ec3Test()
+  @Test(expected = swen90006.machine.NoReturnValueException.class)
+  public void ec3Test()
   {
     //the assertEquals method used to check whether two values are
     //equal, using the equals method
@@ -45,7 +35,7 @@ public class BoundaryTests
     list.add("jmp -1");
     list.add("ret R0");	
     Machine m = new Machine();
-    assertEquals(m.execute(list),NoReturnValueException);
+    m.execute(list);
   }
 
   @Test public void ec4Test()
@@ -59,7 +49,8 @@ public class BoundaryTests
     assertEquals(m.execute(list), 0);
   }
   
-  @Test public void ec5Test()
+  @Test(expected = swen90006.machine.InvalidInstructionException.class)
+  public void ec5Test()
   {
     //the assertEquals method used to check whether two values are
     //equal, using the equals method
@@ -67,17 +58,18 @@ public class BoundaryTests
     list.add("jmp 65536");
     list.add("ret R0");	
     Machine m = new Machine();
-    assertEquals(m.execute(list),InvalidInstructionException);
+    m.execute(list);
   } 
   
-  @Test public void ec6Test()
+  @Test(expected = swen90006.machine.InvalidInstructionException.class)
+  public void ec6Test()
   {
     //the assertEquals method used to check whether two values are
     //equal, using the equals method
 	List<String> list = new ArrayList<String>();
     list.add("ret R32");	
     Machine m = new Machine();
-    assertEquals(m.execute(list), InvalidInstructionException);
+    m.execute(list);
   }
   
   @Test public void ec7Test()
@@ -101,7 +93,8 @@ public class BoundaryTests
     assertEquals(m.execute(list), 65535);
   }
   
-  @Test public void ec9Test()
+  @Test(expected = swen90006.machine.InvalidInstructionException.class)
+  public void ec9Test()
   {
     //the assertEquals method used to check whether two values are
     //equal, using the equals method
@@ -109,21 +102,12 @@ public class BoundaryTests
     list.add("mov R1 65536");
     list.add("ret R1");	
     Machine m = new Machine();
-    assertEquals(m.execute(list),InvalidInstructionException);
-  }
-  
-  @Test public void ec10Test()
-  {
-    //the assertEquals method used to check whether two values are
-    //equal, using the equals method
-	List<String> list = new ArrayList<String>();
-    list.add("jz R1 0");
-    list.add("ret R1");	
-    Machine m = new Machine();
-    assertEquals(m.execute(list),"No result");
+    m.execute(list);
   }
+ 
   
-  @Test public void ec11Test()
+  @Test(expected = swen90006.machine.NoReturnValueException.class)
+  public void ec11Test()
   {
     //the assertEquals method used to check whether two values are
     //equal, using the equals method
@@ -131,7 +115,7 @@ public class BoundaryTests
     list.add("jz R1 3");
     list.add("ret R1");	
     Machine m = new Machine();
-    assertEquals(m.execute(list),NoReturnValueException);
+    m.execute(list);
   }
   
   @Test public void ec12Test()
@@ -157,7 +141,8 @@ public class BoundaryTests
     assertEquals(m.execute(list), 0);
   }
   
-  @Test public void ec14Test()
+  @Test(expected = swen90006.machine.InvalidInstructionException.class)
+  public void ec14Test()
   {
     //the assertEquals method used to check whether two values are
     //equal, using the equals method
@@ -166,10 +151,11 @@ public class BoundaryTests
 	list.add("ldr R0 R1 65536");
     list.add("ret R0");	
     Machine m = new Machine();
-    assertEquals(m.execute(list), InvalidInstructionException);
+    m.execute(list);
   }
   
-  @Test public void ec15Test()
+  @Test(expected = swen90006.machine.InvalidInstructionException.class)
+  public void ec15Test()
   {
     //the assertEquals method used to check whether two values are
     //equal, using the equals method
@@ -178,10 +164,11 @@ public class BoundaryTests
 	list.add("ldr R0 R1 0");
     list.add("ret R0");	
     Machine m = new Machine();
-    assertEquals(m.execute(list), InvalidInstructionException);
+    (m.execute(list);
   }
   
-  @Test public void ec16Test()
+  @Test(expected = swen90006.machine.InvalidInstructionException.class)
+  public void ec16Test()
   {
     //the assertEquals method used to check whether two values are
     //equal, using the equals method
@@ -189,10 +176,11 @@ public class BoundaryTests
 	list.add("ldr R0 R32 0");
     list.add("ret R0");	
     Machine m = new Machine();
-    assertEquals(m.execute(list), InvalidInstructionException);
+    m.execute(list);
   }
   
-  @Test public void ec17Test()
+  @Test(expected = swen90006.machine.InvalidInstructionException.class)
+  public void ec17Test()
   {
     //the assertEquals method used to check whether two values are
     //equal, using the equals method
@@ -200,10 +188,11 @@ public class BoundaryTests
 	list.add("str R0 0 R32");
     list.add("ret R0");	
     Machine m = new Machine();
-    assertEquals(m.execute(list), InvalidInstructionException);
+    m.execute(list);
   }
   
-  @Test public void ec18Test()
+  @Test(expected = swen90006.machine.InvalidInstructionException.class)
+  public void ec18Test()
   {
     //the assertEquals method used to check whether two values are
     //equal, using the equals method
@@ -212,7 +201,7 @@ public class BoundaryTests
 	list.add("str R1 0 R0");
     list.add("ret R0");	
     Machine m = new Machine();
-    assertEquals(m.execute(list), InvalidInstructionException);
+    m.execute(list);
   }
   
   @Test public void ec20Test()
@@ -228,7 +217,8 @@ public class BoundaryTests
     assertEquals(m.execute(list), 65535);
   }
   
-  @Test public void ec21Test()
+  @Test(expected = swen90006.machine.InvalidInstructionException.class)
+  public void ec21Test()
   {
     //the assertEquals method used to check whether two values are
     //equal, using the equals method
@@ -236,7 +226,7 @@ public class BoundaryTests
 	list.add("add R0 R1 R32");
     list.add("ret R0");	
     Machine m = new Machine();
-    assertEquals(m.execute(list), InvalidInstructionException);
+    m.execute(list);
   }
   
   @Test public void ec22Test()
-- 
GitLab