Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove old Finder references in Baselines #17421

Merged
merged 2 commits into from
Nov 22, 2024

Conversation

hernanmd
Copy link
Member

@hernanmd hernanmd commented Nov 21, 2024

Continuation of #17420
This PR needs pharo-spec/NewTools#909 to be integrated first

@hernanmd hernanmd self-assigned this Nov 21, 2024
@hernanmd hernanmd added the Priority: Critical To fix or review as soon as possible label Nov 21, 2024
@hernanmd hernanmd requested review from Ducasse and jecisc November 21, 2024 09:56
@Ducasse
Copy link
Member

Ducasse commented Nov 21, 2024

Strange it does not build.

@hernanmd
Copy link
Member Author

Strange it does not build.

I'm checking now why the Finder examples are broken

✗ #testConcatenationSearch (145ms)
✗ #testIndexOfSearch (144ms)
✗ #testNegatedSearch (141ms)
✗ #testFactorialSearch (137ms)

@jecisc
Copy link
Member

jecisc commented Nov 21, 2024

Error: Package NewTools-Finder depends on the following classes:
  MethodFinder
  MethodFinderSend
You must resolve these dependencies before you will be able to load these definitions: 
  StMethodFinder
  StMethodFinder>>#findMethodsByExampleInput:andExpectedResult:timeout:
  StMethodFinder>>#methodFinderSendClass
  StMethodFinderSend
  StMethodFinderSend>>#=
  StMethodFinderSend>>#children
  StMethodFinderSend>>#copyTo:
  StMethodFinderSend>>#debug
  StMethodFinderSend>>#displayIcon
  StMethodFinderSend>>#evaluateWithTimeOut:
  StMethodFinderSend>>#forFinderPreview:
  StMethodFinderSend>>#handleError:
  StMethodFinderSend>>#hasBrowseAction
  StMethodFinderSend>>#hasChildren

@jecisc jecisc added the Status: Need more work The issue is nearly ready. Waiting some last bits. label Nov 21, 2024
@hernanmd
Copy link
Member Author

Error: Package NewTools-Finder depends on the following classes:
  MethodFinder
  MethodFinderSend
You must resolve these dependencies before you will be able to load these definitions: 
  StMethodFinder
  StMethodFinder>>#findMethodsByExampleInput:andExpectedResult:timeout:
  StMethodFinder>>#methodFinderSendClass
  StMethodFinderSend
  StMethodFinderSend>>#=
  StMethodFinderSend>>#children
  StMethodFinderSend>>#copyTo:
  StMethodFinderSend>>#debug
  StMethodFinderSend>>#displayIcon
  StMethodFinderSend>>#evaluateWithTimeOut:
  StMethodFinderSend>>#forFinderPreview:
  StMethodFinderSend>>#handleError:
  StMethodFinderSend>>#hasBrowseAction
  StMethodFinderSend>>#hasChildren

This needs pharo-spec/NewTools#909 to be integrated but the CI seems to be hanging now

@Ducasse Ducasse closed this Nov 21, 2024
@Ducasse Ducasse reopened this Nov 21, 2024
@Ducasse
Copy link
Member

Ducasse commented Nov 21, 2024

