Add comprehensive unit tests for FeedComponent in COG-GTM/angular2-hn #130
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR adds a new test file
src/app/feeds/feed/feed.component.spec.tswith 40+ unit tests covering the FeedComponent. The tests use Jasmine/Karma with Angular's TestBed and mock the HackerNewsAPIService and ActivatedRoute dependencies.Test coverage includes:
Review & Testing Checklist for Human
npm test) - Tests compiled successfully but could not be executed in the dev environment due to ChromeHeadless unavailability. This is the most critical verification needed.typeSubandpageSubsubscriptions but nongOnDestroyto unsubscribe. Consider if this is a memory leak that should be addressed.Recommended test plan:
npm testand verify all 40+ tests passNotes
selector: 'item'to match the actual component (which doesn't follow Angular naming conventions), requiring a tslint disable commentLink to Devin run: https://app.devin.ai/sessions/25eb34317eb54f8ebe15573d735f5a71
Requested by: mason.batchelor@cognition.ai (@mbatchelor81)