Fix #testActivateActionButtons for MiDuplicationBrowser.

This browser needs some improvements.

src/MooseIDE-Duplication/MiDuplicationBrowser.class.st

@@ -192,7 +192,8 @@ MiDuplicationBrowser >> list: collection1 hasSameElements: collection2 [
 
 { #category : #'accessing - private tests' }
 MiDuplicationBrowser >> lstEntities [
-	^lstEntities 
+
+	^ lstEntities
 ]
 
 { #category : #'accessing - private tests' }
@@ -204,7 +205,7 @@ MiDuplicationBrowser >> lstReplicatedFragments [
 { #category : #accessing }
 MiDuplicationBrowser >> miSelectedItem [
 
-	^ specModel selectedEntities
+	^ specModel miSelectedItem
 ]
 
 { #category : #utilities }

src/MooseIDE-Duplication/MiDuplicationBrowserModel.class.st

@@ -126,7 +126,8 @@ MiDuplicationBrowserModel >> initializeSettings [
 
 { #category : #api }
 MiDuplicationBrowserModel >> isSelected: anEntity [
-	^selectedEntities includes: anEntity 
+
+	^ selectedEntities includes: anEntity
 ]
 
 { #category : #settings }
@@ -183,7 +184,7 @@ MiDuplicationBrowserModel >> maxReplicasShowed [
 { #category : #accessing }
 MiDuplicationBrowserModel >> miSelectedItem [
 
-	^selectedEntities
+	^ selectedEntities ifEmpty: [ entities ]
 ]
 
 { #category : #settings }
@@ -226,17 +227,21 @@ MiDuplicationBrowserModel >> numberOfEntities [
 
 { #category : #api }
 MiDuplicationBrowserModel >> numberOfEntitiesInReplicas [
-	^(replicatedFragmentsHierarchy keys flatCollectAsSet: [ :fragment | fragment replicas collect: #entity ]) size
+
+	^ (replicatedFragmentsHierarchy keys flatCollectAsSet: [ :fragment |
+		   fragment replicas collect: #entity ]) size
 ]
 
 { #category : #api }
 MiDuplicationBrowserModel >> numberOfFragments [
-	^replicatedFragmentsHierarchy size
+
+	^ replicatedFragmentsHierarchy size
 ]
 
 { #category : #api }
 MiDuplicationBrowserModel >> numberOfSelectedEntities [
-	^selectedEntities size
+
+	^ selectedEntities size
 ]
 
 { #category : #api }
@@ -263,23 +268,25 @@ MiDuplicationBrowserModel >> replicatedFragmentsHierarchy: aCollection [
 
 { #category : #api }
 MiDuplicationBrowserModel >> rootFragments [
+
 	| children |
-	children := replicatedFragmentsHierarchy associations flatCollect: #value.
-	^replicatedFragmentsHierarchy keys reject: [ :fragment | children includes: fragment]
-
+	children := replicatedFragmentsHierarchy associations flatCollect:
+		            #value.
+	^ replicatedFragmentsHierarchy keys reject: [ :fragment |
+		  children includes: fragment ]
 ]
 
 { #category : #accessing }
-MiDuplicationBrowserModel >> selectedEntities [
+MiDuplicationBrowserModel >> selectEntities: entityCollection [
 
-	^selectedEntities
+	selectedEntities := entityCollection.
+	browser updateToolbar
 ]
 
 { #category : #accessing }
 MiDuplicationBrowserModel >> selectedEntities: entityCollection [
 
 	selectedEntities := entityCollection.
-	"browser highlightEntity: selectedEntities."
 	browser updateToolbar
 ]
 
@@ -292,7 +299,7 @@ MiDuplicationBrowserModel >> selectedFragments: selectedFragments [
 		ifTrue: [ allReplicas first: (settings getItemValue: self maxReplicasShowed) ]
 		ifFalse: [ allReplicas ].
 
-	self selectedEntities:
+	self selectEntities:
 		(allReplicas collectAsSet: #entity) asMooseGroup.
 	browser showReplicas: allReplicas
 ]

src/MooseIDE-Tests/MiDuplicationBrowserModelTest.class.st

@@ -209,12 +209,16 @@ MiDuplicationBrowserModelTest >> testRootFragments [
 
 { #category : #tests }
 MiDuplicationBrowserModelTest >> testSelectedFragment [
-	model entities: { method1 . method2 . method3 }.
-	self assertEmpty: model selectedEntities.
 
-	model selectedFragments: {fragment2}.
+	model entities: {
+			method1.
+			method2.
+			method3 }.
+	self assertEmpty: model miSelectedItem.
+
+	model selectedFragments: { fragment2 }.
 	self assert: model selectedEntities size equals: 2.
 	self assert: (model isSelected: method1).
 	self assert: (model isSelected: method2).
-	self deny: (model isSelected: method3).
+	self deny: (model isSelected: method3)
 ]

src/MooseIDE-Tests/MiDuplicationBrowserTest.class.st

@@ -7,7 +7,14 @@ Class {
 { #category : #running }
 MiDuplicationBrowserTest >> browserClass [
 
-	^MiDuplicationBrowser 
+	^ MiDuplicationBrowser
+]
+
+{ #category : #running }
+MiDuplicationBrowserTest >> initializeEntityToSelect [
+
+	entityToSelect := (self method: 'method1' withSource: (1 to: 8))
+		                  asMooseGroup
 ]
 
 { #category : #running }
@@ -88,12 +95,6 @@ MiDuplicationBrowserTest >> sourceCode: lineInterval [
 				    to: lineInterval last)
 ]
 
-{ #category : #tests }
-MiDuplicationBrowserTest >> testActivateActionButtons [
-
-	browser actionButtons do: [ :button | self deny: button isEnabled ].
-]
-
 { #category : #'tests - actions' }
 MiDuplicationBrowserTest >> testFollowEntity [
 
@@ -118,16 +119,16 @@ MiDuplicationBrowserTest >> testFormatReplicatedFragment [
 MiDuplicationBrowserTest >> testMiSelectedItem [
 
 	| entities |
-	entities := { 
+	entities := {
 		            (self method: 'method1').
 		            (self method: 'method2') } asMooseGroup.
 
 	browser model entities: entities.
 
-	self assertEmpty: browser miSelectedItem.
-	self deny: browser canPropagate.
+	self denyEmpty: browser miSelectedItem.
+	self assert: browser canPropagate.
 
-	browser model selectedEntities: entities.
+	browser model selectEntities: entities.
 	self assert: browser miSelectedItem size equals: 2.
 	self assert: browser canPropagate
 ]