It does not build now :( I hope that the previous PR I integrated did not break something but the build was green.

@Ducasse
Copy link
Member

Ducasse commented Nov 21, 2024

MetacelloNotification: Loaded -> NewTools-Scopes-Resources-C-Tests --- github://pharo-spec/NewTools:Pharo13/src(29c27a3)
�[31mError: Package NewTools-Finder depends on the following classes:
MethodFinder
MethodFinderSend
You must resolve these dependencies before you will be able to load these definitions:
StMethodFinder
StMethodFinder>>#findMethodsByExampleInput:andExpectedResult:timeout:
StMethodFinder>>#methodFinderSendClass
StMethodFinderSend
StMethodFinderSend>>#=
StMethodFinderSend>>#children
StMethodFinderSend>>#copyTo:
StMethodFinderSend>>#debug
StMethodFinderSend>>#displayIcon
StMethodFinderSend>>#evaluateWithTimeOut:
StMethodFinderSend>>#forFinderPreview:
StMethodFinderSend>>#handleError:
StMethodFinderSend>>#hasBrowseAction
StMethodFinderSend>>#hasChildren
StMethodFinderSend>>#hasHierarchyAction
StMethodFinderSend>>#hasImplementersAction
StMethodFinderSend>>#hasInheritanceAction
StMethodFinderSend>>#hasSendersAction
StMethodFinderSend>>#hasVersionsAction
StMethodFinderSend>>#hash
StMethodFinderSend>>#implementersAction
StMethodFinderSend>>#isClassResult
StMethodFinderSend>>#isSelectorResult
StMethodFinderSend>>#logInfo:
StMethodFinderSend>>#matches:
StMethodFinderSend>>#previewText
StMethodFinderSend>>#profile
StMethodFinderSend>>#resultIn:timeout:
StMethodFinderSend>>#selectorForFinder
StMethodFinderSend>>#sendersAction

�[0mMCPackageLoader(Object)>>notify:
MCPackageLoader>>warnAboutDependencies
MCPackageLoader>>validate
MCPackageLoader>>load
MCVersionLoader>>basicLoad
[ self basicLoad ] in MCVersionLoader>>loadWithNameLike: in Block: [ self basicLoad ]
[ aBlock value ] in MCVersionLoader>>announceLoad:do: in Block: [ aBlock value ]
FullBlockClosure(BlockClosure)>>ensure:
MCVersionLoader>>announceLoad:do:
MCVersionLoader>>loadWithNameLike:
[
self ensurePackage: version package.
self loadWithNameLike: version info name ] in MCVersionLoader>>load in Block: [...
FullBlockClosure(BlockClosure)>>cull:
[ ^ block cull: self ] in [ self prepareForRunning.
CurrentJob
value: self
during: [ ^ block cull: self ] ] in Job>>run in Block: [ ^ block cull: self ]
[ activeProcess
psValueAt: index
put: anObject.
aBlock value ] in CurrentJob(DynamicVariable)>>value:during: in Block: [ activeProcess...
FullBlockClosure(BlockClosure)>>ensure:
CurrentJob(DynamicVariable)>>value:during:
CurrentJob class(DynamicVariable class)>>value:during:
[ self prepareForRunning.
CurrentJob
value: self
during: [ ^ block cull: self ] ] in Job>>run in Block: [ self prepareForRunning....
FullBlockClosure(BlockClosure)>>ensure:
Job>>run
MCVersionLoader>>load
MetacelloMonticelloLoader>>loadAtomicPackageDirectives:
MetacelloMonticelloLoader>>loadPackageDirective:
MetacelloLoadTarget>>visitPackageLoadDirective:
MetacelloPackageLoadDirective>>acceptVisitor:
[ :dir |
dir acceptVisitor: self ] in MetacelloLoadTarget>>visitLinearLoadDirective: in Block: [ :dir |...

@hernanmd
Copy link
Member Author

hernanmd commented Nov 21, 2024

MetacelloNotification: Loaded -> NewTools-Scopes-Resources-C-Tests --- github://pharo-spec/NewTools:Pharo13/src(29c27a3) �[31mError: Package NewTools-Finder depends on the following classes: MethodFinder MethodFinderSend You must resolve these dependencies before you will be able to load these definitions: StMethodFinder StMethodFinder>>#findMethodsByExampleInput:andExpectedResult:timeout: StMethodFinder>>#methodFinderSendClass StMethodFinderSend StMethodFinderSend>>#= StMethodFinderSend>>#children StMethodFinderSend>>#copyTo: StMethodFinderSend>>#debug StMethodFinderSend>>#displayIcon StMethodFinderSend>>#evaluateWithTimeOut: StMethodFinderSend>>#forFinderPreview: StMethodFinderSend>>#handleError: StMethodFinderSend>>#hasBrowseAction StMethodFinderSend>>#hasChildren StMethodFinderSend>>#hasHierarchyAction StMethodFinderSend>>#hasImplementersAction StMethodFinderSend>>#hasInheritanceAction StMethodFinderSend>>#hasSendersAction StMethodFinderSend>>#hasVersionsAction StMethodFinderSend>>#hash StMethodFinderSend>>#implementersAction StMethodFinderSend>>#isClassResult StMethodFinderSend>>#isSelectorResult StMethodFinderSend>>#logInfo: StMethodFinderSend>>#matches: StMethodFinderSend>>#previewText StMethodFinderSend>>#profile StMethodFinderSend>>#resultIn:timeout: StMethodFinderSend>>#selectorForFinder StMethodFinderSend>>#sendersAction

�[0mMCPackageLoader(Object)>>notify: MCPackageLoader>>warnAboutDependencies MCPackageLoader>>validate MCPackageLoader>>load MCVersionLoader>>basicLoad [ self basicLoad ] in MCVersionLoader>>loadWithNameLike: in Block: [ self basicLoad ] [ aBlock value ] in MCVersionLoader>>announceLoad:do: in Block: [ aBlock value ] FullBlockClosure(BlockClosure)>>ensure: MCVersionLoader>>announceLoad:do: MCVersionLoader>>loadWithNameLike: [ self ensurePackage: version package. self loadWithNameLike: version info name ] in MCVersionLoader>>load in Block: [... FullBlockClosure(BlockClosure)>>cull: [ ^ block cull: self ] in [ self prepareForRunning. CurrentJob value: self during: [ ^ block cull: self ] ] in Job>>run in Block: [ ^ block cull: self ] [ activeProcess psValueAt: index put: anObject. aBlock value ] in CurrentJob(DynamicVariable)>>value:during: in Block: [ activeProcess... FullBlockClosure(BlockClosure)>>ensure: CurrentJob(DynamicVariable)>>value:during: CurrentJob class(DynamicVariable class)>>value:during: [ self prepareForRunning. CurrentJob value: self during: [ ^ block cull: self ] ] in Job>>run in Block: [ self prepareForRunning.... FullBlockClosure(BlockClosure)>>ensure: Job>>run MCVersionLoader>>load MetacelloMonticelloLoader>>loadAtomicPackageDirectives: MetacelloMonticelloLoader>>loadPackageDirective: MetacelloLoadTarget>>visitPackageLoadDirective: MetacelloPackageLoadDirective>>acceptVisitor: [ :dir | dir acceptVisitor: self ] in MetacelloLoadTarget>>visitLinearLoadDirective: in Block: [ :dir |...

This is weird because I explicitly defined Message as superclass of StMethodFinderSend here https://github.com/hernanmd/NewTools/blob/d0828c62dc20d5e328306daeeb27d96c7f684d70/src/NewTools-Finder/StMethodFinderSend.class.st#L10

And Object as the superclass of StMethodFinder here https://github.com/hernanmd/NewTools/blob/d0828c62dc20d5e328306daeeb27d96c7f684d70/src/NewTools-Finder/StMethodFinder.class.st#L10

So the build is not using the latest NewTools PR I think

GitHub
All development tools for Pharo, developed with Spec - hernanmd/NewTools
GitHub
All development tools for Pharo, developed with Spec - hernanmd/NewTools

@hernanmd hernanmd closed this Nov 22, 2024
@hernanmd hernanmd reopened this Nov 22, 2024
@jecisc jecisc merged commit 303b9b4 into pharo-project:Pharo13 Nov 22, 2024
1 of 2 checks passed
@hernanmd hernanmd deleted the p13-remove-new-finder-refs branch November 29, 2024 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Critical To fix or review as soon as possible Status: Need more work The issue is nearly ready. Waiting some last bits.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants