keathley / wallaby / 930
85%

DEFAULT BRANCH: master
Ran
Jobs 15
Files 33
Run time 31min
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
930

push

travis-ci

web-flow 
Fixes zombie phantomjs (#325)

* Fix zombie phantom process

This commit contains several high level changes to solve the zombie
phantom process issue.

1. New ProcessWorkspace helper which creates a temporary folder for that
is available for the life of the given processes. This folder is a great
place for writing the process wrapper script, creating local storage
folders, etc. This folder is automatically deleted when the attached
process exits. These folders weren't previously getting cleaned up.

2. Improved run_phantom.sh wrapper script. This script now handles
shutting down when STDIN is closed, shuts down when it receives a INT,
HUP, or TERM signal, and shuts down when the underlying program crashes.
Since phantom spawns several child processes, this script also waits
until all child processes have exited before it exits itself. This helps
prevent Wallaby's ProcessWorkspace from deleting the wrapper script and
other temporary files before a phantomjs instance has fully shut down.

3. New tests to ensure that the Phantom.Server process responds as we'd
expect in the various crashing situations. I also made the wrapper
script echo back the PID of the phantom process so we can kill the
underlying phantom process as well.

There is a slight race condition where we can get zombie processes if we
shutdown the Phantom.Server process before it's had a chance to fully
start and wait for STDIN to close.

* Wait in Server.init/1 until phantom is started

Although multiple servers aren't started at the same time, this makes
sure that phantom is completely started before returning from init and
prevents zombie processes that occur when shutting down before the
script is completely started.

744 of 844 relevant lines covered (88.15%)

170485.29 hits per line

Jobs
ID Job ID Ran Files Coverage
1 930.1 0
36.73
 Travis Job 930.1
2 930.2 (WALLABY_DRIVER=phantom) 0
57.94
 Travis Job 930.2
3 930.3 (WALLABY_DRIVER=chrome) 0
42.89
 Travis Job 930.3
4 930.4 (WALLABY_DRIVER=selenium WALLABY_SELENIUM_VERSION=3) 0
36.97
 Travis Job 930.4
5 930.5 (WALLABY_DRIVER=selenium WALLABY_SELENIUM_VERSION=2) 0
37.09
 Travis Job 930.5
6 930.6 0
36.73
 Travis Job 930.6
7 930.7 (WALLABY_DRIVER=phantom) 0
57.94
 Travis Job 930.7
8 930.8 (WALLABY_DRIVER=chrome) 0
42.89
 Travis Job 930.8
9 930.9 (WALLABY_DRIVER=selenium WALLABY_SELENIUM_VERSION=3) 0
36.97
 Travis Job 930.9
10 930.10 (WALLABY_DRIVER=selenium WALLABY_SELENIUM_VERSION=2) 0
37.09
 Travis Job 930.10
11 930.11 0
43.13
 Travis Job 930.11
12 930.12 (WALLABY_DRIVER=phantom) 0
61.73
 Travis Job 930.12
13 930.13 (WALLABY_DRIVER=chrome) 0
46.33
 Travis Job 930.13
14 930.14 (WALLABY_DRIVER=selenium WALLABY_SELENIUM_VERSION=3) 0
40.52
 Travis Job 930.14
15 930.15 (WALLABY_DRIVER=selenium WALLABY_SELENIUM_VERSION=2) 0
40.4
 Travis Job 930.15
Source Files on build 930
Detailed source file information is not available for this build.