Refactor #18232
Add phantomjs-prebuilt dev dependency for integration tests
Pull request:
Fixed in Releases:
Found in Releases:
Description
Integration tests using JavaScript depend on using phantomjs via poltergeist, but there is no explicit gem dependency to install phantomjs, instead requiring manual installation.
The phantomjs-prebuilt npm package should be added to development dependencies, so it can be installed automatically.
Associated revisions
refs #18232 - make phantomjs optional, not available on all archs
History
#1
Updated by The Foreman Bot over 5 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/4225 added
#2
Updated by Dominic Cleal over 5 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset 2e81c25e87e07926fd76fc271c9d2251b4ed2c11.
#3
Updated by Dominic Cleal over 5 years ago
- Legacy Backlogs Release (now unused) set to 209
#4
Updated by The Foreman Bot over 5 years ago
- Pull request https://github.com/theforeman/foreman/pull/4235 added
fixes #18232 - install phantomjs via npm