Skip to content

Commit 276f9cc

Browse files
committed
Ignore some TakesScreenshots tests, qt webkit take screenshot only from visible viewport
1 parent bf9431d commit 276f9cc

File tree

1 file changed

+6
-11
lines changed

1 file changed

+6
-11
lines changed

java/client/test/org/openqa/selenium/TakesScreenshotTest.java

Lines changed: 6 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -26,14 +26,9 @@
2626
import static org.junit.Assert.assertArrayEquals;
2727
import static org.junit.Assert.assertTrue;
2828
import static org.junit.Assert.fail;
29-
import static org.openqa.selenium.testing.Ignore.Driver.ALL;
30-
import static org.openqa.selenium.testing.Ignore.Driver.CHROME;
31-
import static org.openqa.selenium.testing.Ignore.Driver.FIREFOX;
32-
import static org.openqa.selenium.testing.Ignore.Driver.IE;
33-
import static org.openqa.selenium.testing.Ignore.Driver.IPHONE;
3429
import static org.openqa.selenium.OutputType.BASE64;
3530
import static org.openqa.selenium.OutputType.BYTES;
36-
import static org.openqa.selenium.testing.Ignore.Driver.SAFARI;
31+
import static org.openqa.selenium.testing.Ignore.Driver.*;
3732

3833
import java.awt.*;
3934
import java.awt.image.BufferedImage;
@@ -140,7 +135,7 @@ public void testShouldCaptureScreenshot() throws Exception {
140135
}
141136

142137
@Test
143-
@Ignore(value = {SAFARI, CHROME},
138+
@Ignore(value = {SAFARI, CHROME, QTWEBKIT},
144139
reason = " SAFARI: take only visible viewport."
145140
+ " CHROME: (v1) ok, (v2) take only visible viewport."
146141
)
@@ -172,7 +167,7 @@ public void testShouldCaptureScreenshotWithLongX() throws Exception {
172167
}
173168

174169
@Test
175-
@Ignore(value = {SAFARI, CHROME},
170+
@Ignore(value = {SAFARI, CHROME, QTWEBKIT},
176171
reason = " SAFARI: take only visible viewport."
177172
+ " CHROME: (v1) ok, (v2) take only visible viewport."
178173
)
@@ -204,7 +199,7 @@ public void testShouldCaptureScreenshotWithLongY() throws Exception {
204199
}
205200

206201
@Test
207-
@Ignore(value = {IE, FIREFOX, SAFARI},
202+
@Ignore(value = {IE, FIREFOX, SAFARI, QTWEBKIT},
208203
reason = "IE9: captured image is cat at driver level. it's not yet supported."
209204
+ " FF: unable to grab screenshot with too long size (NS_ERROR_FAILURE)."
210205
+ " SAFARI: take only visible viewport."
@@ -238,7 +233,7 @@ public void testShouldCaptureScreenshotWithTooLongX() throws Exception {
238233
}
239234

240235
@Test
241-
@Ignore(value = {IE, FIREFOX, SAFARI, CHROME},
236+
@Ignore(value = {IE, FIREFOX, SAFARI, CHROME, QTWEBKIT},
242237
reason = "IE: captured image is cat at driver level. it's not yet supported."
243238
+ " FF: unable to grab screenshot with too long size (NS_ERROR_FAILURE)."
244239
+ " SAFARI: take only visible viewport."
@@ -394,7 +389,7 @@ public void testShouldCaptureScreenshotAtIFramePageAfterSwitching() throws Excep
394389
}
395390

396391
@Test
397-
@Ignore(value = {IE, FIREFOX, SAFARI, CHROME},
392+
@Ignore(value = {IE, FIREFOX, SAFARI, CHROME, QTWEBKIT},
398393
reason = "IE9: unable to capture such image due Image initialization failure"
399394
+ " FF: unable to grab screenshot with too long size (NS_ERROR_FAILURE)."
400395
+ " SAFARI: take only visible viewport."

0 commit comments

Comments
 (0)