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.

Sign up for GitHub

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

Jump to bottom

classNamedOrNil test is bogus #740

Closed
seandenigris opened this issue Aug 3, 2015 · 3 comments
Closed

classNamedOrNil test is bogus #740

seandenigris opened this issue Aug 3, 2015 · 3 comments

Comments

@seandenigris
Copy link
Contributor

Originally reported on Google Code with ID 740

There correct test should be 
When we give a metaclass name we should get the metaclass of the class and not nil

testClassNamedOrNil
    "(self selector: #testClassNamedOrNil) debug"
    self assert: (MooseUtilities classNamedOrNil: Model name) = Model.
    self assert: (MooseUtilities classNamedOrNil: FAMIXNameResolver name) = FAMIXNameResolver.
    self assert: (MooseUtilities classNamedOrNil: (FAMIXNameResolver name , FAMIXNameResolver
metaclassTag) asSymbol) = FAMIXNameResolver class

Reported by usman.bhatti on 2011-11-07 16:38:03
@seandenigris
Copy link
Contributor Author

Reported by usman.bhatti on 2011-11-07 16:38:14

  • Status changed: Fixed

@seandenigris
Copy link
Contributor Author 

probably classNamedOrNil: should use

smalltalkClassFromFamixClassName: aFAMIXName ifAbsent: aBlock

Reported by usman.bhatti on 2011-11-07 16:44:50

  • Status changed: Accepted

@seandenigris
Copy link
Contributor Author 

Actually, classNamedOrNil is not used anywhere, except for the test. So, I removed both.

Reported by tudor.girba on 2011-11-16 08:31:04

  • Status changed: Fixed
  • Labels added: Milestone-4.6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@seandenigris