You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hc.apache.org by ol...@apache.org on 2005/03/13 21:00:53 UTC

svn commit: r157352 - jakarta/httpclient/trunk/http-common/src/test/org/apache/http/TestStatusLine.java

Author: olegk
Date: Sun Mar 13 12:00:52 2005
New Revision: 157352

URL: http://svn.apache.org/viewcvs?view=rev&rev=157352
Log:
More test coverage

Modified:
    jakarta/httpclient/trunk/http-common/src/test/org/apache/http/TestStatusLine.java

Modified: jakarta/httpclient/trunk/http-common/src/test/org/apache/http/TestStatusLine.java
URL: http://svn.apache.org/viewcvs/jakarta/httpclient/trunk/http-common/src/test/org/apache/http/TestStatusLine.java?view=diff&r1=157351&r2=157352
==============================================================================
--- jakarta/httpclient/trunk/http-common/src/test/org/apache/http/TestStatusLine.java (original)
+++ jakarta/httpclient/trunk/http-common/src/test/org/apache/http/TestStatusLine.java Sun Mar 13 12:00:52 2005
@@ -63,6 +63,24 @@
 
     // ----------------------------------------------------------- Test Methods
 
+    public void testConstructor() {
+        StatusLine statusline = new StatusLine(HttpVersion.HTTP_1_1, HttpStatus.SC_OK, "OK");
+        assertEquals(HttpVersion.HTTP_1_1, statusline.getHttpVersion()); 
+        assertEquals(HttpStatus.SC_OK, statusline.getStatusCode()); 
+        assertEquals("OK", statusline.getReasonPhrase()); 
+    }
+        
+    public void testConstructorInvalidInput() {
+        try {
+            StatusLine statusline = new StatusLine(null, HttpStatus.SC_OK, "OK");
+            fail("IllegalArgumentException should have been thrown");
+        } catch (IllegalArgumentException e) { /* expected */ }
+        try {
+            StatusLine statusline = new StatusLine(HttpVersion.HTTP_1_1, -1, "OK");
+            fail("IllegalArgumentException should have been thrown");
+        } catch (IllegalArgumentException e) { /* expected */ }
+    }
+        
     public void testIfStatusLine() throws Exception {
         assertTrue(StatusLine.startsWithHTTP("HTTP"));
         assertTrue(StatusLine.startsWithHTTP("         HTTP"));
@@ -137,6 +155,10 @@
             statusLine = StatusLine.parse("HTTP/1.1    ");
             fail();
         } catch (HttpException e) { /* expected */ }
+        try {
+            statusLine = StatusLine.parse("HTTP/1.1");
+            fail();
+        } catch (HttpException e) { /* expected */ }
     }
 
     public void testNullInput() throws Exception {
@@ -144,7 +166,12 @@
             statusLine = StatusLine.parse(null);
             fail("IllegalArgumentException should have been thrown");
         } catch (IllegalArgumentException e) { /* expected */ }
-
     }
     
+    public void testToString() throws Exception {
+        StatusLine statusline = new StatusLine(HttpVersion.HTTP_1_1, HttpStatus.SC_OK, "OK");
+        assertEquals("HTTP/1.1 200 OK", statusline.toString());
+        statusline = new StatusLine(HttpVersion.HTTP_1_1, HttpStatus.SC_OK, null);
+        assertEquals("HTTP/1.1 200", statusline.toString());
+    }
 }