Skip to content
This repository has been archived by the owner on Oct 4, 2021. It is now read-only.

Unit tests may load the wrong mono.cecil #6786

Open
Therzok opened this issue Dec 18, 2018 · 2 comments
Open

Unit tests may load the wrong mono.cecil #6786

Therzok opened this issue Dec 18, 2018 · 2 comments
Assignees
@Therzok
Labels
Area: Unit Testing vs-sync

Comments

@Therzok
Copy link
Contributor

Therzok commented Dec 18, 2018 •
edited by xamarin-release-manager
Loading

Given a unit test project project that references Mono.Cecil 0.10, running tests that will use Mono.Cecil will fail on a bad assembly loaded.

The cause of this is that NUnit3Runner.exe's NUnit3TestAdapter version is shipping its own Mono.Cecil, which seems to win on final copying: nunit/nunit3-vs-adapter#488

The solution is to bump NUnit3 in the NUnit3Runner project.

Should fix running Mono.Addins tests locally.

VS bug #753221
@Therzok Therzok self-assigned this Dec 18, 2018
@Therzok
Copy link
Contributor Author

Therzok commented Dec 18, 2018

Seems like NUnit.Engine still uses it 😕

@Therzok Therzok mentioned this issue Dec 18, 2018
Closed
@KirillOsenkov
Copy link
Member

I've filed an issue here: nunit/nunit3-vs-adapter#662

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@Therzok Therzok

Labels
Area: Unit Testing vs-sync
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Therzok @KirillOsenkov