Skip to content

Commit afb0ee3

Browse files
devversionjelbourn
authored andcommitted
chore: remove screenshot testing (#12155)
Removes the screenshot test tool since Angular Material is visually being tested inside of Google.
1 parent dc0732b commit afb0ee3

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

70 files changed

+3
-12924
lines changed

.gitignore

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44
/dist
55
/tmp
66
/deploy
7-
/screenshots
87
/bazel-out
98

109
# Example module file will be auto-generated.

.travis.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,8 +40,6 @@ jobs:
4040
if: type = push
4141
- env: "DEPLOY_MODE=docs-content"
4242
if: type = push
43-
- env: "DEPLOY_MODE=screenshot-tool"
44-
if: type = cron
4543
- env: "DEPLOY_MODE=dashboard"
4644
if: type = cron
4745
# Closure Compiler CI check is temporarily disabled until a new version of

DEV_ENVIRONMENT.md

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,3 @@ To run lint, run `gulp lint`.
2424

2525
### Running benchmarks
2626
Not yet implemented.
27-
28-
### Running screenshot diff tests
29-
Not yet implemented.

e2e/components/block-scroll-strategy-e2e.spec.ts

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
import {browser, Key, element, by} from 'protractor';
2-
import {screenshot} from '../screenshot';
32
import {getScrollPosition} from '../util/index';
43

54

@@ -18,8 +17,6 @@ describe('scroll blocking', () => {
1817
clickOn('disable');
1918
scrollPage(0, 300);
2019
expect((await getScrollPosition()).y).toBe(300, 'Exected page to be scrollable again.');
21-
22-
screenshot();
2320
});
2421

2522
it('should not be able to scroll programmatically along the y axis', async () => {
@@ -33,8 +30,6 @@ describe('scroll blocking', () => {
3330
clickOn('disable');
3431
scrollPage(300, 0);
3532
expect((await getScrollPosition()).x).toBe(300, 'Exected page to be scrollable again.');
36-
37-
screenshot();
3833
});
3934

4035
it('should not be able to scroll via the keyboard along the y axis', async () => {
@@ -55,8 +50,6 @@ describe('scroll blocking', () => {
5550
await body.sendKeys(Key.ARROW_DOWN);
5651
expect((await getScrollPosition()).y)
5752
.toBeGreaterThan(100, 'Expected the page to be scrollable again.');
58-
59-
screenshot();
6053
});
6154

6255
it('should not be able to scroll via the keyboard along the x axis', async () => {
@@ -77,8 +70,6 @@ describe('scroll blocking', () => {
7770
await body.sendKeys(Key.ARROW_RIGHT);
7871
expect((await getScrollPosition()).x)
7972
.toBeGreaterThan(100, 'Expected the page to be scrollable again.');
80-
81-
screenshot();
8273
});
8374

8475
it('should not be able to scroll the page after reaching the end of an element along the y axis',
@@ -94,8 +85,6 @@ describe('scroll blocking', () => {
9485
scroller.sendKeys(Key.ARROW_DOWN);
9586
scroller.sendKeys(Key.ARROW_DOWN);
9687
expect((await getScrollPosition()).y).toBe(100, 'Expected the page not to have scrolled.');
97-
98-
screenshot();
9988
});
10089

10190
it('should not be able to scroll the page after reaching the end of an element along the x axis',
@@ -111,8 +100,6 @@ describe('scroll blocking', () => {
111100
scroller.sendKeys(Key.ARROW_RIGHT);
112101
scroller.sendKeys(Key.ARROW_RIGHT);
113102
expect((await getScrollPosition()).x).toBe(100, 'Expected the page not to have scrolled.');
114-
115-
screenshot();
116103
});
117104
});
118105

e2e/components/button-e2e.spec.ts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,4 @@
11
import {browser, by, element, ExpectedConditions} from 'protractor';
2-
import {screenshot} from '../screenshot';
3-
42

53
describe('button', () => {
64
describe('disabling behavior', () => {
@@ -12,15 +10,13 @@ describe('button', () => {
1210

1311
await browser.wait(ExpectedConditions.not(
1412
ExpectedConditions.presenceOf(element(by.css('div.mat-ripple-element')))));
15-
screenshot('clicked once');
1613

1714
element(by.id('disable-toggle')).click();
1815
element(by.id('test-button')).click();
1916
expect(await element(by.id('click-counter')).getText()).toEqual('1');
2017

2118
await browser.wait(ExpectedConditions.not(
2219
ExpectedConditions.presenceOf(element(by.css('div.mat-ripple-element')))));
23-
screenshot('click disabled');
2420
});
2521
});
2622
});
Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,11 @@
11
import {browser, by, element} from 'protractor';
2-
import {screenshot} from '../screenshot';
32

43
describe('button-toggle', () => {
54

65
beforeEach(() => browser.get('/button-toggle'));
76

87
it('should show a button-toggle', async () => {
98
expect(element(by.tagName('mat-button-toggle'))).toBeDefined();
10-
screenshot();
119
});
1210

1311
});

e2e/components/card-e2e.spec.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
import {browser, by, element} from 'protractor';
2-
import {screenshot} from '../screenshot';
32

43
describe('mat-card', () => {
54

@@ -8,8 +7,6 @@ describe('mat-card', () => {
87
it('should show a card', async () => {
98
const card = element(by.tagName('mat-card'));
109
expect(card).toBeDefined();
11-
12-
screenshot('fancy card example');
1310
});
1411

1512
});

e2e/components/checkbox-e2e.spec.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,4 @@
11
import {browser, by, element, Key, ExpectedConditions} from 'protractor';
2-
import {screenshot} from '../screenshot';
3-
42

53
describe('checkbox', () => {
64

@@ -11,15 +9,13 @@ describe('checkbox', () => {
119
let checkboxEl = element(by.id('test-checkbox'));
1210
let inputEl = element(by.css('input[id=test-checkbox-input]'));
1311

14-
screenshot('start');
1512
checkboxEl.click();
1613

1714
expect(inputEl.getAttribute('checked'))
1815
.toBeTruthy('Expect checkbox "checked" property to be true');
1916

2017
await browser.wait(ExpectedConditions.not(
2118
ExpectedConditions.presenceOf(element(by.css('div.mat-ripple-element')))));
22-
screenshot('checked');
2319

2420
checkboxEl.click();
2521

@@ -28,7 +24,6 @@ describe('checkbox', () => {
2824

2925
await browser.wait(ExpectedConditions.not(
3026
ExpectedConditions.presenceOf(element(by.css('div.mat-ripple-element')))));
31-
screenshot('unchecked');
3227
});
3328

3429
it('should toggle the checkbox when pressing space', () => {

e2e/components/dialog-e2e.spec.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,16 +6,13 @@ import {
66
clickElementAtPoint,
77
waitForElement,
88
} from '../util/index';
9-
import {screenshot} from '../screenshot';
10-
119

1210
describe('dialog', () => {
1311
beforeEach(() => browser.get('/dialog'));
1412

1513
it('should open a dialog', () => {
1614
element(by.id('default')).click();
1715
expectToExist('mat-dialog-container');
18-
screenshot('simple dialog opened');
1916
});
2017

2118
it('should open a template dialog', () => {

e2e/components/expansion-e2e.spec.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,15 @@
11
import {browser, by, element} from 'protractor';
2-
import {screenshot} from '../screenshot';
32

43
describe('expansion', () => {
54

65
beforeEach(() => browser.get('/expansion'));
76

87
it('should show an accordion', async () => {
98
expect(element(by.css('.mat-accordion'))).toBeDefined();
10-
screenshot();
119
});
1210

1311
it('should show two panels', async () => {
1412
expect(await element.all(by.css('.mat-expansion-panel')).count()).toBe(2);
15-
screenshot();
1613
});
1714

1815
it('should hide contents of the expansion panel on click', async () => {

0 commit comments

Comments
 (0)