• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

typhonius / acquia-php-sdk-v2 / 298
99%
master: 99%

Build:
Build:
LAST BUILD BRANCH: php73
DEFAULT BRANCH: master
Ran 08 Jan 2020 12:14PM UTC
Jobs 3
Files 164
Run time 2min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

pending completion
298

Pull #39

travis-ci

web-flow
Adds final test case and resends the Exception.

The reason I have chosen to throw an exception
instead of catching the exception and printing
a message is twofold.

Firstly, we cannot test the use of an exit()
or die() that would be required to catch the
error and present a message alerting the user
of the issue.

Secondly, by handling the error, it prevents
consumming libraries from easily detecting an
error has been thown and handling it gracefully.

Both of these reasons have lead me to determining
that Exceptions should be handled in code by
consuming libraries. I'll document this on the
README so users can handle the exceptions if they
wish. I'll also have to use this in Acquia Cli
and Acquia Logstream. I don't catch errors in
these at all right now so it would be an
enhancement!
Pull Request #39: Catches API errors to display more cleanly.

9 of 9 new or added lines in 3 files covered. (100.0%)

1606 of 1630 relevant lines covered (98.53%)

13.02 hits per line

Jobs
ID Job ID Ran Files Coverage
1 298.1 08 Jan 2020 12:14PM UTC 0
98.53
Travis Job 298.1
2 298.2 08 Jan 2020 12:14PM UTC 0
98.53
Travis Job 298.2
3 298.3 08 Jan 2020 12:16PM UTC 0
98.51
Travis Job 298.3
Source Files on build 298
Detailed source file information is not available for this build.
  • Back to Repo
  • Travis Build #298
  • Pull Request #39
  • PR Base - master (#286)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc