Polyterative / Patcher

The everything modular manager and database
https://patcher.xyz
GNU Affero General Public License v3.0
23 stars 2 forks source link

Add .spec test for home component #82

Closed sweep-ai[bot] closed 1 year ago

sweep-ai[bot] commented 1 year ago

PR Feedback (click)

Summary of Changes

Please review and merge this PR to ensure the correctness of the home component.

Fixes #74.


πŸŽ‰ Latest improvements to Sweep:


πŸ’‘ To get Sweep to edit this pull request, you can:

sweep-ai[bot] commented 1 year ago

GitHub actions yielded the following error.

The command that failed is yarn test-headless. Here are the relevant lines from the logs:

Chrome Headless 117.0.5938.92 (Windows 10) HomeComponent should create the HomeComponent FAILED
NullInjectorError: R3InjectorError(DynamicTestModule)[SupabaseService -> SupabaseService]:
NullInjectorError: No provider for SupabaseService!
Chrome Headless 117.0.5938.92 (Windows 10) HomeComponent should display the correct title FAILED
NullInjectorError: R3InjectorError(DynamicTestModule)[SupabaseService -> SupabaseService]:
NullInjectorError: No provider for SupabaseService!

Please note that the error is related to the SupabaseService not being provided.

This is likely a linting or type-checking issue with the source code. Update the code changed by the PR. Don't modify the existing tests.

I'm getting the same errors 3 times in a row, so I will stop working on fixing this PR.

Polyterative commented 1 year ago

add provide for SupabaseService in the test file

sweep-ai[bot] commented 1 year ago

πŸš€ Wrote Changes

add provide for SupabaseService in the test file

Hi @Polyterative,

I decided to make the following changes:

File Path Proposed Changes
src/app/features/backbone/home/home.component.spec.ts Modify src/app/features/backbone/home/home.component.spec.ts with contents:
β€’ Import SupabaseService from '../../backend/supabase.service' in the test file.
β€’ Add providers: [SupabaseService] to the TestBed.configureTestingModule function in the beforeEach block.
Polyterative commented 1 year ago

add provide for SupabaseService in the test file

sweep-ai[bot] commented 1 year ago

πŸš€ Wrote Changes

add provide for SupabaseService in the test file

Hi @Polyterative,

I decided to make the following changes:

File Path Proposed Changes
src/app/features/backbone/home/home.component.spec.ts Modify src/app/features/backbone/home/home.component.spec.ts with contents:
β€’ Import SupabaseService from '../../backend/supabase.service' in the test file.
β€’ Add SupabaseService to the TestBed.configureTestingModule providers array.