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

Made '__base' method available for mocking #18

Merged
merged 1 commit into from
Feb 18, 2015

Conversation

L0stSoul
Copy link
Contributor

I have two classes

     var A = inherit({
            m : function() {
                //some unknown dependencies.
                return 'A';
            }
        })
       var B = inherit(A, {
            m : function() {
                return this.__base() + 'B';
            }
        })

And when i want to test 'm' method in class B, i can't mock method 'm' in base class. So for test purpose i need to mock all deependencies that base method 'm' have. It's really inconvenient and hard to maintain.

this PR is first try to somehow fix this situation.

@L0stSoul L0stSoul force-pushed the ls.__base branch 2 times, most recently from b782e0b to d78a259 Compare January 12, 2015 15:30
@dfilatov
Copy link
Owner

Thanks!

@dfilatov dfilatov closed this Feb 18, 2015
@dfilatov dfilatov reopened this Feb 18, 2015
dfilatov added a commit that referenced this pull request Feb 18, 2015
Made '__base' method available for mocking
@dfilatov dfilatov merged commit fd0d943 into dfilatov:master Feb 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants