Skip to content

refactor: remove unused ChangeDetectionStrategy from LoginComponent and LoggedInComponent - #591

Merged
timdeschryver merged 2 commits into
testing-library:mainfrom
JeevanMahesha:fix/issue-491-karma
Jul 30, 2026
Merged

refactor: remove unused ChangeDetectionStrategy from LoginComponent and LoggedInComponent#591
timdeschryver merged 2 commits into
testing-library:mainfrom
JeevanMahesha:fix/issue-491-karma

Conversation

@JeevanMahesha

@JeevanMahesha JeevanMahesha commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@JeevanMahesha

Copy link
Copy Markdown
Contributor Author

@timdeschryver The 4 failing tests in projects/testing-library/src/tests/zoneless.spec.ts are not caused by this PR. They were pre-existing before this branch — running the tests on a clean checkout (no local changes) produces the same 4 failures.

@timdeschryver

Copy link
Copy Markdown
Member

@JeevanMahesha yea, I also noticed it. Let me fix those first before merging this PR.

@timdeschryver

Copy link
Copy Markdown
Member

@JeevanMahesha feel free to update this branch, those tests should succeed now. Sorry for the inconvenience.

@timdeschryver timdeschryver left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @JeevanMahesha !

@timdeschryver
timdeschryver merged commit cac48a2 into testing-library:main Jul 30, 2026
4 checks passed
@timdeschryver

Copy link
Copy Markdown
Member

@all-contributors please add @JeevanMahesha for docs

@allcontributors

Copy link
Copy Markdown
Contributor

@timdeschryver

I've put up a pull request to add @JeevanMahesha! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants