Skip to content

Test failure: NoPublicTypesUnderLibGit2SharpCoreNamespace #1312

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

Closed
AArnott opened this issue May 5, 2016 · 3 comments
Closed

Test failure: NoPublicTypesUnderLibGit2SharpCoreNamespace #1312

AArnott opened this issue May 5, 2016 · 3 comments

Comments

@AArnott
Copy link
Contributor

AArnott commented May 5, 2016

This test fails in the master branch (as of 1fb7fc5)

Test Name:  LibGit2Sharp.Tests.MetaFixture.NoPublicTypesUnderLibGit2SharpCoreNamespace
Test FullName:  LibGit2Sharp.Tests.MetaFixture.NoPublicTypesUnderLibGit2SharpCoreNamespace
Test Source:    C:\Users\andarno\git\libgit2sharp\LibGit2Sharp.Tests\MetaFixture.cs : line 298
Test Outcome:   Failed
Test Duration:  0:00:00.003

Result StackTrace:  at LibGit2Sharp.Tests.MetaFixture.NoPublicTypesUnderLibGit2SharpCoreNamespace() in C:\Users\andarno\git\libgit2sharp\LibGit2Sharp.Tests\MetaFixture.cs:line 324
Result Message: Public type 'LibGit2Sharp.Core.LeaksContainer' under the 'LibGit2Sharp.Core' namespace.
@carlosmn
Copy link
Member

carlosmn commented May 6, 2016

This happens because the compilation defines LEAKS_IDENTIFYING which makes us create this flag, but the tests got compiled by VS itself which did not, thus not making us include LeaksContainer in the list of classes that are OK to export.

We should probably get rid of this long-term as we'd actually want to expose things in core.

@carlosmn
Copy link
Member

carlosmn commented May 6, 2016

Does #1317 fix it for you?

@ethomson
Copy link
Member

ethomson commented May 6, 2016

I think #1317 should fix this - @AArnott please feel free to reopen if it doesn't.

@ethomson ethomson closed this as completed May 6, 2016
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

No branches or pull requests

3 participants