Merge pull request #64 from moosetechnology/feat-number-of-commit-by-…

…tickets

feat: add commitsNumberByJiraTicket metric

src/GitLabHealth-Model-Analysis-Tests/CommitsNumberByJiraTicketsMetricTest.class.st

@@ -0,0 +1,120 @@
+"
+A CommitsNumberByJiraTicketsMetricTest is a test class for testing the behavior of CommitsNumberByJiraTicketsMetric
+"
+Class {
+	#name : #CommitsNumberByJiraTicketsMetricTest,
+	#superclass : #TestCase,
+	#category : #'GitLabHealth-Model-Analysis-Tests'
+}
+
+{ #category : #tests }
+CommitsNumberByJiraTicketsMetricTest >> testCalculate [
+
+	| result glhImporter user jiraImporter commitsNumber |
+	"Given"
+	user := GLHUser new
+		        username: 'test';
+		        contributedProjects:
+			        { (GLHProject new repository: GLHRepository new) }.
+
+	glhImporter := GLPHImporterMock new.
+	glhImporter mergeRequests: { (GLPHEMergeRequest new
+			 author: user;
+			 created_at: '09-05-2024';
+			 merged_at: '09-06-2024' asDate;
+			 title: '205 feat do something') }.
+
+	glhImporter commits: { (GLHCommit new
+			 id: 1;
+			 created_at: '09-05-2024' asDate;
+			 commitCreator: user;
+			 deletions: 5) }.
+
+	jiraImporter := JiraImporterMock new.
+	jiraImporter issues: { (JPIssue new
+			 key: '205';
+			 timeEstimate: 25 hours asDuration) }.
+
+
+	commitsNumber := CommitsNumberByJiraTicketsMetric new
+		                 user: user;
+		                 glhImporter: glhImporter;
+		                 jiraImporter: jiraImporter;
+		                 setPeriodSince: '09-04-2024' until: '09-04-2024';
+		                 over: Week.
+
+	"When"
+	result := commitsNumber calculate.
+
+	"Then"
+	self assert: result equals: 1
+]
+
+{ #category : #tests }
+CommitsNumberByJiraTicketsMetricTest >> testCalculateNoCommits [
+
+	| result glhImporter user jiraImporter commitsNumber |
+	"Given"
+	user := GLHUser new
+		        username: 'test';
+		        contributedProjects:
+			        { (GLHProject new repository: GLHRepository new) }.
+
+	glhImporter := GLPHImporterMock new.
+	glhImporter mergeRequests: { (GLPHEMergeRequest new
+			 author: user;
+			 created_at: '09-05-2024';
+			 merged_at: '09-06-2024' asDate;
+			 title: '205 feat do something') }.
+
+	jiraImporter := JiraImporterMock new.
+	jiraImporter issues: { (JPIssue new
+			 key: '205';
+			 timeEstimate: 25 hours asDuration) }.
+
+
+	commitsNumber := CommitsNumberByJiraTicketsMetric new
+		                 user: user;
+		                 glhImporter: glhImporter;
+		                 jiraImporter: jiraImporter;
+		                 setPeriodSince: '09-04-2024' until: '09-04-2024';
+		                 over: Week.
+
+	"When"
+	result := commitsNumber calculate.
+
+	"Then"
+	self assert: result equals: 0
+]
+
+{ #category : #tests }
+CommitsNumberByJiraTicketsMetricTest >> testCalculateNoMergeRequests [
+
+	| result glhImporter user jiraImporter commitsNumber |
+	"Given"
+	user := GLHUser new
+		        username: 'test';
+		        contributedProjects:
+			        { (GLHProject new repository: GLHRepository new) }.
+
+	glhImporter := GLPHImporterMock new.
+
+	jiraImporter := JiraImporterMock new.
+	jiraImporter issues: { (JPIssue new
+			 key: '205';
+			 timeEstimate: 25 hours asDuration) }.
+
+
+	commitsNumber := CommitsNumberByJiraTicketsMetric new
+		                 user: user;
+		                 glhImporter: glhImporter;
+		                 jiraImporter: jiraImporter;
+		                 setPeriodSince: '09-04-2024' until: '09-04-2024';
+		                 over: Week.
+
+	"When"
+	result := commitsNumber calculate.
+
+	"Then"
+	self assert: result equals: 0
+]

src/GitLabHealth-Model-Analysis/CommitsNumberByJiraTicketsMetric.class.st

@@ -0,0 +1,39 @@
+Class {
+	#name : #CommitsNumberByJiraTicketsMetric,
+	#superclass : #UserJiraMetric,
+	#category : #'GitLabHealth-Model-Analysis'
+}
+
+{ #category : #calculating }
+CommitsNumberByJiraTicketsMetric >> calculate [
+
+	| groupedByDate dateOver |
+	userMergeRequests ifNil: [ self load ].
+	userMergeRequests ifEmpty: [ ^ 0 ].
+	groupedByDate := self setupGroupedDate.
+	userMergeRequests do: [ :userMergeRequest |
+		dateOver := self transformDate: userMergeRequest created_at to: over.
+		groupedByDate
+			at: dateOver printString
+			ifPresent: [ :value | value add: userMergeRequest ] ].
+	groupedByDate := groupedByDate collect: [ :group |
+		                 | average |
+		                 average := (group collect: [ :mergeRequest |
+			                             mergeRequest commits size ]) average ].
+	^ groupedByDate average
+]
+
+{ #category : #accessing }
+CommitsNumberByJiraTicketsMetric >> description [
+
+	^ 'average number of commit by merge request associated to a jira ticket'
+]
+
+{ #category : #loading }
+CommitsNumberByJiraTicketsMetric >> load [
+
+	userMergeRequests := self
+		                     loadUserCompleteMergeRequestsWithJiraIssueSince:
+		                     (period at: #since)
+		                     until: (period at: #until).
+]

src/GitLabHealth-Model-Analysis/GitMetricExporter.class.st

@@ -454,7 +454,8 @@ GitMetricExporter >> initialize [
 		           ClosedMergeRequestMetric.
 		           MergedMergeRequestMetric.
 		           SelfMergedUnderAMinuteMergeRequestMetric.
-		           ClosedTicketsMetric. JiraTimeMRTimeDifferenceMetric }
+		           ClosedTicketsMetric.
+		           JiraTimeMRTimeDifferenceMetric. CommitsNumberByJiraTicketsMetric }
 ]
 
 { #category : #accessing }
@@ -473,6 +474,12 @@ GitMetricExporter >> maxCommitWindow: anInteger [
 	maxCommitWindow := anInteger abs
 ]
 
+{ #category : #accessing }
+GitMetricExporter >> metrics: anObject [
+
+	metrics := anObject
+]
+
 { #category : #adding }
 GitMetricExporter >> onlyImportProjectsOfGroup: groupId [