Skip to content

Commit

Permalink
test(multiple): remove unnecessary e2e tests
Browse files Browse the repository at this point in the history
As discussed, these changes remove the e2e tests that were testing the same functionality as the unit tests. These tests didn't help us to catch any additional issues and were increasing our maintenance cost. I've left in a few tests that make sense to run as e2e (e.g. virtual scrolling and scroll blocking). I've also removed some shared utilities that either weren't used or had one call site left.
  • Loading branch information
crisbeto authored and angular-robot[bot] committed Apr 28, 2023
1 parent 1f51ecc commit 65446f2
Show file tree
Hide file tree
Showing 122 changed files with 42 additions and 2,820 deletions.
4 changes: 0 additions & 4 deletions src/cdk-experimental/scrolling/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -39,15 +39,11 @@ ng_web_test_suite(
ng_e2e_test_library(
name = "e2e_test_sources",
srcs = glob(["**/*.e2e.spec.ts"]),
deps = [
"//src/cdk/testing/private/e2e",
],
)

e2e_test_suite(
name = "e2e_tests",
deps = [
":e2e_test_sources",
"//src/cdk/testing/private/e2e",
],
)
4 changes: 0 additions & 4 deletions src/cdk/overlay/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -75,16 +75,12 @@ ng_web_test_suite(
ng_e2e_test_library(
name = "e2e_test_sources",
srcs = glob(["**/*.e2e.spec.ts"]),
deps = [
"//src/cdk/testing/private/e2e",
],
)

e2e_test_suite(
name = "e2e_tests",
deps = [
":e2e_test_sources",
"//src/cdk/testing/private/e2e",
],
)

Expand Down
18 changes: 17 additions & 1 deletion src/cdk/overlay/scroll/block-scroll-strategy.e2e.spec.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import {browser, Key, element, by} from 'protractor';
import {getScrollPosition} from '../../testing/private/e2e';

describe('scroll blocking', () => {
beforeEach(() => browser.get('/block-scroll-strategy'));
Expand Down Expand Up @@ -116,3 +115,20 @@ async function clickOn(id: string) {
async function scrollPage(x: number, y: number) {
await browser.executeScript(`window.scrollTo(${x}, ${y});`);
}

/**
* Determines the current scroll position of the page.
*/
async function getScrollPosition(): Promise<{x: number; y: number}> {
const snippet = `
var documentRect = document.documentElement.getBoundingClientRect();
var x = -documentRect.left || document.body.scrollLeft || window.scrollX ||
document.documentElement.scrollLeft || 0;
var y = -documentRect.top || document.body.scrollTop || window.scrollY ||
document.documentElement.scrollTop || 0;
return {x: x, y: y};
`;

return await browser.executeScript(snippet);
}
10 changes: 0 additions & 10 deletions src/cdk/testing/private/e2e/BUILD.bazel

This file was deleted.

27 changes: 0 additions & 27 deletions src/cdk/testing/private/e2e/actions.ts

This file was deleted.

52 changes: 0 additions & 52 deletions src/cdk/testing/private/e2e/asserts.ts

This file was deleted.

11 changes: 0 additions & 11 deletions src/cdk/testing/private/e2e/index.ts

This file was deleted.

48 changes: 0 additions & 48 deletions src/cdk/testing/private/e2e/query.ts

This file was deleted.

20 changes: 0 additions & 20 deletions src/e2e-app/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ ng_module(
testonly = True,
srcs = glob(
["**/*.ts"],
exclude = ["test-util/**"],
),
assets = glob(
[
Expand All @@ -35,31 +34,12 @@ ng_module(
),
deps = [
"//src/cdk-experimental/scrolling",
"//src/cdk/dialog",
"//src/cdk/drag-drop",
"//src/cdk/overlay",
"//src/cdk/scrolling",
"//src/cdk/testing/tests:test_components",
"//src/components-examples/private",
"//src/material/button",
"//src/material/card",
"//src/material/checkbox",
"//src/material/chips",
"//src/material/core",
"//src/material/dialog",
"//src/material/grid-list",
"//src/material/icon",
"//src/material/input",
"//src/material/menu",
"//src/material/progress-bar",
"//src/material/progress-spinner",
"//src/material/radio",
"//src/material/select",
"//src/material/sidenav",
"//src/material/slide-toggle",
"//src/material/slider",
"//src/material/table",
"//src/material/tabs",
"@npm//@angular/animations",
"@npm//@angular/core",
"@npm//@angular/forms",
Expand Down
17 changes: 0 additions & 17 deletions src/e2e-app/button-toggle/button-toggle-e2e-module.ts

This file was deleted.

17 changes: 0 additions & 17 deletions src/e2e-app/button-toggle/button-toggle-e2e.ts

This file was deleted.

18 changes: 0 additions & 18 deletions src/e2e-app/button/button-e2e-module.ts

This file was deleted.

37 changes: 0 additions & 37 deletions src/e2e-app/button/button-e2e.html

This file was deleted.

18 changes: 0 additions & 18 deletions src/e2e-app/button/button-e2e.ts

This file was deleted.

17 changes: 0 additions & 17 deletions src/e2e-app/card/card-e2e-module.ts

This file was deleted.

Loading

0 comments on commit 65446f2

Please sign in to comment.