TeamMentor / Master

TEAM Mentor 3.x Released Code
16 stars 17 forks source link

As a user I should be able to resize the TM to any size browser, such that it is still usable (JADE) #898

Open romichg opened 10 years ago

romichg commented 10 years ago
michaelhidalgo commented 9 years ago

I've done manual testing at Sauce Labs (which has been recorded), I guess it would be better to include the SauceLabs url of the test scenario ?

-> Manual testing https://saucelabs.com/tests/1932cb594ff044ec8467c90a49e7303e

I used a Windows 7 machine and Internet Explorer 10 . I manually resized the window to see how it looks like :

image1

image2

image3

image4

romichg commented 9 years ago

Can we do this as part of an automated test though and take screenshots? that way we can look at multiple browsers fairly quickly.

michaelhidalgo commented 9 years ago

Yeah absolutely, Is there any specific platform (operative system, device) you believe it is important to have automated?

By platforms I mean https://saucelabs.com/platforms

2014-12-12 14:44 GMT-06:00 RomichG notifications@github.com:

Can we do this as part of an automated test though and take screenshots? that way we can look at multiple browsers fairly quickly.

— Reply to this email directly or view it on GitHub https://github.com/TeamMentor/Master/issues/898#issuecomment-66832381.

Michael Hidalgo http://michaelhidalgocr.blogspot.com

The future has many names: For the weak, it means the unattainable. For the fearful, it means the unknown. For the courageous, it means opportunity. (1802-1885) French Poet, Dramatist, Writer

romichg commented 9 years ago

Well if we have one automated test we can run it on multiple right? I think we should cover win7/ie11/ie8/ff/chrome OSX and latest Safari

michaelhidalgo commented 9 years ago

Yes, it should be the same test, but we need to define the platforms.

2014-12-12 15:22 GMT-06:00 RomichG notifications@github.com:

Well if we have one automated test we can run it on multiple right? I think we should cover win7/ie11/ie8/ff/chrome OSX and latest Safari

— Reply to this email directly or view it on GitHub https://github.com/TeamMentor/Master/issues/898#issuecomment-66837361.

Michael Hidalgo http://michaelhidalgocr.blogspot.com

The future has many names: For the weak, it means the unattainable. For the fearful, it means the unknown. For the courageous, it means opportunity. (1802-1885) French Poet, Dramatist, Writer

romichg commented 9 years ago

So yes lets go for Windows Vista IE8 Windows 7 - IE11/IE8 FF (Latest) Chrome (Latest) OSX Safari (Latest)

Lets see how we do with those for now and than we can expand.

michaelhidalgo commented 9 years ago

I wrote several test using different resolutions :

a) 500X500 b)600X400 c)300X300 d)200X200

However, there is a known issue in Selenium that does not take the full browser screenshot, rather than that, It takes part of the screen being processed at the time of taking the screenshot.

A couple of threads that explains this behavior : 1.https://code.google.com/p/chromedriver/issues/detail?id=294 2.https://support.saucelabs.com/entries/42638820-My-screenshots-aren-t-full-page-or-have-black-bars-in-IE-Chrome-Safari

SauceLabs takes screenshots of all the test and it saves it in the cloud for 30 days, they have a REST API to fetch the images (which are full screen).

chrome_31_windows 7_resolution 500x500_loginpage_browserresize500x500 firefox_25_windows 7_resolution 500x500_loginpage_browserresize500x500 firefox_25_windows 8 1_resolution 500x500_loginpage_browserresize500x500

DinisCruz commented 9 years ago

Humm, isn't there a way to scroll the page and stitch the screenshots?

romichg commented 9 years ago

It looks like people had issues with scrolling and stiching, plus having different tests for different browsers is going to be messy. I am wondering if we can force the actual SauceLabs VM to take the screenshot.

michaelhidalgo commented 9 years ago

Actually Saucelabs folks do that for us :) even, they record a video. Actually we can download the Job (which will download the metadata, png images and video recorded).

https://docs.saucelabs.com/reference/rest-api/#get-job-asset-files

2014-12-15 14:36 GMT-06:00 RomichG notifications@github.com:

It looks like people had issues with scrolling and stiching, plus having different tests for different browsers is going to be messy. I am wondering if we can force the actual SauceLabs VM to take the screenshot.

— Reply to this email directly or view it on GitHub https://github.com/TeamMentor/Master/issues/898#issuecomment-67060950.

Michael Hidalgo http://michaelhidalgocr.blogspot.com

The future has many names: For the weak, it means the unattainable. For the fearful, it means the unknown. For the courageous, it means opportunity. (1802-1885) French Poet, Dramatist, Writer