About a week ago, Chrome 41 was made the new stable and our tests on Travis-CI started failing.

This is the Chrome version that passes: 40.0.2214.115-1

And this one fails: 41.0.2272.76-1

While it's possible that this isn't the actual cause of the failures (and the update was just a cooincidence), this is my best guess so far.

CommentFileSizeAuthor
#6 panopoly_test-chrome41-2447475-6.patch1002 bytesdsnopek
#5 panopoly_test-chrome41-2447475-5.patch1.57 KBdsnopek
#4 panopoly_test-chrome41-2447475-4.patch1.58 KBdsnopek
#3 panopoly_test-chrome41-2447475-3.patch1.58 KBdsnopek
#2 panopoly_test-chrome41-2447475-2.patch1.35 KBdsnopek

Comments

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented

I was able to confirm that it was the Chrome update that did it by downgrading the Chrome version used on Travis - the tests then succeeded!

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
Status: Active » Needs review
StatusFileSize
new panopoly_test-chrome41-2447475-2.patch1.35 KB

Attached is a patch which uses a video with a shorter title. I have this suspicion that the title is too long, pushing the preview image on top of the button we're trying to click.

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
StatusFileSize
new panopoly_test-chrome41-2447475-3.patch1.58 KB

Blergh. Removing check for video title which will be incorrect with new video.

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
StatusFileSize
new panopoly_test-chrome41-2447475-4.patch1.58 KB

Heh, so the shorter title worked. :-) Here's a new patch using a Panopoly related video.

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
StatusFileSize
new panopoly_test-chrome41-2447475-5.patch1.57 KB

Well, that didn't work! Trying another one of my ideas - focusing on the link first.

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
StatusFileSize
new panopoly_test-chrome41-2447475-6.patch1002 bytes

Ok, well, switching back to the Minecraft video. If it works again, I'll commit this patch.

  • dsnopek committed a824128 on 7.x-1.x 
    Update Panopoly Test for Issue #2447475 by dsnopek: YouTube test fails...
dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
Status: Needs review » Fixed

Committed!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.