-
Notifications
You must be signed in to change notification settings - Fork 31
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
815 reference checker should handled the input files the same for the check dir than the check project #875
Conversation
I have the impression that a test is missing showing that if we have
We treat ABC and not ACBC |
@@ -161,7 +161,7 @@ MicFileCollectorTest >> testOnlyGoUpInFoldersStartingFromAnotherRoot [ | |||
|
|||
{ #category : 'tests' } | |||
MicFileCollectorTest >> testUnreferencedFileAreNotHandled [ | |||
"This test verifies that the collector only collects files that are referenced from a root and not all the files in a folder." | |||
"This test verifies that the collector only collects files that are referenced from a root and not all the files in a folder. " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove tab in the comment please
{ #category : 'visiting' } | ||
MicFileCollector >> add: aFile [ | ||
|
||
worklist add: aFile . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No space before the .
{ #category : 'visiting' } | ||
MicFileCollector >> workList [ | ||
|
||
^ worklist . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no Space before the .
|
||
collector | ||
rootDirectory: FileSystem memory ; | ||
visit: collector workList first . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no space before the .
@@ -37,6 +37,7 @@ MicResult class >> headerString [ | |||
|
|||
{ #category : 'accessing' } | |||
MicResult >> explanation [ | |||
"just test " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove it.
|
||
collector | ||
rootDirectory: FileSystem memory ; | ||
visit: collector workList first . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do not get why we only start the first worklist and why it is not removed from the worklist and why
we do not handled the next ones?
@@ -794,8 +792,8 @@ MicReferenceCheckerTest >> testReportingUnknownAnchorDir [ | |||
file2 ensureCreateFile. | |||
|
|||
visitor := MicReferenceChecker new. | |||
|
|||
self deny: (visitor checkDirectory: dir). | |||
visitor checkDirectory: dir . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why this change?
@@ -342,6 +353,18 @@ MicFileTest >> section4InputsParentSection3 [ | |||
' ] | |||
] | |||
|
|||
{ #category : 'helpers - only inputfiles' } | |||
MicFileTest >> section4InputsParentSection3withUknownAnchors [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do not see where this is used.
fix checkDirectory