There’s only one way to succeed, but many ways to fail...

From: Andy Caldwell [mailto:notifications@github.com]
Sent: 08 May 2013 14:40
To: Metaswitch/clearwater-live-test
Cc: Keith Wansbrough
Subject: Re: [clearwater-live-test] [Reviewer: Andy] Make return code reflect success/failure. (#3)


In lib/live-test.rb:

> @@ -38,6 +38,8 @@ def run_tests(domain, glob="*")

>    TestDefinition.run_all($domain, Wildcard[glob, true])

>

>    destroy_leaked_numbers

> +

> +  exit (TestDefinition.failures == 0) ? 0 : 1

Damn Bash and its backwards TRUE/FALSE...

—
Reply to this email directly or view it on GitHub<https://github.com/Metaswitch/clearwater-live-test/pull/3/files#r4133836>.