• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

moosetechnology / GitProjectHealth / 10880677211

16 Sep 2024 09:01AM UTC coverage: 20.332% (-0.2%) from 20.538%
10880677211

push

github

web-flow
Merge pull request #54 from moosetechnology/metrics-fixing

Metrics fixing

0 of 178 new or added lines in 19 files covered. (0.0%)

18 existing lines in 6 files now uncovered.

1961 of 9645 relevant lines covered (20.33%)

0.2 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

31.13
/src/GitLabProjectHealth-Model-Importer/GLPHModelImporter.class.st
1
Class {
2
        #name : #GLPHModelImporter,
3
        #superclass : #GLHModelImporter,
4
        #category : 'GitLabProjectHealth-Model-Importer'
5
}
6

7
{ #category : #'as yet unclassified' }
8
GLPHModelImporter >> blockForDiffRangeEquality [
×
9

×
10
        ^ [ :existing :new |
×
11
          existing diff mooseID = new diff mooseID and: [
×
12
                  existing originalLineRange = new originalLineRange and: [
×
13
                          existing newLineRange = new newLineRange ] ] ]
×
14
]
×
15

16
{ #category : #commit }
17
GLPHModelImporter >> commitsOfProject: aGLHProject forRefName: refName until: toDate [
×
18

×
19
        | newlyFoundCommit page |
×
20
        page := 0.
×
21
        newlyFoundCommit := { true }.
×
22
        [ newlyFoundCommit isNotEmpty ] whileTrue: [
×
23
                | results parsedResults existingCommits |
×
24
                page := page + 1.
×
25
                ('import commit page ' , page printString) recordInfo.
×
26
                results := self glhApi
×
27
                                   commitsOfProject: aGLHProject id
×
28
                                   forRefName: refName
×
29
                                   since: nil
×
30
                                   until: nil
×
31
                                   path: nil
×
32
                                   author: nil
×
33
                                   all: nil
×
34
                                   with_stats: nil
×
35
                                   firstParent: nil
×
36
                                   order: nil
×
37
                                   trailers: nil
×
38
                                   perPage: 100
×
39
                                   page: page.
×
40
                parsedResults := self parseCommitsResult: results.
×
41
                existingCommits := aGLHProject mooseModel allWithType: GLHCommit.
×
42

×
43
                newlyFoundCommit := parsedResults reject: [ :commitParsed |
×
44
                                            (toDate isNil or: [
×
45
                                                     commitParsed committed_date
×
46
                                                     < toDate asDateAndTime ]) or: [
×
47
                                                    existingCommits anySatisfy: [ :existingCommit |
×
48
                                                            existingCommit id = commitParsed id ] ] ].
×
49
                self glhModel addAll: newlyFoundCommit unless: self blockOnIdEquality.
×
50
                aGLHProject repository commits addAll: newlyFoundCommit unless: self blockOnIdEquality  ].
×
51
        self withCommitDiffs ifTrue: [
×
52
                aGLHProject repository commits do: [ :commit |
×
53
                        self importDiffOfCommit: commit ] ]
×
54
]
×
55

56
{ #category : #'private - api' }
57
GLPHModelImporter >> completeImportProject: aGLHProject [
×
58
        |completedProject|
×
59
        completedProject := super completeImportProject: aGLHProject.
×
60

×
61
        "for this initial import, with only look at some but not all MR"
×
62
        withInitialMergeRequest ifTrue: [ self
×
63
                importMergeRequests: completedProject
×
64
                since: DateAndTime today
×
65
                until: DateAndTime now. ].
×
66

×
67
        ^ completedProject
×
68
]
×
69

70
{ #category : #'private - configure reader' }
71
GLPHModelImporter >> configureReaderForMergeRequest: reader [
1✔
72
        "declare quil y a un array a mapper"
1✔
73

1✔
74
        reader for: #ArrayOfMergeRequest customDo: [ :customMappting |
1✔
75
                customMappting listOfElementSchema: GLPHEMergeRequest ].
1✔
76

1✔
77
        "declare la liste des properties"
1✔
78
        reader for: GLPHEMergeRequest do: [ :mapping |
1✔
79
                mapping mapInstVars:
1✔
80
                        #( blocking_discussions_resolved changes_count description
1✔
81
                           detailed_merge_status discussion_locked downvotes draft first_deployed_to_production_at
1✔
82
                           force_remove_source_branch has_conflicts id iid labels latest_build_finished_at
1✔
83
                           latest_build_started_at merge_commit_sha merge_status
1✔
84
                           merge_when_pipeline_succeeds merged_at milestone project_id
1✔
85
                           reference references_full references_relative
1✔
86
                           references_short sha should_remove_source_branch
1✔
87
                           source_branch source_project_id squash squash_commit_sha
1✔
88
                           squash_on_merge state subscribed target_branch target_project_id
1✔
89
                           task_completion_status_completed_count
1✔
90
                           task_completion_status_count time_stats_human_time_estimate
1✔
91
                           time_stats_human_total_time_spent
1✔
92
                           time_stats_time_estimate time_stats_total_time_spent
1✔
93
                           title updated_at upvotes user_notes_count web_url work_in_progress ).
1✔
94
                (mapping mapInstVar: #created_at) valueSchema: DateAndTime.
1✔
95
                (mapping mapInstVar: #updated_at) valueSchema: DateAndTime.
1✔
96
                (mapping mapInstVar: #merged_at) valueSchema: DateAndTime.
1✔
97
                (mapping mapInstVar: #closed_at) valueSchema: DateAndTime.
1✔
98
                "(mapping mapInstVar: #assignee) valueSchema: GLHUser."
1✔
99
                mapping
1✔
100
                        mapProperty: #author
1✔
101
                        getter: [  ]
1✔
102
                        setter: [ :object :value |
1✔
103
                        object cacheAt: #authorID put: (value at: #id) ].
1✔
104
                mapping
1✔
105
                        mapProperty: #merge_user
1✔
106
                        getter: [  ]
1✔
107
                        setter: [ :object :value | 
1✔
108
                                value ifNotNil: [
1✔
109
                                        object cacheAt: #mergeUserID put: (value at: #id) ] ] ].
1✔
110

1✔
111
        "(mapping mapInstVar: #closed_by) valueSchema: GLHUser.
1✔
112
        (mapping mapInstVar: #mergeCommit) valueSchema: GLHCommit."
1✔
113
        "indique ce que doit faire le reader lorsqu'il parse une DateAndTime object"
1✔
114
        reader for: DateAndTime customDo: [ :mapping |
1✔
115
                mapping decoder: [ :string |
1✔
116
                        string ifNil: [ nil ] ifNotNil: [ DateAndTime fromString: string ] ] ]
1✔
117
]
1✔
118

119
{ #category : #import }
120
GLPHModelImporter >> importCommitsOfBranch: aGLHBranch [
×
121

×
122
        | commits |
×
123
        "        result := self glhApi
×
124
                          commitsOfProject: aGLHBranch repository project id
×
125
                          forRefName: aGLHBranch name."
×
126
        commits := self
×
127
                           importCommitsOfBranch: aGLHBranch
×
128
                           forRefName: aGLHBranch name
×
129
                           since: withCommitsSince.
×
130

×
131
        self chainsCommitsFrom: commits.
×
132

×
133
        commits do: [ :aCommit |
×
134
                aCommit repository: aGLHBranch repository.
×
135
                self completeImportedCommit: aCommit. 
×
136
                 ]
×
137
]
×
138

139
{ #category : #import }
140
GLPHModelImporter >> importCommitsOfMergeResquest: aGLPHEMergeRequest [
×
141

×
142
        | foundCommits |
×
143
        foundCommits := OrderedCollection new.
×
144

×
145
        ('Import commit sha of MR:  ' , aGLPHEMergeRequest iid printString)
×
146
                recordInfo.
×
147
        "the founds commits are added to the model during their respective import"
×
148
        aGLPHEMergeRequest mergeRequestCommit: ((self
×
149
                          importCommitOfProject: aGLPHEMergeRequest project
×
150
                          withId: aGLPHEMergeRequest sha) ifNotNil: [ :commit |
×
151
                         foundCommits add: commit ]).
×
152

×
153
        ('Import commit merge_commit_sha of MR:  '
×
154
         , aGLPHEMergeRequest iid printString) recordInfo.
×
155
        aGLPHEMergeRequest mergedCommit: ((self
×
156
                          importCommitOfProject: aGLPHEMergeRequest project
×
157
                          withId: aGLPHEMergeRequest merge_commit_sha) ifNotNil: [ :commit |
×
158
                         foundCommits add: commit ]).
×
159

×
160
        ('Import commit squash_commit_sha of MR:  '
×
161
         , aGLPHEMergeRequest iid printString) recordInfo.
×
162
        aGLPHEMergeRequest squashCommit: ((self
×
163
                          importCommitOfProject: aGLPHEMergeRequest project
×
164
                          withId: aGLPHEMergeRequest squash_commit_sha) ifNotNil: [ :commit |
×
165
                         foundCommits add: commit ]).
×
166

×
167

×
168
        self chainsCommitsFrom: foundCommits.
×
169
        ^ foundCommits
×
170
]
×
171

172
{ #category : #api }
173
GLPHModelImporter >> importDiffOfCommit: aCommit [
×
174

×
175
        super importDiffOfCommit: aCommit.
×
176
        
×
177
        "changes are added into the model during the import"
×
178
        aCommit diffs do: [ :diff | self importDiffRangesForDiff: diff ].
×
179

×
180
        ^ aCommit diffs
×
181
]
×
182

183
{ #category : #'as yet unclassified' }
184
GLPHModelImporter >> importDiffOfMergeRequest: aMergeRequest [
×
185

×
186
        | result diffsResult |
×
187
        aMergeRequest diffs ifNotEmpty: [
×
188
                'Diff of already importer: '
×
189
                , aMergeRequest iid printString recordInfo.
×
190
                ^ aMergeRequest diffs ].
×
191
        ('Import diff commits of MR ' , aMergeRequest iid printString)
×
192
                recordInfo.
×
193
        result := self glhApi
×
194
                          diffsMergeRequestOfProject: aMergeRequest project_id
×
195
                          withId: aMergeRequest iid.
×
196

×
197
        diffsResult := self newParseDiffResult: result.
×
198

×
199

×
200
        aMergeRequest diffs
×
201
                addAll: diffsResult
×
202
                unless: self blockForDiffEquality.
×
203
        self glhModel
×
204
                addAll: aMergeRequest diffs
×
205
                unless: self blockForDiffEquality.
×
206

×
207
        aMergeRequest diffs do: [ :diff | self importDiffRangesForDiff: diff ].
×
208

×
209
        ^ aMergeRequest diffs
×
210
]
×
211

212
{ #category : #import }
213
GLPHModelImporter >> importDiffRangesForDiff: aGLHDiff [
×
214

×
215
        | diffRanges |
×
216
        aGLHDiff diffRanges ifNotEmpty: [
×
217
                ('Diffs already imported for ' , aGLHDiff printString) recordInfo.
×
218
                ^ aGLHDiff diffRanges ].
×
219

×
220
        ('Import changes of diff ' , aGLHDiff printString) recordInfo.
×
221
        "the import of 'diffRanges' entites is technically done inside the parseDiffString"
×
222

×
223
        diffRanges := self glhModel
×
224
                              addAll: (self parseDiffString: aGLHDiff)
×
225
                              unless: self blockForDiffRangeEquality.
×
226
        ^ aGLHDiff diffRanges
×
227
                  addAll: diffRanges
×
228
                  unless: self blockForDiffRangeEquality
×
229
]
×
230

231
{ #category : #'import - merge request' }
232
GLPHModelImporter >> importMergeRequests: aGLHProject [
×
233

×
234
        | results parsedResults mrs|
×
235
        ('Import merge request of Project: ' , aGLHProject id printString)
×
236
                recordInfo.
×
237

×
238
        results := self glhApi mergeRequestOfProject: aGLHProject id.
×
239
        parsedResults := self parseMergeRequestResult: results.
×
240

×
241
        aGLHProject mergeRequests
×
242
                addAll: parsedResults
×
243
                unless: self blockOnIdEquality.
×
244

×
245
        mrs := self glhModel
×
246
                addAll: aGLHProject mergeRequests
×
247
                unless: self blockOnIdEquality.
×
248

×
249

×
250
        "gets it related commits"
×
251
        aGLHProject mergeRequests do: [ :mr |
×
252
                self importCommitsOfMergeResquest: mr ].
×
253

×
254

×
255
        self withCommitDiffs ifTrue: [
×
256
                aGLHProject mergeRequests do: [ :mr |
×
257
                        self importDiffOfMergeRequest: mr ] ].
×
258
        
×
259
        ^ mrs
×
260
]
×
261

262
{ #category : #'import - merge request' }
263
GLPHModelImporter >> importMergeRequests: aGLHProject since: fromDate until: toDate [
×
264

×
265
        | newlyFoundMR page foundMR |
×
266
        ('import MR of Project ' , aGLHProject name) recordInfo.
×
267

×
268
        page := 0.
×
269
        foundMR := OrderedCollection new.
×
270
        newlyFoundMR := { true }.
×
271
        [ newlyFoundMR isNotEmpty ] whileTrue: [
×
272
                | results |
×
273
                page := page + 1.
×
274
                ('import MR page ' , page printString) recordInfo.
×
275
                results := self glhApi
×
276
                                   mergeRequestsOfProject: aGLHProject id
×
277
                                   createdAfter:
×
278
                                   (fromDate ifNotNil: [ fromDate asDateAndTime asString ])
×
279
                                   createdBefore:
×
280
                                   (toDate ifNotNil: [ toDate asDateAndTime asString ])
×
281
                                   authorId: nil
×
282
                                   authorUsername: nil
×
283
                                   scope: 'all'
×
284
                                   orderBy: 'created_at'
×
285
                                   page: page.
×
286

×
287

×
288
                newlyFoundMR := aGLHProject mergeRequests
×
289
                                        addAll: (self parseMergeRequestResult: results)
×
290
                                        unless: self blockOnIdEquality.
×
291
                foundMR addAll: newlyFoundMR ].
×
292

×
293

×
294
        "gets it related commits"
×
295
        aGLHProject mergeRequests do: [ :mr |
×
296
                self importCommitsOfMergeResquest: mr ].
×
297

×
298
        self withCommitDiffs ifTrue: [
×
299
                aGLHProject mergeRequests do: [ :mr |
×
300
                        self importDiffOfMergeRequest: mr ] ].
×
301

×
302
        self glhModel addAll: foundMR unless: (self blockEqualityOn: #iid).
×
303

×
304
        ^ foundMR
×
305
]
×
306

307
{ #category : #'import - merge request' }
308
GLPHModelImporter >> importMergeResquestApprovals: aGLPHEMergeRequest [
×
309

×
310
        | results parsedResult |
×
311
        (String streamContents: [ :str |
×
312
                 str << 'Check approvals of '.
×
313
                 aGLPHEMergeRequest printOn: str ]) recordInfo.
×
314
        results := self glhApi
×
315
                           approvalsOfMergeQuest: aGLPHEMergeRequest iid
×
316
                           ofProject: aGLPHEMergeRequest project id.
×
317

×
318
        parsedResult := generalReader
×
319
                                on: results readStream;
×
320
                                next.
×
321

×
322
        (parsedResult at: #approved_by) do: [ :approvedUser |
×
323
                aGLPHEMergeRequest addApproved_by:
×
324
                        (self importUser: ((approvedUser at: #user) at: #id)) ].
×
325
        aGLPHEMergeRequest approved: (parsedResult at: #approved).
×
326
        ^ aGLPHEMergeRequest
×
327
]
×
328

329
{ #category : #'import - merge request' }
330
GLPHModelImporter >> importMergeResquestAuthor: aGLPHEMergeRequest [
1✔
331

1✔
332
        | authorID |
1✔
333
        aGLPHEMergeRequest author ifNotNil: [ ^ aGLPHEMergeRequest author ].
1✔
334
        authorID := aGLPHEMergeRequest cacheAt: #authorID ifAbsent: [
1✔
335
                            (generalReader
1✔
336
                                     on: (self glhApi
1✔
337
                                                      mergeRequestOfProject:
1✔
338
                                                              aGLPHEMergeRequest project_id
1✔
339
                                                      withId: aGLPHEMergeRequest iid) readStream;
1✔
340
                                     next) at: #author at: #id ].
1✔
341
        aGLPHEMergeRequest author: (self importUser: authorID)
1✔
342
]
1✔
343

344
{ #category : #'import - merge request' }
345
GLPHModelImporter >> importMergeResquestMerger: aGLPHEMergeRequest [
1✔
346

1✔
347
        | authorID |
1✔
348
        aGLPHEMergeRequest merge_user ifNotNil: [
1✔
349
                ^ aGLPHEMergeRequest merge_user ].
1✔
350
        authorID := aGLPHEMergeRequest cacheAt: #mergeUserID ifAbsent: [
1✔
351
                            (generalReader
1✔
352
                                     on: (self glhApi
1✔
353
                                                      mergeRequestOfProject:
1✔
354
                                                              aGLPHEMergeRequest project_id
1✔
355
                                                      withId: aGLPHEMergeRequest iid) readStream;
1✔
356
                                     next)
1✔
357
                                    at: #merge_user
1✔
358
                                    ifPresent: [ :mergeUser |
1✔
359
                                    mergeUser ifNotNil: [ :mruser | mruser at: #id ] ] ].
1✔
360
        aGLPHEMergeRequest merge_user: (self importUser: authorID)
1✔
361
]
1✔
362

363
{ #category : #'private - api' }
UNCOV
364
GLPHModelImporter >> importProject: aProjectID [
×
365

×
366
        | result projectResult |
×
367
        ('Import project with id:  ' , aProjectID printString) recordInfo.
×
368

×
369
        result := self glhApi project: aProjectID.
×
370
        projectResult := self parseProjectResult: result.
×
371

×
372
        ^ self completeImportProject: projectResult
×
373
]
×
374

375
{ #category : #'private - api' }
376
GLPHModelImporter >> importRepository: aGLHRepository [
×
377

×
378
        super importRepository: aGLHRepository.
×
379

×
380
        withInitialCommits ifTrue: [
×
381
                aGLHRepository branches do: [ :branch |
×
382
                        self importCommitsOfBranch: branch ] ]
×
383
]
×
384

385
{ #category : #initialization }
386
GLPHModelImporter >> initReader [
1✔
387

1✔
388
        super initReader.
1✔
389
        self configureReaderForMergeRequest: generalReader
1✔
390
]
1✔
391

392
{ #category : #initialization }
393
GLPHModelImporter >> initialize [
1✔
394

1✔
395
        super initialize.
1✔
396
        withCommitDiffs := true
1✔
397
]
1✔
398

399
{ #category : #parsing }
400
GLPHModelImporter >> parseDiffString: aDiff [
1✔
401
        "parse diff string to create entities for each lines"
1✔
402

1✔
403
        | lines range index firstChangeRange |
1✔
404
        lines := aDiff diffString lines.
1✔
405
        index := 0.
1✔
406
        range := nil.
1✔
407

1✔
408
        lines ifEmpty: [ ^ aDiff diffRanges ].
1✔
409
        lines := self filterCommitChanges: lines.
1✔
410
        (lines first beginsWith: '@@') ifFalse: [ ^ aDiff diffRanges ].
1✔
411

1✔
412
        lines do: [ :line |
1✔
413
                | aChange |
1✔
414
                firstChangeRange := false.
1✔
415
                (line trim beginsWith: #'@@') ifTrue: [
1✔
416
                        range := GLPHEDiffRange newFrom: line.
1✔
417
                        firstChangeRange := true.
1✔
418
                        range := aDiff diffRanges add: range unless: [ :existing :new |
1✔
419
                                         existing originalLineRange = new originalLineRange and: [
1✔
420
                                                 existing newLineRange = new newLineRange ] ] ].
1✔
421

1✔
422
                self withCommitDiffs ifTrue: [
1✔
423
                        firstChangeRange ifFalse: [
1✔
424
                                aChange := GLPHEChange newFrom: line.
1✔
425
                                aChange relativeIndex: index.
1✔
426
                                aChange := range changes add: aChange unless: [ :existing :new |
1✔
427
                                                   existing sourceCode = new sourceCode and: [
1✔
428
                                                           existing index = new index ] ].
1✔
429
                                index := index + 1 ] ] ].
1✔
430

1✔
431
        "add changes to the diff range"
1✔
432
        aDiff diffRanges do: [ :diffRange |
1✔
433
                self glhModel addAll: diffRange changes unless: [ :existing :new |
1✔
434
                        existing diffRange mooseID = new diffRange mooseID and: [
1✔
435
                                existing sourceCode = new sourceCode and: [
1✔
436
                                        existing index = new index ] ] ] ].
1✔
437

1✔
438
        ^ aDiff diffRanges
1✔
439
]
1✔
440

441
{ #category : #parsing }
442
GLPHModelImporter >> parseMergeRequestResult: result [
1✔
443

1✔
444
        generalReader on: result readStream.
1✔
445
        ^ generalReader nextAs: #ArrayOfMergeRequest
1✔
446
]
1✔
447

448
{ #category : #parsing }
UNCOV
449
GLPHModelImporter >> parseProjectResult: aResult [ 
×
450
                | reader |
×
451
        reader := NeoJSONReader on: aResult readStream.
×
452
        reader for: GLHProject do: [ :mapping |
×
453
                mapping mapInstVars. ].
×
454
"        reader mapInstVarsFor: GLHProject."
×
455

×
456
        ^ reader nextAs: GLHProject
×
457
]
×
458

×
NEW
459
{ #category : #accessing }
×
460
GLPHModelImporter >> withCommitsSince: someDays [
×
461
        "substract the current date with a given number of days. Use to retrieve the commits submit in the last giving days"
×
462

×
463
        "(GLPHModelImporter new withCommitsSince: 1 week ) >>> (Date today - 1 week) asDateAndTime"
×
464

×
465
        "(GLPHModelImporter new withCommitsSince: 30 day ) >>> (Date today - 30 day) asDateAndTime"
×
466

×
467
        (someDays isKindOf: DateAndTime)
×
468
                ifTrue: [ withCommitsSince := someDays ]
×
469
                ifFalse: [
×
470
                withCommitsSince := (Date today - someDays) asDateAndTime ]
×
471
]
×
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2025 Coveralls, Inc