2163
57%

Ran 21 Nov 2016 08:10PM UTC

Jobs 1

Files 48

Run time 21s

Badge

Embed ▾

pending completion

Build # 2163

Build Type

push

travis-ci

Committed by

coolo

Commit Message

Use Carp for more helpful caller context in testapi (#646)

Instead of the pretty useless information that a command died in testapi.pm at
a certain line the test should give information which line in the test code
caused a test failure.

E.g. instead of

```
22:06:58.3323 28950 >>> testapi::wait_serial: OZvtF-\d+-: fail
22:06:58.3323 28950 # Test died:
command 'foo' failed: died here at ../../testapi.pm line 627.
```

you now get

```
22:12:21.4122 588 >>> testapi::wait_serial: OZvtF-\d+-: fail
22:12:21.4125 588 # Test died: command 'foo' failed: died here at /var/lib/openqa/share/tests/opensuse/tests/boot/boot_to_desktop.pm line 21.

```

for an `assert_script_run` with the corresponding 'fail_message'.
Same or similar output can not be expected for other functions, too, e.g.
shutdown, select_console, etc.

Also in 'runtest' and 'run_post_fail' the unnecessary warn telling about the
`run_post_fail` call from within basetest.pm is removed as well as redundant
newlines. ONe line break after the message should be enough. We should not
need 2 additional blank lines.

Run Details

1303 of 4996 relevant lines covered (26.08%)

5.64 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	2163.1	21 Nov 2016 08:10PM UTC	48	26.08	Travis Job 2163.1

Source Files on build 2163

Coverage	∆	File	Lines	Relevant	Covered	Missed	Hits/Line

os-autoinst / os-autoinst / 2163
57%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 2163

os-autoinst / os-autoinst / 2163 57%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 2163

os-autoinst / os-autoinst / 2163
57%

README BADGES
x