Skip to content
This repository was archived by the owner on Nov 23, 2024. It is now read-only.

Use Chrome instead of PhantomJS to run tests - #255

Merged
Turbo87 merged 1 commit into
ember-cli:masterfrom
Gaurav0:drop_phantomjs
Oct 15, 2018
Merged

Use Chrome instead of PhantomJS to run tests#255
Turbo87 merged 1 commit into
ember-cli:masterfrom
Gaurav0:drop_phantomjs

Conversation

@Gaurav0

@Gaurav0 Gaurav0 commented Oct 14, 2018

Copy link
Copy Markdown
Contributor

Breaking.

Turbo87
Turbo87 reviewed Oct 14, 2018
View reviewed changes
Comment thread .travis.yml
Turbo87
Turbo87 reviewed Oct 14, 2018
View reviewed changes
Comment thread .travis.yml Outdated
node_js:
- "6"

sudo: false

@Turbo87 Turbo87 Oct 14, 2018

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is already set on line 1

@Gaurav0
Gaurav0 force-pushed the drop_phantomjs branch 3 times, most recently from 8e00a2a to 9da551c Compare October 14, 2018 18:04
rwjblue
rwjblue reviewed Oct 15, 2018
View reviewed changes
Comment thread .travis.yml Outdated
cache:
yarn: true

before_script:

@rwjblue rwjblue Oct 15, 2018

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, this shouldn't be needed...

@Gaurav0 Gaurav0 Oct 15, 2018 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are correct. It should be using the --no-sandbox and --headless mode. It isn't.

My two theories are that 1) it isn't finding/using testem.js and 2) it doesn't think it is in ci mode. I'm leaning toward (1).

@rwjblue rwjblue Oct 15, 2018 •
edited
Loading

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Possibly hardcoded resolutions forcing an older version of testem that isn't aware of the new syntax (nested ci property under Chrome section)?

@Gaurav0 Gaurav0 Oct 15, 2018

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I already tried forcing the latest testem. Let me try using an older syntax in testem.js

@Gaurav0 Gaurav0 Oct 15, 2018

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rwjblue Yay that worked!

rwjblue
rwjblue reviewed Oct 15, 2018
View reviewed changes
Comment thread .travis.yml Outdated
@Gaurav0 Gaurav0 mentioned this pull request Oct 15, 2018
Merged
@Turbo87

Turbo87 commented Oct 15, 2018

Copy link
Copy Markdown
Member

@Gaurav0 can you squash the commits so that the history is a little easier to read?

@Gaurav0

Gaurav0 commented Oct 15, 2018

Copy link
Copy Markdown
Contributor Author

Sure thing.

@Gaurav0
Drop support for phantomjs
9e37906 
fix node tests

use trusty

try something

add logs

try something else

try another thing

try using node 8

try yet another thing

more logs

fix suid error

use xvfb

cleanup logs

revert path changes

try forcing latest version of testem

try removing comments in travis.yml

revert try forcing latest version of testem

try older syntax in testem.js

don't need sudo
@Turbo87 Turbo87 changed the title Drop support for phantomjs Use Chrome instead of PhantomJS to run tests Oct 15, 2018
@Turbo87
Turbo87 merged commit b015986 into ember-cli:master Oct 15, 2018
@Gaurav0
Gaurav0 deleted the drop_phantomjs branch October 15, 2018 18:23
bors Bot added a commit to danmcclain/ember-deferred-content that referenced this pull request Nov 9, 2018
@bors @greenkeeper
Merge #53
d2396b4 
53: Update ember-cli-eslint to the latest version 🚀 r=danmcclain a=greenkeeper[bot]


## The devDependency [ember-cli-eslint](https://github.com/ember-cli/ember-cli-eslint) was updated from `4.2.3` to `5.0.0`.
This version is **not covered** by your **current version range**.

If you don’t accept this pull request, your project will work just like it did before. However, you might be missing out on a bunch of new features, fixes and/or performance improvements from the dependency update.

---

<details>
<summary>Release Notes for v5.0.0</summary>

