Skip to content
Merged
Changes from 1 commit
Commits
Show all changes
106 commits
Select commit Hold shift + click to select a range
c6f90a6
Initial removing all entities by directly import same data to every u…
CeEv Jan 17, 2025
9ca120b
fix import order
CeEv Jan 17, 2025
b2bae87
Add linter rule for deep imports into .modules and .app.modules.
CeEv Jan 17, 2025
738f432
Merge branch 'main' into BC-8551-remove-all-entities
CeEv Jan 20, 2025
a0241e5
Reduce available entities for feathers to minimal needed.
CeEv Jan 21, 2025
f13549e
Merge branch 'main' into BC-8551-remove-all-entities
CeEv Jan 21, 2025
d90b433
Change linter rule to match for file names with numbers.
CeEv Jan 21, 2025
2d10afe
Change linter rule from error to warn. Error was wrong commited.
CeEv Jan 22, 2025
558e01b
Merge branch 'main' into BC-8551-remove-all-entities
CeEv Jan 23, 2025
7b083be
Update mikro-orm-cli.config.ts
CeEv Jan 23, 2025
0e7f1a8
fix unresolved merge conflict
CeEv Jan 23, 2025
fcdb973
BC-8551 - reduce used entities of `board-collaboration.app.module.ts`
bergatco Jan 23, 2025
3c03f90
BC-8551 - remove `MikroOrmModule` from `common-cartridge.module.ts`
bergatco Jan 24, 2025
14ca6a9
BC-8551 - remove `MikroOrmModule` from `deletion-console.app.module.ts`
bergatco Jan 24, 2025
3b08726
BC-8551 - reduce used entities of `idp-console.app.module.ts`
bergatco Jan 24, 2025
42d6ab4
BC-8551 - reduce used entities of `management-server.app.module.ts`
bergatco Jan 24, 2025
57439fd
@bergatco
bergatco Jan 24, 2025
043a5d3
BC-8551 - reduce used entities of `server.app.module.ts`
bergatco Jan 24, 2025
050a1e5
BC-8551 - fix entities array in `server.app.module.ts`
bergatco Jan 24, 2025
dc391de
BC-8551 - readd and reduce used entities of `deletion-console.app.mod…
bergatco Jan 24, 2025
4dd09a3
@bergatco
bergatco Jan 24, 2025
1b3507c
Merge branch 'main' into BC-8551-remove-all-entities
bergatco Jan 24, 2025
399debb
Make more entities available for feathers tests
CeEv Jan 24, 2025
23ff8c7
Merge branch 'BC-8551-remove-all-entities' of https://github.com/hpi-…
CeEv Jan 24, 2025
2e3a005
Merge branch 'main' into BC-8551-remove-all-entities
CeEv Jan 24, 2025
b7bbc82
Add todos
CeEv Jan 24, 2025
6c51ce7
Merge branch 'BC-8551-remove-all-entities' of https://github.com/hpi-…
CeEv Jan 24, 2025
187d424
Try regex search for entities
CeEv Jan 24, 2025
73c91e0
Change entity detection for legacy to regex. Rename file that do not …
CeEv Jan 24, 2025
2835248
Cleanup code
CeEv Jan 24, 2025
f5e32e1
Refactor entity paths in setupNestServices for consistency
SevenWaysDP Jan 27, 2025
db526d6
BC-8551 - readd and reduce entities to `files-storage-test.module.ts`
bergatco Jan 27, 2025
bd922ce
BC-8551 - remove `courseFactory` from `files-storage.consumer.spec.ts`
bergatco Jan 27, 2025
62210d8
BC-8551 - remove `courseFactory` from `files-storage-copy-files.api.s…
bergatco Jan 27, 2025
134fd5b
Merge branch 'main' into BC-8551-remove-all-entities
CeEv Jan 27, 2025
26ad5a7
Fix test setup for legacy registrationPins. It want to try cleanup pi…
CeEv Jan 27, 2025
352a363
Resolve unnecessary dependencies to other entities by using UserAndAc…
CeEv Jan 27, 2025
640e7e2
Refactor import path for DashboardEntity and GridElement in dashboard…
SevenWaysDP Jan 27, 2025
0555a25
Remove added entities in files-storage for tests.
CeEv Jan 27, 2025
c9353d0
Merge branch 'main' into BC-8551-remove-all-entities
CeEv Jan 27, 2025
1ea9298
Merge remote-tracking branch 'origin/BC-8551-remove-all-entities' int…
SevenWaysDP Jan 27, 2025
e6481a4
Refactor import path for GridElement in dashboardElement.entity.spec.ts
SevenWaysDP Jan 27, 2025
42c2d55
Modified helper to work without persists.
CeEv Jan 27, 2025
2f48846
linter
CeEv Jan 27, 2025
7b4fd21
Merge branch 'main' into BC-8551-remove-all-entities
CeEv Jan 28, 2025
5ccfe8b
Add loginByUser method to TestApiClient for JWT authentication
SevenWaysDP Jan 28, 2025
04e3a6b
Refactor H5P editor tests to use loginByUser method
SevenWaysDP Jan 28, 2025
615b5fe
Add getAuthHeader method and refactor file storage tests to use TestA…
SevenWaysDP Jan 28, 2025
532b093
Update CurrentUserMapper tests to use user school ID and add User and…
SevenWaysDP Jan 28, 2025
967993e
Refactor entity imports in FwuLearningContents modules for better org…
SevenWaysDP Jan 29, 2025
6012321
Integrate MikroOrmModule for MongoDB support in FilesStorage modules …
SevenWaysDP Jan 29, 2025
562cea5
Refactor H5P editor module to use centralized entity exports for impr…
SevenWaysDP Jan 29, 2025
a606f33
Refactor management module to centralize entity imports for improved …
SevenWaysDP Jan 30, 2025
7f81788
Refactor MikroORM configuration to separate entity imports into a ded…
SevenWaysDP Jan 30, 2025
41afc94
Refactor server module to centralize entity imports into a dedicated …
SevenWaysDP Jan 30, 2025
e3b827a
Refactor board module to centralize entity imports into a dedicated f…
SevenWaysDP Jan 30, 2025
c6379d7
Refactor files-storage module to separate test entities for improved …
SevenWaysDP Jan 30, 2025
d3b4a9b
Refactor fwu-learning-contents module to separate test entities for i…
SevenWaysDP Jan 30, 2025
4107f40
Refactor admin API server module to centralize entity imports and imp…
SevenWaysDP Jan 30, 2025
152c283
Refactor admin API server entity imports to include additional tool e…
SevenWaysDP Jan 30, 2025
6d86f2d
Refactor server module to include TEST_ENTITIES for improved testing …
SevenWaysDP Jan 30, 2025
359671c
Merge branch 'main' into BC-8551-remove-all-entities
SevenWaysDP Jan 30, 2025
d29a2fa
Merge remote-tracking branch 'origin/BC-8551-remove-all-entities' int…
SevenWaysDP Jan 30, 2025
155b811
Refactor MongoMemoryDatabaseModule to use TEST_ENTITIES for improved …
SevenWaysDP Jan 30, 2025
1f8aae0
Add migrations.entity.imports for centralized entity imports in serve…
SevenWaysDP Jan 30, 2025
9159ae5
Refactor setupNestServices to use TEST_ENTITIES for improved testing …
SevenWaysDP Jan 31, 2025
75d0b5f
Remove TEST_ENTITIES from MongoMemoryDatabaseModule for improved conf…
SevenWaysDP Jan 31, 2025
2e90bb5
Merge branch 'main' into BC-8551-remove-all-entities
SevenWaysDP Jan 31, 2025
f0e1318
Update MongoMemoryDatabaseModule imports to specify entities for *.r…
SevenWaysDP Jan 31, 2025
041c476
Add entities to repo tests
bischofmax Jan 31, 2025
449ef07
fixup! Update MongoMemoryDatabaseModule imports to specify entities f…
SevenWaysDP Jan 31, 2025
504c312
Merge remote-tracking branch 'origin/BC-8551-remove-all-entities' int…
SevenWaysDP Jan 31, 2025
d6e0143
Refactor H5P editor module to use TEST_ENTITIES for MongoMemoryDataba…
SevenWaysDP Jan 31, 2025
cec4322
Add entities to integration tests
bischofmax Jan 31, 2025
9213dd3
Merge branch 'BC-8551-remove-all-entities' of github.com:hpi-schul-cl…
bischofmax Jan 31, 2025
7450d45
Update MongoMemoryDatabaseModule imports to specify entities in ident…
SevenWaysDP Jan 31, 2025
be9efe9
Refactor deletion-console module to centralize entity imports and use…
SevenWaysDP Jan 31, 2025
06bf1b1
Merge remote-tracking branch 'origin/BC-8551-remove-all-entities' int…
SevenWaysDP Jan 31, 2025
49ec7dc
Add entities to more tests
bischofmax Jan 31, 2025
cc6878d
Refactor server-console module to management-console, removing unused…
SevenWaysDP Jan 31, 2025
88f88a8
Add entities to tests
bischofmax Jan 31, 2025
0631e7a
Merge remote-tracking branch 'origin/main' into BC-8551-remove-all-en…
SevenWaysDP Feb 3, 2025
d2d75db
Refactor database management module: move service and tests to manage…
SevenWaysDP Feb 3, 2025
41c0333
Refactor IDP console module: consolidate entity imports and update te…
SevenWaysDP Feb 3, 2025
20cf968
Fix import issue in setup-entities.ts: temporarily disable eslint rul…
SevenWaysDP Feb 3, 2025
9e6541a
Remove unused MikroORM CLI configuration and entity imports; update d…
SevenWaysDP Feb 4, 2025
f3e1123
fixup! Remove unused MikroORM CLI configuration and entity imports; u…
SevenWaysDP Feb 4, 2025
aec7720
Update sonar-project.properties: add setup-entities.ts to CPD exclusions
SevenWaysDP Feb 4, 2025
864e9fa
Merge remote-tracking branch 'origin/main' into BC-8551-remove-all-en…
SevenWaysDP Feb 4, 2025
620ef66
Update sonar-project.properties to correct CPD exclusions for setup-e…
SevenWaysDP Feb 4, 2025
2b35c62
code review
SevenWaysDP Feb 4, 2025
7690a35
check entities in deletion-console entity imports
SevenWaysDP Feb 4, 2025
edd6a39
Add missing entity imports to admin-api-server.entity.imports.ts
SevenWaysDP Feb 4, 2025
3f8f5f8
Refactor user module to integrate FilesStorageClientAdapterService an…
SevenWaysDP Feb 5, 2025
209e447
Refactor management-console module imports for clarity and consistency
SevenWaysDP Feb 5, 2025
0958e65
Merge branch 'main' into BC-8551-remove-all-entities
SevenWaysDP Feb 5, 2025
988adde
Integrate RabbitMQWrapperModule into deletion-console and files-stora…
SevenWaysDP Feb 5, 2025
6026e0c
Remove RabbitMQWrapperModule and related test modules from deletion-c…
SevenWaysDP Feb 6, 2025
f8d9db6
revert remove MikroORM from common-cartridge module
SevenWaysDP Feb 6, 2025
004e887
Revert "Refactor user module to integrate FilesStorageClientAdapterSe…
SevenWaysDP Feb 6, 2025
6900e4f
Merge remote-tracking branch 'origin/main' into BC-8551-remove-all-en…
SevenWaysDP Feb 6, 2025
e322d63
Merge remote-tracking branch 'origin/main' into BC-8551-remove-all-en…
SevenWaysDP Feb 7, 2025
20c135e
Merge branch 'main' into BC-8551-remove-all-entities
SevenWaysDP Feb 7, 2025
74c16c8
Merge remote-tracking branch 'origin/main' into BC-8551-remove-all-en…
SevenWaysDP Feb 10, 2025
4be23a7
Remove unused PseudonymEntity imports from various modules
SevenWaysDP Feb 10, 2025
63bb618
Merge branch 'main' into BC-8551-remove-all-entities
CeEv Feb 10, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix unresolved merge conflict
  • Loading branch information
