10880334038

Committed 16 Sep 2024 08:38AM UTC coverage: 20.325% (-0.2%) from 20.538%

Build # 10880334038

Build Type

Pull #54

github

Committed by

web-flow

Commit Message

Merge 9db8bd0be into aef27716a

Pull Request Pull Request #54: Metrics fixing

Run Details

20 of 234 new or added lines in 20 files covered. (8.55%)

252 existing lines in 18 files now uncovered.

1961 of 9648 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

Class {
        #name : 'GLPHModelImporter',
        #superclass : 'GLHModelImporter',
        #category : 'GitLabProjectHealth-Model-Importer',
        #package : 'GitLabProjectHealth-Model-Importer'
}

{ #category : 'as yet unclassified' }
GLPHModelImporter >> blockForDiffRangeEquality [

        ^ [ :existing :new |
          existing diff mooseID = new diff mooseID and: [
                  existing originalLineRange = new originalLineRange and: [
                          existing newLineRange = new newLineRange ] ] ]
]

{ #category : #commit }
GLPHModelImporter >> commitsOfProject: aGLHProject forRefName: refName until: toDate [

        | newlyFoundCommit page |
        page := 0.
        newlyFoundCommit := { true }.
        [ newlyFoundCommit isNotEmpty ] whileTrue: [
                | results parsedResults existingCommits |
                page := page + 1.
                ('import commit page ' , page printString) recordInfo.
                results := self glhApi
                                   commitsOfProject: aGLHProject id
                                   forRefName: refName
                                   since: nil
                                   until: nil
                                   path: nil
                                   author: nil
                                   all: nil
                                   with_stats: nil
                                   firstParent: nil
                                   order: nil
                                   trailers: nil
                                   perPage: 100
                                   page: page.
                parsedResults := self parseCommitsResult: results.
                existingCommits := aGLHProject mooseModel allWithType: GLHCommit.

                newlyFoundCommit := parsedResults reject: [ :commitParsed |
                                            (toDate isNil or: [
                                                     commitParsed committed_date
                                                     < toDate asDateAndTime ]) or: [
                                                    existingCommits anySatisfy: [ :existingCommit |
                                                            existingCommit id = commitParsed id ] ] ].
                self glhModel addAll: newlyFoundCommit unless: self blockOnIdEquality.
                aGLHProject repository commits addAll: newlyFoundCommit unless: self blockOnIdEquality  ].
        self withCommitDiffs ifTrue: [
                aGLHProject repository commits do: [ :commit |
                        self importDiffOfCommit: commit ] ]
]

{ #category : 'private - api' }
GLPHModelImporter >> completeImportProject: aGLHProject [
        |completedProject|
        completedProject := super completeImportProject: aGLHProject.

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

        ^ completedProject
]

{ #category : 'private - configure reader' }
GLPHModelImporter >> configureReaderForMergeRequest: reader [
        "declare quil y a un array a mapper"

        reader for: #ArrayOfMergeRequest customDo: [ :customMappting |
                customMappting listOfElementSchema: GLPHEMergeRequest ].

        "declare la liste des properties"
        reader for: GLPHEMergeRequest do: [ :mapping |
                mapping mapInstVars:
                        #( blocking_discussions_resolved changes_count description
                           detailed_merge_status discussion_locked downvotes draft first_deployed_to_production_at
                           force_remove_source_branch has_conflicts id iid labels latest_build_finished_at
                           latest_build_started_at merge_commit_sha merge_status
                           merge_when_pipeline_succeeds merged_at milestone project_id
                           reference references_full references_relative
                           references_short sha should_remove_source_branch
                           source_branch source_project_id squash squash_commit_sha
                           squash_on_merge state subscribed target_branch target_project_id
                           task_completion_status_completed_count
                           task_completion_status_count time_stats_human_time_estimate
                           time_stats_human_total_time_spent
                           time_stats_time_estimate time_stats_total_time_spent
                           title updated_at upvotes user_notes_count web_url work_in_progress ).
                (mapping mapInstVar: #created_at) valueSchema: DateAndTime.
                (mapping mapInstVar: #updated_at) valueSchema: DateAndTime.
                (mapping mapInstVar: #merged_at) valueSchema: DateAndTime.
                (mapping mapInstVar: #closed_at) valueSchema: DateAndTime.
                "(mapping mapInstVar: #assignee) valueSchema: GLHUser."
                mapping
                        mapProperty: #author
                        getter: [  ]
                        setter: [ :object :value |
                        object cacheAt: #authorID put: (value at: #id) ].
                mapping
                        mapProperty: #merge_user
                        getter: [  ]
                        setter: [ :object :value | 
                                value ifNotNil: [
                                        object cacheAt: #mergeUserID put: (value at: #id) ] ] ].

        "(mapping mapInstVar: #closed_by) valueSchema: GLHUser.
        (mapping mapInstVar: #mergeCommit) valueSchema: GLHCommit."
        "indique ce que doit faire le reader lorsqu'il parse une DateAndTime object"
        reader for: DateAndTime customDo: [ :mapping |
                mapping decoder: [ :string |
                        string ifNil: [ nil ] ifNotNil: [ DateAndTime fromString: string ] ] ]
]

{ #category : 'import' }
GLPHModelImporter >> importCommitsOfBranch: aGLHBranch [

        | commits |
        "        result := self glhApi
                          commitsOfProject: aGLHBranch repository project id
                          forRefName: aGLHBranch name."
        commits := self
                           importCommitsOfBranch: aGLHBranch
                           forRefName: aGLHBranch name
                           since: withCommitsSince.

        self chainsCommitsFrom: commits.

        commits do: [ :aCommit |
                aCommit repository: aGLHBranch repository.
                self completeImportedCommit: aCommit. 
                 ]
]

{ #category : 'import' }
GLPHModelImporter >> importCommitsOfMergeResquest: aGLPHEMergeRequest [

        | foundCommits |
        foundCommits := OrderedCollection new.

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

        ('Import commit merge_commit_sha of MR:  '
         , aGLPHEMergeRequest iid printString) recordInfo.
        aGLPHEMergeRequest mergedCommit: ((self
                          importCommitOfProject: aGLPHEMergeRequest project
                          withId: aGLPHEMergeRequest merge_commit_sha) ifNotNil: [ :commit |
                         foundCommits add: commit ]).

        ('Import commit squash_commit_sha of MR:  '
         , aGLPHEMergeRequest iid printString) recordInfo.
        aGLPHEMergeRequest squashCommit: ((self
                          importCommitOfProject: aGLPHEMergeRequest project
                          withId: aGLPHEMergeRequest squash_commit_sha) ifNotNil: [ :commit |
                         foundCommits add: commit ]).


        self chainsCommitsFrom: foundCommits.
        ^ foundCommits
]

{ #category : 'api' }
GLPHModelImporter >> importDiffOfCommit: aCommit [

        super importDiffOfCommit: aCommit.
        
        "changes are added into the model during the import"
        aCommit diffs do: [ :diff | self importDiffRangesForDiff: diff ].

        ^ aCommit diffs
]

{ #category : 'as yet unclassified' }
GLPHModelImporter >> importDiffOfMergeRequest: aMergeRequest [

        | result diffsResult |
        aMergeRequest diffs ifNotEmpty: [
                'Diff of already importer: '
                , aMergeRequest iid printString recordInfo.
                ^ aMergeRequest diffs ].
        ('Import diff commits of MR ' , aMergeRequest iid printString)
                recordInfo.
        result := self glhApi
                          diffsMergeRequestOfProject: aMergeRequest project_id
                          withId: aMergeRequest iid.

        diffsResult := self newParseDiffResult: result.


        aMergeRequest diffs
                addAll: diffsResult
                unless: self blockForDiffEquality.
        self glhModel
                addAll: aMergeRequest diffs
                unless: self blockForDiffEquality.

        aMergeRequest diffs do: [ :diff | self importDiffRangesForDiff: diff ].

        ^ aMergeRequest diffs
]

{ #category : 'import' }
GLPHModelImporter >> importDiffRangesForDiff: aGLHDiff [

        | diffRanges |
        aGLHDiff diffRanges ifNotEmpty: [
                ('Diffs already imported for ' , aGLHDiff printString) recordInfo.
                ^ aGLHDiff diffRanges ].

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

        diffRanges := self glhModel
                              addAll: (self parseDiffString: aGLHDiff)
                              unless: self blockForDiffRangeEquality.
        ^ aGLHDiff diffRanges
                  addAll: diffRanges
                  unless: self blockForDiffRangeEquality
]

{ #category : 'import - merge request' }
GLPHModelImporter >> importMergeRequests: aGLHProject [

        | results parsedResults mrs|
        ('Import merge request of Project: ' , aGLHProject id printString)
                recordInfo.

        results := self glhApi mergeRequestOfProject: aGLHProject id.
        parsedResults := self parseMergeRequestResult: results.

        aGLHProject mergeRequests
                addAll: parsedResults
                unless: self blockOnIdEquality.

        mrs := self glhModel
                addAll: aGLHProject mergeRequests
                unless: self blockOnIdEquality.


        "gets it related commits"
        aGLHProject mergeRequests do: [ :mr |
                self importCommitsOfMergeResquest: mr ].


        self withCommitDiffs ifTrue: [
                aGLHProject mergeRequests do: [ :mr |
                        self importDiffOfMergeRequest: mr ] ].
        
        ^ mrs
]

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

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

        page := 0.
        foundMR := OrderedCollection new.
        newlyFoundMR := { true }.
        [ newlyFoundMR isNotEmpty ] whileTrue: [
                | results |
                page := page + 1.
                ('import MR page ' , page printString) recordInfo.
                results := self glhApi
                                   mergeRequestsOfProject: aGLHProject id
                                   createdAfter:
                                   (fromDate ifNotNil: [ fromDate asDateAndTime asString ])
                                   createdBefore:
                                   (toDate ifNotNil: [ toDate asDateAndTime asString ])
                                   authorId: nil
                                   authorUsername: nil
                                   scope: 'all'
                                   orderBy: 'created_at'
                                   page: page.


                newlyFoundMR := aGLHProject mergeRequests
                                        addAll: (self parseMergeRequestResult: results)
                                        unless: self blockOnIdEquality.
                foundMR addAll: newlyFoundMR ].


        "gets it related commits"
        aGLHProject mergeRequests do: [ :mr |
                self importCommitsOfMergeResquest: mr ].

        self withCommitDiffs ifTrue: [
                aGLHProject mergeRequests do: [ :mr |
                        self importDiffOfMergeRequest: mr ] ].

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

        ^ foundMR
]

{ #category : 'import - merge request' }
GLPHModelImporter >> importMergeResquestApprovals: aGLPHEMergeRequest [

        | results parsedResult |
        (String streamContents: [ :str |
                 str << 'Check approvals of '.
                 aGLPHEMergeRequest printOn: str ]) recordInfo.
        results := self glhApi
                           approvalsOfMergeQuest: aGLPHEMergeRequest iid
                           ofProject: aGLPHEMergeRequest project id.

        parsedResult := generalReader
                                on: results readStream;
                                next.

        (parsedResult at: #approved_by) do: [ :approvedUser |
                aGLPHEMergeRequest addApproved_by:
                        (self importUser: ((approvedUser at: #user) at: #id)) ].
        aGLPHEMergeRequest approved: (parsedResult at: #approved).
        ^ aGLPHEMergeRequest
]

{ #category : 'import - merge request' }
GLPHModelImporter >> importMergeResquestAuthor: aGLPHEMergeRequest [

        | authorID |
        aGLPHEMergeRequest author ifNotNil: [ ^ aGLPHEMergeRequest author ].
        authorID := aGLPHEMergeRequest cacheAt: #authorID ifAbsent: [
                            (generalReader
                                     on: (self glhApi
                                                      mergeRequestOfProject:
                                                              aGLPHEMergeRequest project_id
                                                      withId: aGLPHEMergeRequest iid) readStream;
                                     next) at: #author at: #id ].
        aGLPHEMergeRequest author: (self importUser: authorID)
]

{ #category : 'import - merge request' }
GLPHModelImporter >> importMergeResquestMerger: aGLPHEMergeRequest [

        | authorID |
        aGLPHEMergeRequest merge_user ifNotNil: [
                ^ aGLPHEMergeRequest merge_user ].
        authorID := aGLPHEMergeRequest cacheAt: #mergeUserID ifAbsent: [
                            (generalReader
                                     on: (self glhApi
                                                      mergeRequestOfProject:
                                                              aGLPHEMergeRequest project_id
                                                      withId: aGLPHEMergeRequest iid) readStream;
                                     next)
                                    at: #merge_user
                                    ifPresent: [ :mergeUser |
                                    mergeUser ifNotNil: [ :mruser | mruser at: #id ] ] ].
        aGLPHEMergeRequest merge_user: (self importUser: authorID)
]


{ #category : #'private - api' }
GLPHModelImporter >> importProject: aProjectID [

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

        result := self glhApi project: aProjectID.
        projectResult := self parseProjectResult: result.

        ^ self completeImportProject: projectResult
]

{ #category : 'private - api' }
GLPHModelImporter >> importRepository: aGLHRepository [

        super importRepository: aGLHRepository.

        withInitialCommits ifTrue: [
                aGLHRepository branches do: [ :branch |
                        self importCommitsOfBranch: branch ] ]
]

{ #category : 'initialization' }
GLPHModelImporter >> initReader [

        super initReader.
        self configureReaderForMergeRequest: generalReader
]

{ #category : 'initialization' }
GLPHModelImporter >> initialize [

        super initialize.
        withCommitDiffs := true
]

{ #category : 'parsing' }
GLPHModelImporter >> parseDiffString: aDiff [
        "parse diff string to create entities for each lines"

        | lines range index firstChangeRange |
        lines := aDiff diffString lines.
        index := 0.
        range := nil.

        lines ifEmpty: [ ^ aDiff diffRanges ].
        lines := self filterCommitChanges: lines.
        (lines first beginsWith: '@@') ifFalse: [ ^ aDiff diffRanges ].

        lines do: [ :line |
                | aChange |
                firstChangeRange := false.
                (line trim beginsWith: #'@@') ifTrue: [
                        range := GLPHEDiffRange newFrom: line.
                        firstChangeRange := true.
                        range := aDiff diffRanges add: range unless: [ :existing :new |
                                         existing originalLineRange = new originalLineRange and: [
                                                 existing newLineRange = new newLineRange ] ] ].

                self withCommitDiffs ifTrue: [
                        firstChangeRange ifFalse: [
                                aChange := GLPHEChange newFrom: line.
                                aChange relativeIndex: index.
                                aChange := range changes add: aChange unless: [ :existing :new |
                                                   existing sourceCode = new sourceCode and: [
                                                           existing index = new index ] ].
                                index := index + 1 ] ] ].

        "add changes to the diff range"
        aDiff diffRanges do: [ :diffRange |
                self glhModel addAll: diffRange changes unless: [ :existing :new |
                        existing diffRange mooseID = new diffRange mooseID and: [
                                existing sourceCode = new sourceCode and: [
                                        existing index = new index ] ] ] ].

        ^ aDiff diffRanges
]

{ #category : 'parsing' }
GLPHModelImporter >> parseMergeRequestResult: result [

        generalReader on: result readStream.
        ^ generalReader nextAs: #ArrayOfMergeRequest
]


{ #category : 'parsing' }
GLPHModelImporter >> parseProjectResult: aResult [ 
                | reader |
        reader := NeoJSONReader on: aResult readStream.
        reader for: GLHProject do: [ :mapping |
                mapping mapInstVars. ].
"        reader mapInstVarsFor: GLHProject."

        ^ reader nextAs: GLHProject
]

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

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

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

        (someDays isKindOf: DateAndTime)
                ifTrue: [ withCommitsSince := someDays ]
                ifFalse: [
                withCommitsSince := (Date today - someDays) asDateAndTime ]
]

1	Class {
2	#name : 'GLPHModelImporter',
3	#superclass : 'GLHModelImporter',
4	#category : 'GitLabProjectHealth-Model-Importer',
5	#package : 'GitLabProjectHealth-Model-Importer'
6	}
7
8	{ #category : 'as yet unclassified' }
9	GLPHModelImporter >> blockForDiffRangeEquality [	×
10		×
11	^ [ :existing :new \|	×
12	existing diff mooseID = new diff mooseID and: [	×
13	existing originalLineRange = new originalLineRange and: [	×
14	existing newLineRange = new newLineRange ] ] ]	×
UNCOV 15	]	×
16
17	{ #category : #commit }
18	GLPHModelImporter >> commitsOfProject: aGLHProject forRefName: refName until: toDate [	×
19		×
20	\| newlyFoundCommit page \|	×
21	page := 0.	×
22	newlyFoundCommit := { true }.	×
23	[ newlyFoundCommit isNotEmpty ] whileTrue: [	×
24	\| results parsedResults existingCommits \|	×
25	page := page + 1.	×
26	('import commit page ' , page printString) recordInfo.	×
27	results := self glhApi	×
28	commitsOfProject: aGLHProject id	×
29	forRefName: refName	×
30	since: nil	×
31	until: nil	×
32	path: nil	×
33	author: nil	×
34	all: nil	×
35	with_stats: nil	×
36	firstParent: nil	×
37	order: nil	×
38	trailers: nil	×
39	perPage: 100	×
40	page: page.	×
41	parsedResults := self parseCommitsResult: results.	×
42	existingCommits := aGLHProject mooseModel allWithType: GLHCommit.	×
43		×
44	newlyFoundCommit := parsedResults reject: [ :commitParsed \|	×
45	(toDate isNil or: [	×
46	commitParsed committed_date	×
47	< toDate asDateAndTime ]) or: [	×
48	existingCommits anySatisfy: [ :existingCommit \|	×
49	existingCommit id = commitParsed id ] ] ].	×
50	self glhModel addAll: newlyFoundCommit unless: self blockOnIdEquality.	×
51	aGLHProject repository commits addAll: newlyFoundCommit unless: self blockOnIdEquality ].	×
52	self withCommitDiffs ifTrue: [	×
53	aGLHProject repository commits do: [ :commit \|	×
54	self importDiffOfCommit: commit ] ]	×
UNCOV 55	]	×
56
57	{ #category : 'private - api' }
58	GLPHModelImporter >> completeImportProject: aGLHProject [	×
59	\|completedProject\|	×
60	completedProject := super completeImportProject: aGLHProject.	×
61		×
62	"for this initial import, with only look at some but not all MR"	×
63	withInitialMergeRequest ifTrue: [ self	×
64	importMergeRequests: completedProject	×
65	since: DateAndTime today	×
66	until: DateAndTime now. ].	×
67		×
68	^ completedProject	×
UNCOV 69	]	×
70
71	{ #category : 'private - configure reader' }
72	GLPHModelImporter >> configureReaderForMergeRequest: reader [	1✔
73	"declare quil y a un array a mapper"	1✔
74		1✔
75	reader for: #ArrayOfMergeRequest customDo: [ :customMappting \|	1✔
76	customMappting listOfElementSchema: GLPHEMergeRequest ].	1✔
77		1✔
78	"declare la liste des properties"	1✔
79	reader for: GLPHEMergeRequest do: [ :mapping \|	1✔
80	mapping mapInstVars:	1✔
81	#( blocking_discussions_resolved changes_count description	1✔
82	detailed_merge_status discussion_locked downvotes draft first_deployed_to_production_at	1✔
83	force_remove_source_branch has_conflicts id iid labels latest_build_finished_at	1✔
84	latest_build_started_at merge_commit_sha merge_status	1✔
85	merge_when_pipeline_succeeds merged_at milestone project_id	1✔
86	reference references_full references_relative	1✔
87	references_short sha should_remove_source_branch	1✔
88	source_branch source_project_id squash squash_commit_sha	1✔
89	squash_on_merge state subscribed target_branch target_project_id	1✔
90	task_completion_status_completed_count	1✔
91	task_completion_status_count time_stats_human_time_estimate	1✔
92	time_stats_human_total_time_spent	1✔
93	time_stats_time_estimate time_stats_total_time_spent	1✔
94	title updated_at upvotes user_notes_count web_url work_in_progress ).	1✔
95	(mapping mapInstVar: #created_at) valueSchema: DateAndTime.	1✔
96	(mapping mapInstVar: #updated_at) valueSchema: DateAndTime.	1✔
97	(mapping mapInstVar: #merged_at) valueSchema: DateAndTime.	1✔
98	(mapping mapInstVar: #closed_at) valueSchema: DateAndTime.	1✔
99	"(mapping mapInstVar: #assignee) valueSchema: GLHUser."	1✔
100	mapping	1✔
101	mapProperty: #author	1✔
102	getter: [ ]	1✔
103	setter: [ :object :value \|	1✔
104	object cacheAt: #authorID put: (value at: #id) ].	1✔
105	mapping	1✔
106	mapProperty: #merge_user	1✔
107	getter: [ ]	1✔
108	setter: [ :object :value \|	1✔
109	value ifNotNil: [	1✔
110	object cacheAt: #mergeUserID put: (value at: #id) ] ] ].	1✔
111		1✔
112	"(mapping mapInstVar: #closed_by) valueSchema: GLHUser.	1✔
113	(mapping mapInstVar: #mergeCommit) valueSchema: GLHCommit."	1✔
114	"indique ce que doit faire le reader lorsqu'il parse une DateAndTime object"	1✔
115	reader for: DateAndTime customDo: [ :mapping \|	1✔
116	mapping decoder: [ :string \|	1✔
117	string ifNil: [ nil ] ifNotNil: [ DateAndTime fromString: string ] ] ]	1✔
118	]	1✔
119
120	{ #category : 'import' }
121	GLPHModelImporter >> importCommitsOfBranch: aGLHBranch [	×
122		×
123	\| commits \|	×
124	" result := self glhApi	×
125	commitsOfProject: aGLHBranch repository project id	×
126	forRefName: aGLHBranch name."	×
127	commits := self	×
128	importCommitsOfBranch: aGLHBranch	×
129	forRefName: aGLHBranch name	×
130	since: withCommitsSince.	×
131		×
132	self chainsCommitsFrom: commits.	×
133		×
134	commits do: [ :aCommit \|	×
135	aCommit repository: aGLHBranch repository.	×
136	self completeImportedCommit: aCommit.	×
137	]	×
UNCOV 138	]	×
139
140	{ #category : 'import' }
141	GLPHModelImporter >> importCommitsOfMergeResquest: aGLPHEMergeRequest [	×
142		×
143	\| foundCommits \|	×
144	foundCommits := OrderedCollection new.	×
145		×
146	('Import commit sha of MR: ' , aGLPHEMergeRequest iid printString)	×
147	recordInfo.	×
148	"the founds commits are added to the model during their respective import"	×
149	aGLPHEMergeRequest mergeRequestCommit: ((self	×
150	importCommitOfProject: aGLPHEMergeRequest project	×
151	withId: aGLPHEMergeRequest sha) ifNotNil: [ :commit \|	×
152	foundCommits add: commit ]).	×
153		×
154	('Import commit merge_commit_sha of MR: '	×
155	, aGLPHEMergeRequest iid printString) recordInfo.	×
156	aGLPHEMergeRequest mergedCommit: ((self	×
157	importCommitOfProject: aGLPHEMergeRequest project	×
158	withId: aGLPHEMergeRequest merge_commit_sha) ifNotNil: [ :commit \|	×
159	foundCommits add: commit ]).	×
160		×
161	('Import commit squash_commit_sha of MR: '	×
162	, aGLPHEMergeRequest iid printString) recordInfo.	×
163	aGLPHEMergeRequest squashCommit: ((self	×
164	importCommitOfProject: aGLPHEMergeRequest project	×
165	withId: aGLPHEMergeRequest squash_commit_sha) ifNotNil: [ :commit \|	×
166	foundCommits add: commit ]).	×
167		×
168		×
169	self chainsCommitsFrom: foundCommits.	×
170	^ foundCommits	×
UNCOV 171	]	×
172
173	{ #category : 'api' }
174	GLPHModelImporter >> importDiffOfCommit: aCommit [	×
175		×
176	super importDiffOfCommit: aCommit.	×
177		×
178	"changes are added into the model during the import"	×
179	aCommit diffs do: [ :diff \| self importDiffRangesForDiff: diff ].	×
180		×
181	^ aCommit diffs	×
UNCOV 182	]	×
183
184	{ #category : 'as yet unclassified' }
185	GLPHModelImporter >> importDiffOfMergeRequest: aMergeRequest [	×
186		×
187	\| result diffsResult \|	×
188	aMergeRequest diffs ifNotEmpty: [	×
189	'Diff of already importer: '	×
190	, aMergeRequest iid printString recordInfo.	×
191	^ aMergeRequest diffs ].	×
192	('Import diff commits of MR ' , aMergeRequest iid printString)	×
193	recordInfo.	×
194	result := self glhApi	×
195	diffsMergeRequestOfProject: aMergeRequest project_id	×
196	withId: aMergeRequest iid.	×
197		×
198	diffsResult := self newParseDiffResult: result.	×
199		×
200		×
201	aMergeRequest diffs	×
202	addAll: diffsResult	×
203	unless: self blockForDiffEquality.	×
204	self glhModel	×
205	addAll: aMergeRequest diffs	×
206	unless: self blockForDiffEquality.	×
207		×
208	aMergeRequest diffs do: [ :diff \| self importDiffRangesForDiff: diff ].	×
209		×
210	^ aMergeRequest diffs	×
UNCOV 211	]	×
212
213	{ #category : 'import' }
214	GLPHModelImporter >> importDiffRangesForDiff: aGLHDiff [	×
215		×
216	\| diffRanges \|	×
217	aGLHDiff diffRanges ifNotEmpty: [	×
218	('Diffs already imported for ' , aGLHDiff printString) recordInfo.	×
219	^ aGLHDiff diffRanges ].	×
220		×
221	('Import changes of diff ' , aGLHDiff printString) recordInfo.	×
222	"the import of 'diffRanges' entites is technically done inside the parseDiffString"	×
223		×
224	diffRanges := self glhModel	×
225	addAll: (self parseDiffString: aGLHDiff)	×
226	unless: self blockForDiffRangeEquality.	×
227	^ aGLHDiff diffRanges	×
228	addAll: diffRanges	×
229	unless: self blockForDiffRangeEquality	×
UNCOV 230	]	×
231
232	{ #category : 'import - merge request' }
233	GLPHModelImporter >> importMergeRequests: aGLHProject [	×
234		×
235	\| results parsedResults mrs\|	×
236	('Import merge request of Project: ' , aGLHProject id printString)	×
237	recordInfo.	×
238		×
239	results := self glhApi mergeRequestOfProject: aGLHProject id.	×
240	parsedResults := self parseMergeRequestResult: results.	×
241		×
242	aGLHProject mergeRequests	×
243	addAll: parsedResults	×
244	unless: self blockOnIdEquality.	×
245		×
246	mrs := self glhModel	×
247	addAll: aGLHProject mergeRequests	×
248	unless: self blockOnIdEquality.	×
249		×
250		×
251	"gets it related commits"	×
252	aGLHProject mergeRequests do: [ :mr \|	×
253	self importCommitsOfMergeResquest: mr ].	×
254		×
255		×
256	self withCommitDiffs ifTrue: [	×
257	aGLHProject mergeRequests do: [ :mr \|	×
258	self importDiffOfMergeRequest: mr ] ].	×
259		×
260	^ mrs	×
UNCOV 261	]	×
262
263	{ #category : 'import - merge request' }
264	GLPHModelImporter >> importMergeRequests: aGLHProject since: fromDate until: toDate [	×
265		×
266	\| newlyFoundMR page foundMR \|	×
267	('import MR of Project ' , aGLHProject name) recordInfo.	×
268		×
269	page := 0.	×
270	foundMR := OrderedCollection new.	×
271	newlyFoundMR := { true }.	×
272	[ newlyFoundMR isNotEmpty ] whileTrue: [	×
273	\| results \|	×
274	page := page + 1.	×
275	('import MR page ' , page printString) recordInfo.	×
276	results := self glhApi	×
277	mergeRequestsOfProject: aGLHProject id	×
278	createdAfter:	×
279	(fromDate ifNotNil: [ fromDate asDateAndTime asString ])	×
280	createdBefore:	×
281	(toDate ifNotNil: [ toDate asDateAndTime asString ])	×
282	authorId: nil	×
283	authorUsername: nil	×
284	scope: 'all'	×
285	orderBy: 'created_at'	×
286	page: page.	×
287		×
288		×
289	newlyFoundMR := aGLHProject mergeRequests	×
290	addAll: (self parseMergeRequestResult: results)	×
291	unless: self blockOnIdEquality.	×
292	foundMR addAll: newlyFoundMR ].	×
293		×
294		×
295	"gets it related commits"	×
296	aGLHProject mergeRequests do: [ :mr \|	×
297	self importCommitsOfMergeResquest: mr ].	×
298		×
299	self withCommitDiffs ifTrue: [	×
300	aGLHProject mergeRequests do: [ :mr \|	×
301	self importDiffOfMergeRequest: mr ] ].	×
302		×
303	self glhModel addAll: foundMR unless: (self blockEqualityOn: #iid).	×
304		×
305	^ foundMR	×
UNCOV 306	]	×
307
308	{ #category : 'import - merge request' }
309	GLPHModelImporter >> importMergeResquestApprovals: aGLPHEMergeRequest [	×
310		×
311	\| results parsedResult \|	×
312	(String streamContents: [ :str \|	×
313	str << 'Check approvals of '.	×
314	aGLPHEMergeRequest printOn: str ]) recordInfo.	×
315	results := self glhApi	×
316	approvalsOfMergeQuest: aGLPHEMergeRequest iid	×
317	ofProject: aGLPHEMergeRequest project id.	×
318		×
319	parsedResult := generalReader	×
320	on: results readStream;	×
321	next.	×
322		×
323	(parsedResult at: #approved_by) do: [ :approvedUser \|	×
324	aGLPHEMergeRequest addApproved_by:	×
325	(self importUser: ((approvedUser at: #user) at: #id)) ].	×
326	aGLPHEMergeRequest approved: (parsedResult at: #approved).	×
327	^ aGLPHEMergeRequest	×
UNCOV 328	]	×
329
330	{ #category : 'import - merge request' }
331	GLPHModelImporter >> importMergeResquestAuthor: aGLPHEMergeRequest [	1✔
332		1✔
333	\| authorID \|	1✔
334	aGLPHEMergeRequest author ifNotNil: [ ^ aGLPHEMergeRequest author ].	1✔
335	authorID := aGLPHEMergeRequest cacheAt: #authorID ifAbsent: [	1✔
336	(generalReader	1✔
337	on: (self glhApi	1✔
338	mergeRequestOfProject:	1✔
339	aGLPHEMergeRequest project_id	1✔
340	withId: aGLPHEMergeRequest iid) readStream;	1✔
341	next) at: #author at: #id ].	1✔
342	aGLPHEMergeRequest author: (self importUser: authorID)	1✔
343	]	1✔
344
345	{ #category : 'import - merge request' }
346	GLPHModelImporter >> importMergeResquestMerger: aGLPHEMergeRequest [	1✔
347		1✔
348	\| authorID \|	1✔
349	aGLPHEMergeRequest merge_user ifNotNil: [	1✔
350	^ aGLPHEMergeRequest merge_user ].	1✔
351	authorID := aGLPHEMergeRequest cacheAt: #mergeUserID ifAbsent: [	1✔
352	(generalReader	1✔
353	on: (self glhApi	1✔
354	mergeRequestOfProject:	1✔
355	aGLPHEMergeRequest project_id	1✔
356	withId: aGLPHEMergeRequest iid) readStream;	1✔
357	next)	1✔
358	at: #merge_user	1✔
359	ifPresent: [ :mergeUser \|	1✔
360	mergeUser ifNotNil: [ :mruser \| mruser at: #id ] ] ].	1✔
361	aGLPHEMergeRequest merge_user: (self importUser: authorID)	1✔
362	]	1✔
363
364
365	{ #category : #'private - api' }
366	GLPHModelImporter >> importProject: aProjectID [	×
367		×
368	\| result projectResult \|	×
369	('Import project with id: ' , aProjectID printString) recordInfo.	×
370		×
371	result := self glhApi project: aProjectID.	×
372	projectResult := self parseProjectResult: result.	×
373		×
UNCOV 374	^ self completeImportProject: projectResult	×
NEW 375	]	×
376
377	{ #category : 'private - api' }
378	GLPHModelImporter >> importRepository: aGLHRepository [	×
379		×
380	super importRepository: aGLHRepository.	×
381		×
382	withInitialCommits ifTrue: [	×
383	aGLHRepository branches do: [ :branch \|	×
UNCOV 384	self importCommitsOfBranch: branch ] ]	×
NEW 385	]	×
386
387	{ #category : 'initialization' }
388	GLPHModelImporter >> initReader [	1✔
389		1✔
390	super initReader.	1✔
391	self configureReaderForMergeRequest: generalReader	1✔
392	]	1✔
393
394	{ #category : 'initialization' }
395	GLPHModelImporter >> initialize [	1✔
396		1✔
397	super initialize.	1✔
398	withCommitDiffs := true	1✔
399	]	1✔
400
401	{ #category : 'parsing' }
402	GLPHModelImporter >> parseDiffString: aDiff [	1✔
403	"parse diff string to create entities for each lines"	1✔
404		1✔
405	\| lines range index firstChangeRange \|	1✔
406	lines := aDiff diffString lines.	1✔
407	index := 0.	1✔
408	range := nil.	1✔
409		1✔
410	lines ifEmpty: [ ^ aDiff diffRanges ].	1✔
411	lines := self filterCommitChanges: lines.	1✔
412	(lines first beginsWith: '@@') ifFalse: [ ^ aDiff diffRanges ].	1✔
413		1✔
414	lines do: [ :line \|	1✔
415	\| aChange \|	1✔
416	firstChangeRange := false.	1✔
417	(line trim beginsWith: #'@@') ifTrue: [	1✔
418	range := GLPHEDiffRange newFrom: line.	1✔
419	firstChangeRange := true.	1✔
420	range := aDiff diffRanges add: range unless: [ :existing :new \|	1✔
421	existing originalLineRange = new originalLineRange and: [	1✔
422	existing newLineRange = new newLineRange ] ] ].	1✔
423		1✔
424	self withCommitDiffs ifTrue: [	1✔
425	firstChangeRange ifFalse: [	1✔
426	aChange := GLPHEChange newFrom: line.	1✔
427	aChange relativeIndex: index.	1✔
428	aChange := range changes add: aChange unless: [ :existing :new \|	1✔
429	existing sourceCode = new sourceCode and: [	1✔
430	existing index = new index ] ].	1✔
431	index := index + 1 ] ] ].	1✔
432		1✔
433	"add changes to the diff range"	1✔
434	aDiff diffRanges do: [ :diffRange \|	1✔
435	self glhModel addAll: diffRange changes unless: [ :existing :new \|	1✔
436	existing diffRange mooseID = new diffRange mooseID and: [	1✔
437	existing sourceCode = new sourceCode and: [	1✔
438	existing index = new index ] ] ] ].	1✔
439		1✔
440	^ aDiff diffRanges	1✔
441	]	1✔
442
443	{ #category : 'parsing' }
444	GLPHModelImporter >> parseMergeRequestResult: result [	1✔
445		1✔
446	generalReader on: result readStream.	1✔
447	^ generalReader nextAs: #ArrayOfMergeRequest	1✔
448	]	1✔
449
450
451	{ #category : 'parsing' }
452	GLPHModelImporter >> parseProjectResult: aResult [	×
453	\| reader \|	×
454	reader := NeoJSONReader on: aResult readStream.	×
455	reader for: GLHProject do: [ :mapping \|	×
456	mapping mapInstVars. ].	×
457	" reader mapInstVarsFor: GLHProject."	×
458		×
NEW 459	^ reader nextAs: GLHProject	×
460	]	×
461		×
462	{ #category : 'accessing' }	×
463	GLPHModelImporter >> withCommitsSince: someDays [	×
464	"substract the current date with a given number of days. Use to retrieve the commits submit in the last giving days"	×
465		×
466	"(GLPHModelImporter new withCommitsSince: 1 week ) >>> (Date today - 1 week) asDateAndTime"	×
467		×
468	"(GLPHModelImporter new withCommitsSince: 30 day ) >>> (Date today - 30 day) asDateAndTime"	×
469		×
470	(someDays isKindOf: DateAndTime)	×
471	ifTrue: [ withCommitsSince := someDays ]	×
UNCOV 472	ifFalse: [	×
UNCOV 473	withCommitsSince := (Date today - someDays) asDateAndTime ]	×
UNCOV 474	]	×

moosetechnology / GitProjectHealth / 10880334038

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

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