Skip to content

Commit f37f071

Browse files
rm useless
rm logs rm commit type add eng lang
1 parent 111dc39 commit f37f071

File tree

3 files changed

+18
-82
lines changed

3 files changed

+18
-82
lines changed

src/models/releaseFactory.ts

Lines changed: 7 additions & 47 deletions
Original file line numberDiff line numberDiff line change
@@ -21,54 +21,17 @@ export default class ReleasesFactory {
2121
constructor(dbConnection: Db, dataLoaders: DataLoaders) {
2222
this.collection = dbConnection.collection('releases');
2323
this.dataLoaders = dataLoaders;
24-
console.log("[ReleasesFactory] Initialized with collection 'releases'");
2524
}
2625

2726
/**
2827
* Get a release by its identifier using DataLoader
29-
* @param id - release identifier
28+
* @param releaseId - release identifier
3029
*/
31-
public async getReleaseById(id: string): Promise<ReleaseDBScheme | null> {
32-
console.log(`[ReleasesFactory] getReleaseById called with id: ${id}`);
30+
public async findById(releaseId: string): Promise<ReleaseDBScheme | null> {
3331
try {
34-
const release = await this.dataLoaders.releaseById.load(id);
35-
console.log(`[ReleasesFactory] getReleaseById result:`, release);
36-
return release;
32+
return await this.dataLoaders.releaseById.load(releaseId);
3733
} catch (error) {
38-
console.error(`[ReleasesFactory] Error in getReleaseById:`, error);
39-
throw error;
40-
}
41-
}
42-
43-
/**
44-
* Get all releases
45-
*/
46-
public async getAllReleases(): Promise<ReleaseDBScheme[]> {
47-
console.log(`[ReleasesFactory] getAllReleases called`);
48-
try {
49-
const releases = await this.collection.find({}).toArray();
50-
console.log(`[ReleasesFactory] getAllReleases returned ${releases.length} releases`);
51-
return releases;
52-
} catch (error) {
53-
console.error(`[ReleasesFactory] Error in getAllReleases:`, error);
54-
throw error;
55-
}
56-
}
57-
58-
/**
59-
* Get releases with pagination
60-
* @param page - page number (starting from 1)
61-
* @param limit - number of items per page
62-
*/
63-
public async getReleasesPaginated(page: number, limit: number): Promise<ReleaseDBScheme[]> {
64-
const skip = (page - 1) * limit;
65-
console.log(`[ReleasesFactory] getReleasesPaginated called with page: ${page}, limit: ${limit}, skip: ${skip}`);
66-
try {
67-
const releases = await this.collection.find({}).skip(skip).limit(limit).toArray();
68-
console.log(`[ReleasesFactory] getReleasesPaginated returned ${releases.length} releases`);
69-
return releases;
70-
} catch (error) {
71-
console.error(`[ReleasesFactory] Error in getReleasesPaginated:`, error);
34+
console.error(`[ReleasesFactory] Error in findById:`, error);
7235
throw error;
7336
}
7437
}
@@ -77,14 +40,11 @@ export default class ReleasesFactory {
7740
* Get releases by project identifier
7841
* @param projectId - project identifier
7942
*/
80-
public async getReleasesByProjectId(projectId: string): Promise<ReleaseDBScheme[]> {
81-
console.log(`[ReleasesFactory] getReleasesByProjectId called with projectId: ${projectId}`);
43+
public async findManyByProjectId(projectId: string): Promise<ReleaseDBScheme[]> {
8244
try {
83-
const releases = await this.collection.find({ projectId: projectId }).toArray();
84-
console.log(`[ReleasesFactory] getReleasesByProjectId returned ${releases.length} releases`);
85-
return releases;
45+
return await this.collection.find({ projectId: projectId }).toArray();
8646
} catch (error) {
87-
console.error(`[ReleasesFactory] Error in getReleasesByProjectId:`, error);
47+
console.error(`[ReleasesFactory] Error in findManyByProjectId:`, error);
8848
throw error;
8949
}
9050
}

src/resolvers/release.ts

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3,21 +3,22 @@ import ReleasesFactory from '../models/releaseFactory';
33
export default {
44
Query: {
55
/**
6-
* Fetch all releases or releases filtered by projectId
6+
* Fetch releases by projectId
77
* @param {ResolverObj} _ - Parent object, not used
8-
* @param {ResolverArgs} args - Query arguments
8+
* @param {ResolverArgs} args - Query arguments containing required projectId
99
* @param {ContextFactories} context - Global GraphQL context with factories
1010
* @returns {Promise<Release[]>}
1111
*/
12-
getReleases: async (_: any, args: { projectId?: string }, { factories }: any) => {
12+
getReleases: async (_: any, args: { projectId: string }, { factories }: any) => {
13+
if (!args.projectId) {
14+
throw new Error('projectId is required to fetch releases');
15+
}
16+
1317
try {
14-
if (args.projectId) {
15-
return await factories.releasesFactory.getReleasesByProjectId(args.projectId);
16-
}
17-
return await factories.releasesFactory.getAllReleases();
18+
return await factories.releasesFactory.findManyByProjectId(args.projectId);
1819
} catch (error) {
1920
console.error('Error fetching releases:', error);
20-
throw new Error('Не удалось получить релизы');
21+
throw new Error('Failed to get the releases');
2122
}
2223
},
2324
},

src/typeDefs/release.ts

Lines changed: 2 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -1,31 +1,6 @@
11
import { gql } from 'apollo-server-express';
22

33
export default gql`
4-
"""
5-
Release commit
6-
"""
7-
type Commit {
8-
"""
9-
Hash of the commit
10-
"""
11-
hash: String!
12-
13-
"""
14-
Commit author
15-
"""
16-
author: String!
17-
18-
"""
19-
Commit title
20-
"""
21-
title: String!
22-
23-
"""
24-
Commit creation date
25-
"""
26-
date: DateTime!
27-
}
28-
294
"""
305
Source map file details
316
"""
@@ -48,7 +23,7 @@ export default gql`
4823
"""
4924
Release name
5025
"""
51-
releaseName: String! @renameFrom(name: "release")
26+
release: String!
5227
5328
"""
5429
Project ID associated with the release
@@ -58,7 +33,7 @@ export default gql`
5833
"""
5934
Release commits
6035
"""
61-
commits: [Commit!]!
36+
commits: [Commit!]
6237
6338
"""
6439
Source maps associated with the release

0 commit comments

Comments
 (0)