<h4><g-emoji class="g-emoji" alias="boom" fallback-src="https://assets-cdn.github.com/images/icons/emoji/unicode/1f4a5.png">💥</g-emoji> Breaking Change</h4>
<ul>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/pull/284" data-hovercard-type="pull_request" data-hovercard-url="/ember-cli/ember-cli-eslint/pull/284/hovercard">#284</a> Upgrade <code>broccoli-lint-eslint</code> to v5.0.0 (<a href="https://urls.greenkeeper.io/Gaurav0">@Gaurav0</a>)</li>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/pull/256" data-hovercard-type="pull_request" data-hovercard-url="/ember-cli/ember-cli-eslint/pull/256/hovercard">#256</a> Drop support for IE9 and IE10 (<a href="https://urls.greenkeeper.io/Gaurav0">@Gaurav0</a>)</li>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/pull/254" data-hovercard-type="pull_request" data-hovercard-url="/ember-cli/ember-cli-eslint/pull/254/hovercard">#254</a> Drop support for Node.js 4 (<a href="https://urls.greenkeeper.io/Gaurav0">@Gaurav0</a>)</li>
</ul>
<h4><g-emoji class="g-emoji" alias="house" fallback-src="https://assets-cdn.github.com/images/icons/emoji/unicode/1f3e0.png">🏠</g-emoji> Internal</h4>
<ul>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/pull/251" data-hovercard-type="pull_request" data-hovercard-url="/ember-cli/ember-cli-eslint/pull/251/hovercard">#251</a> Upgrade to ember cli 3.4 via ember-cli-update (<a href="https://urls.greenkeeper.io/Gaurav0">@Gaurav0</a>)</li>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/pull/255" data-hovercard-type="pull_request" data-hovercard-url="/ember-cli/ember-cli-eslint/pull/255/hovercard">#255</a> Use Chrome instead of PhantomJS to run tests (<a href="https://urls.greenkeeper.io/Gaurav0">@Gaurav0</a>)</li>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/pull/263" data-hovercard-type="pull_request" data-hovercard-url="/ember-cli/ember-cli-eslint/pull/263/hovercard">#263</a> Remove obsolete <code>es6-promise</code> dev dependency (<a href="https://urls.greenkeeper.io/Turbo87">@Turbo87</a>)</li>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/pull/262" data-hovercard-type="pull_request" data-hovercard-url="/ember-cli/ember-cli-eslint/pull/262/hovercard">#262</a> yarn: Add <code>integrity</code> hashes (<a href="https://urls.greenkeeper.io/Turbo87">@Turbo87</a>)</li>
</ul>
<h4>Committers: 2</h4>
<ul>
<li>Gaurav Munjal (<a href="https://urls.greenkeeper.io/Gaurav0">@Gaurav0</a>)</li>
<li>Tobias Bieniek (<a href="https://urls.greenkeeper.io/Turbo87">@Turbo87</a>)</li>
</ul>
</details>

<details>
<summary>Commits</summary>
<p>The new version differs by 68 commits.</p>
<ul>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/commit/3b7f29605a727c3f84d6f7652ded7ff97e83604c"><code>3b7f296</code></a> <code>v5.0.0</code></li>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/commit/0beb1151e51af3864b5beef3e69ded4932d7181f"><code>0beb115</code></a> <code>Update Changelog</code></li>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/commit/5c23ac0bb4782d227e1e59dadbbff66e732238c9"><code>5c23ac0</code></a> <code>Add <code>lerna-changelog</code> dev dependency</code></li>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/commit/5657eecc4ae9395029be54db0b7eb96d05c0ea67"><code>5657eec</code></a> <code>Merge pull request #284 from Gaurav0/eslint_5</code></li>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/commit/0d484e6d64248499d0007ae7881e9a690ece1025"><code>0d484e6</code></a> <code>Upgrade to broccoli-lint-eslint 5</code></li>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/commit/4f88dbc23f0d8fe20e113fb05c3aa08d3e361885"><code>4f88dbc</code></a> <code>Bump qunit-dom from 0.7.1 to 0.8.0 (#287)</code></li>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/commit/6da2ee24d74575df66a65c27351b1090b442ef18"><code>6da2ee2</code></a> <code>Merge pull request #286 from ember-cli/dependabot/npm_and_yarn/lodash-4.17.11</code></li>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/commit/50645c3ef0248f225f07863ac72f06becd8b0d0c"><code>50645c3</code></a> <code>[Security] Bump lodash from 4.17.4 to 4.17.11</code></li>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/commit/d48f347c7bf338081919d370005dc5b257a95cf9"><code>d48f347</code></a> <code>Merge pull request #285 from ember-cli/dependabot/npm_and_yarn/debug-2.6.9</code></li>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/commit/89101ac2596aa9ad0778152ee6eb7099f0b4c69e"><code>89101ac</code></a> <code>[Security] Bump debug from 2.6.3 to 2.6.9</code></li>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/commit/92d4394c3667b3e38f1a18e234d4de11b073e13b"><code>92d4394</code></a> <code>Merge pull request #251 from Gaurav0/upgrade_to_ember_cli_3_4</code></li>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/commit/7b01707044da0653db32a9900c46559d7cbf9659"><code>7b01707</code></a> <code>fix typo</code></li>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/commit/293abba13f91f8376e6c862b9fe6b78d8b7ddc16"><code>293abba</code></a> <code>remove chai-things</code></li>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/commit/1810da28a4a96928f3307e134b086d76ae13fed3"><code>1810da2</code></a> <code>address comments</code></li>
<li><a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/commit/3948b782e05e0b693fedfae70346d6ae7ec6c342"><code>3948b78</code></a> <code>Do not fork tap reporter / match regexes</code></li>
</ul>
<p>There are 68 commits in total.</p>
<p>See the <a href="https://urls.greenkeeper.io/ember-cli/ember-cli-eslint/compare/a0e0ac2f48ae338ee085f2fd0a5813cacdad3c6d...3b7f29605a727c3f84d6f7652ded7ff97e83604c">full diff</a></p>
</details>

<details>
  <summary>FAQ and help</summary>

  There is a collection of [frequently asked questions](https://greenkeeper.io/faq.html). If those don’t help, you can always [ask the humans behind Greenkeeper](https://github.com/greenkeeperio/greenkeeper/issues/new).
</details>

---


Your [Greenkeeper](https://greenkeeper.io) bot 🌴



Co-authored-by: greenkeeper[bot] <greenkeeper[bot]@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

2 more reviewers
@rwjblue rwjblue rwjblue approved these changes
@Turbo87 Turbo87 Turbo87 approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

internal

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Gaurav0 @Turbo87 @rwjblue