TestCase >> addDependentToHierachy: seems obsolete

pharo-project / pharo

Pharo is a dynamic reflective pure object-oriented language supporting live programming inspired by Smalltalk.

http://pharo.org

Other

1.22k stars 358 forks source link

TestCase >> addDependentToHierachy: seems obsolete #11124

Open aboubacardiawara opened 2 years ago

aboubacardiawara commented 2 years ago

TestCase >> addDependentToHierachy: anObject 
    "an empty method. for Composite compatibility with TestSuite"

Does nothing and it is only used here:

TestSuite >> addDependentToHierachy: anObject
    self addDependent: anObject.
    self tests do: [ :each | each addDependentToHierachy: anObject]

is It possible :

to remove the empty method
and the line where it is used ?

welcome[bot] commented 2 years ago

Thanks for opening your first issue! Please check the CONTRIBUTING documents for some tips about which information should be provided. You can find information of how to do a Pull Request here: https://github.com/pharo-project/pharo/wiki/Contribute-a-fix-to-Pharo

GitHub
Contribute a fix to Pharo · pharo-project/pharo Wiki
Pharo is a dynamic reflective pure object-oriented language supporting live programming inspired by Smalltalk. - Contribute a fix to Pharo · pharo-project/pharo Wiki