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 test case for application startup and homepage display #75

Closed sweep-ai[bot] closed 1 year ago

sweep-ai[bot] commented 1 year ago

PR Feedback (click)

Summary of Changes

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:

ERROR: 'NG0304: 'app-toolbar' is not a known element (used in the 'AppComponent' component template):
1. If 'app-toolbar' is an Angular component, then verify that it is a part of an @NgModule where this component is declared.
2. If 'app-toolbar' is a Web Component then add 'CUSTOM_ELEMENTS_SCHEMA' to the '@NgModule.schemas' of this component to suppress this message.'

ERROR: 'NG0304: 'router-outlet' is not a known element (used in the 'AppComponent' component template):
1. If 'router-outlet' is an Angular component, then verify that it is a part of an @NgModule where this component is declared.
2. If 'router-outlet' is a Web Component then add 'CUSTOM_ELEMENTS_SCHEMA' to the '@NgModule.schemas' of this component to suppress this message.'

ERROR: 'NG0304: 'app-footer' is not a known element (used in the 'AppComponent' component template):
1. If 'app-footer' is an Angular component, then verify that it is a part of an @NgModule where this component is declared.
2. If 'app-footer' is a Web Component then add 'CUSTOM_ELEMENTS_SCHEMA' to the '@NgModule.schemas' of this component to suppress this message.'

Chrome Headless 117.0.5938.92 (Windows 10) AppComponent should render the homepage component FAILED
Error: Expected null to be truthy.
at <Jasmine>
at UserContext.apply (src/app/app.component.spec.ts:21:52)
at _ZoneDelegate.invoke (node_modules/zone.js/fesm2015/zone.js:372:26)
at ProxyZoneSpec.onInvoke (node_modules/zone.js/fesm2015/zone-testing.js:287:39)

Please note that the error message is repeated multiple times for different elements (app-toolbar, router-outlet, app-footer).

There are a lot of errors. This is likely due to a small parsing issue or a missing import with the files changed in the PR.

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.