CeEv committed Jan 23, 2025
commit 0e7f1a870b20198abd8778556fff0da001ccf42a
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { Configuration } from '@hpi-schul-cloud/commons/lib';
import { DB_PASSWORD, DB_URL, DB_USERNAME } from '@src/imports-from-feathers';
import { DB_PASSWORD, DB_URL, DB_USERNAME } from '@imports-from-feathers';
import { ConsoleWriterModule } from '@infra/console/console-writer/console-writer.module';
import { KeycloakModule } from '@infra/identity-management/keycloak/keycloak.module';
import { SyncModule } from '@infra/sync/sync.module';

Unchanged files with check annotations Beta

// application imports
import { LegacyLogger, Logger } from '@core/logger';
import { RedisIoAdapter } from '@infra/socketio';

Check warning on line 10 in apps/server/src/apps/board-collaboration.app.ts

GitHub Actions / nest_lint

'@infra/socketio' import is restricted from being used by a pattern. apps-modules may NOT import from @apps, @infra, @shared, or migrations
import { BoardCollaborationModule } from '@modules/board/board-collaboration.app.module';
import { ExpressAdapter } from '@nestjs/platform-express';
import { SwaggerDocumentOptions } from '@nestjs/swagger';
import { DeletionConsoleModule } from '@modules/deletion-console/deletion-console.app.module';
import { BootstrapConsole } from 'nestjs-console';
async function run() {

Check warning on line 5 in apps/server/src/apps/deletion-console.app.ts

GitHub Actions / nest_lint

Missing return type on function
const bootstrap = new BootstrapConsole({
module: DeletionConsoleModule,
useDecorators: true,
export class AppStartLoggable implements Loggable {
constructor(private readonly info: AppStartInfo) {}
getLogMessage(): LogMessage {

Check warning on line 13 in apps/server/src/apps/helpers/app-start-loggable.ts

GitHub Actions / nest_lint

Missing accessibility modifier on method definition getLogMessage
const data: LogMessageData = { appName: this.info.appName };
if (this.info.port !== undefined) {
export class PrometheusMetricsSetupStateLoggable implements Loggable {
constructor(private readonly state: PrometheusMetricsSetupState) {}
getLogMessage(): LogMessage {

Check warning on line 22 in apps/server/src/apps/helpers/metrics/prometheus-metrics.ts

GitHub Actions / nest_lint

Missing accessibility modifier on method definition getLogMessage
return {
message: 'Setting up Prometheus metrics...',
data: {
}
}
export const addPrometheusMetricsMiddlewaresIfEnabled = (logger: Logger, app: Express) => {

Check warning on line 32 in apps/server/src/apps/helpers/metrics/prometheus-metrics.ts

GitHub Actions / nest_lint

Missing return type on function
if (!PrometheusMetricsConfig.instance.isEnabled) {
logger.debug(
new PrometheusMetricsSetupStateLoggable(
);
};
export const createAndStartPrometheusMetricsAppIfEnabled = (logger: Logger) => {

Check warning on line 52 in apps/server/src/apps/helpers/metrics/prometheus-metrics.ts

GitHub Actions / nest_lint

Missing return type on function
if (!PrometheusMetricsConfig.instance.isEnabled) {
logger.debug(
new PrometheusMetricsSetupStateLoggable(PrometheusMetricsSetupState.FEATURE_DISABLED_APP_WILL_NOT_BE_CREATED)
this._collectMetricsRouteMetrics = Configuration.get('PROMETHEUS_METRICS_COLLECT_METRICS_ROUTE_METRICS') as boolean;
}
public static get instance() {

Check warning on line 44 in apps/server/src/apps/helpers/metrics/prometheus/config.ts

GitHub Actions / nest_lint

Public accessibility modifier on get property accessor instance

Check warning on line 44 in apps/server/src/apps/helpers/metrics/prometheus/config.ts

GitHub Actions / nest_lint

Missing return type on function
if (this._instance === undefined) {
this._instance = new this();
}
return this._instance;
}
public static reload() {

Check warning on line 52 in apps/server/src/apps/helpers/metrics/prometheus/config.ts

GitHub Actions / nest_lint

Missing return type on function
this._instance = new this();
}
}
});
it('return HTTP 500 if the Prometheus client metrics() method call fails', async () => {
clientRegisterMetricsSpy.mockImplementation(async (): Promise<string> => Promise.reject(Error('test error')));

Check warning on line 33 in apps/server/src/apps/helpers/metrics/prometheus/handler.spec.ts

GitHub Actions / nest_lint

Async arrow function has no 'await' expression
const response = await request(app).get('/metrics');
import { BoardWsApiModule } from './board-ws-api.module';
import { BoardModule } from './board.module';

Check failure on line 66 in apps/server/src/modules/board/board-collaboration.app.module.ts

GitHub Actions / nest_lint

Delete `⏎`
export const ENTITIES = [
AccountEntity,
LegacyBoard,
ENV NODE_ENV=production
ENV NO_COLOR="true"
CMD npm run start

Check warning on line 27 in Dockerfile

GitHub Actions / build_and_push

JSON arguments recommended for ENTRYPOINT/CMD to prevent unintended behavior related to OS signals

JSONArgsRecommended: JSON arguments recommended for CMD to prevent unintended behavior related to OS signals More info: https://docs.docker.com/go/dockerfile/rule/json-args-recommended/
ARG BASE_IMAGE
FROM $BASE_IMAGE

Check warning on line 2 in Dockerfile.filepreview

GitHub Actions / build_and_push

Default value for global ARG results in an empty or invalid base image name

InvalidDefaultArgInFrom: Default value for ARG $BASE_IMAGE results in empty or invalid base image name More info: https://docs.docker.com/go/dockerfile/rule/invalid-default-arg-in-from/
RUN apk add --no-cache imagemagick imagemagick-heic imagemagick-jpeg imagemagick-pdf imagemagick-raw imagemagick-svg imagemagick-tiff imagemagick-webp