Support for multi-stubs + Update to xUnit 1.8 #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… the solution build.
…der to support multi-stubs.
…amicProxy2.dll was removed and some entities have to be public after merge (took list from http://nmock3.codeplex.com/discussions/245846 where the same problem was discussed).
…a inherited interface which has the same property as the base interface.
…dly copy list from http://nmock3.codeplex.com/discussions/245846 - just consider the namespace changes.
…changes since first 3.6 release.
…fications' in ISpecification.cs (it should be 'Rhino.Mocks.Impl.Invocation.Specifications').
…lement the partial stub feature.
…PI is necessary to avoid breaking changes in the previous stub behavior for virtual methods).
…n't blindly copy list from http://nmock3.codeplex.com/discussions/245846 - just consider the namespace changes." I now agree with Cygon - but maybe there is a better solution... This reverts commit bf2c5a7.
…a inherited interface which has the same property as the base interface (also read-only properties are effected!).
…sses that call abstract members in the constructor. To be able to implement a useful behavior in the future I only declared in the tests that it should be able create such mocks/stubs. I know that normally you should avoid such a (bad) class design - but for some specific scenarios it makes perfectly sense to set/initialize a abstract property in the constructor. Note that this is a minor breaking change - but because the creation of such stubs/mocks throws an exception previous to this change I hope no one is affected.
…bs for abstract classes that call abstract members in the constructor."-commit 7a4ed10. Preserved marker interface IPartialMockMarker in order to minimize public API change.
Added Castle.Core.pdb to be merged into Rhino.Mocks.pdb. Use commit label for file name of the release package (the ccnet label isn't available).
- Updated psake to v4.00 (only changed default platform to .Net 4) - Updated build targets framework version of all projects. - Made use of Castle.Core.dll for .Net 4.
… with delegates") - but it seems that this is already working.
…d7f52f20cc60a6b885fb02cfa1f492cec9a4)
…adataToken can differ even on identical types).
The value of a variable used as a ref parameter should be used as a constraint on an expectation even when if it is marked with an InteropServices.OutAttribute
Fixed ref parameters on COM interfaces. The value of a variable used as a ref parameter should be used as a constraint on an expectation even when if it is marked with an InteropServices.OutAttribute.
…his in the changelog.
Changed reflection lookup to look for nonpublic properties too
… to enforce AAA style.
…Type, Type[], params object[]) to follow MockRepsoitory.GenerateStrictMock(Type type, Type[] extraTypes, params object[] argumentsForConstructor).
Fixes property behavior for internal properties.
…xception to throw or an action to execute - even in AAA syntax.
…hould verify that the given arguments are valid for the methods signature.
…5 is installed on the build machine).
…PI loses some testing, I still find it useful to prove that all functionality can be achieved with the AAA syntax.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No breaking changes. Just included the patch for multi-stubs from https://github.com/shiqinwen and added a generic method to generate multi-stubs. Then i've updated xUnit.
As far as I see two useful extensions. This is my first pull request - so please let me know if there is something wrong.
Br,
Andreas