Add status code detection to the response class

Author: elpete

.gitignore

@@ -2,4 +2,5 @@
 /tests/results
 /tests/resources/app/coldbox
 /node_modules
-/modules
+/modules
+jmimemagic.log

models/HyperResponse.cfc

@@ -20,6 +20,27 @@ component accessors="true" {
         return deserializeJSON( getData() );
     }
 
+    function isSuccess() {
+        return left( getStatusCode(), 1 ) == "2";
+    }
+
+    function isRedirect() {
+        return left( getStatusCode(), 1 ) == "3";
+    }
+
+    function isError() {
+        return left( getStatusCode(), 1 ) == "4" ||
+            left( getStatusCode(), 1 ) == "5";
+    }
+
+    function isClientError() {
+        return left( getStatusCode(), 1 ) == "4";
+    }
+
+    function isServerError() {
+        return left( getStatusCode(), 1 ) == "5";
+    }
+
     private function populateFromCFHTTP( res ) {
         variables.charset = res.charset;
         variables.statusCode = res.responseheader.status_code;
@@ -35,8 +56,4 @@ component accessors="true" {
         return variables.headers[ lcase( name ) ];
     }
 
-    function isRedirect() {
-        return left( getStatusCode(), 1 ) == "3";
-    }
-
 }

tests/specs/unit/HyperResponseSpec.cfc

@@ -14,6 +14,68 @@ component extends="testbox.system.BaseSpec" {
                     var json = res.json();
                 } ).toThrow( "DeserializeJsonException" );
             } );
+
+            describe( "status code detection", function() {
+                it( "can tell if a request is a success", function() {
+                    var res = new Hyper.models.HyperResponse( createStub(), {
+                        "charset" = "UTF-8",
+                        "responseheader" = {
+                            "status_code" = "200"
+                        },
+                        "filecontent" = ""
+                    } );
+                    expect( res.isSuccess() ).toBeTrue();
+                    expect( res.isRedirect() ).toBeFalse();
+                    expect( res.isError() ).toBeFalse();
+                    expect( res.isClientError() ).toBeFalse();
+                    expect( res.isServerError() ).toBeFalse();
+                } );
+
+                it( "can tell if a request is a redirect", function() {
+                    var res = new Hyper.models.HyperResponse( createStub(), {
+                        "charset" = "UTF-8",
+                        "responseheader" = {
+                            "status_code" = "302"
+                        },
+                        "filecontent" = ""
+                    } );
+                    expect( res.isSuccess() ).toBeFalse();
+                    expect( res.isRedirect() ).toBeTrue();
+                    expect( res.isError() ).toBeFalse();
+                    expect( res.isClientError() ).toBeFalse();
+                    expect( res.isServerError() ).toBeFalse();
+                } );
+
+                it( "can tell if a request is a client error", function() {
+                    var res = new Hyper.models.HyperResponse( createStub(), {
+                        "charset" = "UTF-8",
+                        "responseheader" = {
+                            "status_code" = "422"
+                        },
+                        "filecontent" = ""
+                    } );
+                    expect( res.isSuccess() ).toBeFalse();
+                    expect( res.isRedirect() ).toBeFalse();
+                    expect( res.isError() ).toBeTrue();
+                    expect( res.isClientError() ).toBeTrue();
+                    expect( res.isServerError() ).toBeFalse();
+                } );
+
+                it( "can tell if a request is a server error", function() {
+                    var res = new Hyper.models.HyperResponse( createStub(), {
+                        "charset" = "UTF-8",
+                        "responseheader" = {
+                            "status_code" = "500"
+                        },
+                        "filecontent" = ""
+                    } );
+                    expect( res.isSuccess() ).toBeFalse();
+                    expect( res.isRedirect() ).toBeFalse();
+                    expect( res.isError() ).toBeTrue();
+                    expect( res.isClientError() ).toBeFalse();
+                    expect( res.isServerError() ).toBeTrue();
+                } );
+            } );
         } );
     }