Pretty print objects passed to fail method

Fixes #1090

spec/core/integration/EnvSpec.js

@@ -181,6 +181,12 @@ describe("Env integration", function() {
             }
           })]
         }));
+        expect(specDone).toHaveBeenCalledWith(jasmine.objectContaining({
+          description: 'pretty prints objects',
+          failedExpectations: [jasmine.objectContaining({
+            message: 'Failed: Object({ prop: \'value\', arr: [ \'works\', true ] })'
+          })]
+        }));
         done();
       }
     });
@@ -197,6 +203,10 @@ describe("Env integration", function() {
       env.it('has a message and stack trace from an Error', function() {
         env.fail(new Error('error message'));
       });
+
+      env.it('pretty prints objects', function() {
+        env.fail({prop: 'value', arr: ['works', true]});
+      })
     });
 
     env.execute();

src/core/Env.js

@@ -516,9 +516,17 @@ getJasmineRequireObj().Env = function(j$) {
 
     this.fail = function(error) {
       var message = 'Failed';
+      var isError = error instanceof Error;
       if (error) {
         message += ': ';
-        message += error.message || error;
+        if (isError) {
+          message += error.message;
+        } else if (typeof error === 'object') {
+          // pretty print all kind of objects. This includes arrays.
+          message += jasmine.pp(error);
+        } else {
+          message += error;
+        }
       }
 
       currentRunnable().addExpectationResult(false, {
@@ -527,7 +535,7 @@ getJasmineRequireObj().Env = function(j$) {
         expected: '',
         actual: '',
         message: message,
-        error: error && error.message ? error : null
+        error: isError ? error : null
       });
     };
   }