NUnit V2

Add AllowPartiallyTrustedCallersAttribute to nunit.framework assembly to allow use in medium trust scenarios

Bug #746247 reported by Michael Ketting
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
NUnit V2
Fix Released
Medium
Charlie Poole
NUnit V2 2.6.0b1

Bug Description

/well, it's not a bug, it's a feature request/

Please consder adding the AllowPartiallyTrustedCallersAttribute to the nunit.framework assembly. We're running certain integration tests in a medium trust scenario to ensure compatibility of our library in these cases and right now, have to make a custom build of nunit.framework to enable this. The only alternative is to forgo the nunit asserations, but we've really come to like them ;)

Regards, Michael

Tags: feature

Related branches

lp://staging/nunitv2
Charlie Poole (charlie.poole)
tags: added: feature
Charlie Poole (charlie.poole)
Changed in nunitv2:
status: New → Triaged
importance: Undecided → Medium
assignee: nobody → Charlie Poole (charlie.poole)
milestone: none → 2.6.0b1
Revision history for this message
Charlie Poole (charlie.poole) wrote :

Is it only the framework assembly that requires this for your purposes?

Revision history for this message
Michael Ketting (michael-ketting) wrote :

Hi Charlie!

Yes, the only immediate need is the nunit.framework assembly. We're using reflection to discover the TestFixtures and Test-methods since our tests are simple enough and don't take advantage of the more sophisticated nunit features.

Regarding the other assemlies: I've taken a quick look around and yes, it would be nice if you would also mark nunit.core.interfaces, nunit.core, and nunit.util as partially trusted. In the end, a nice, attribute (or the likes) syntax would be great, with which one could specify a test fixture to run in a partially trusted sandbox with a set of permissions specified by the test author. From the (very quick) look of it, RemoteTestRunner might actually be already half-way there. If there's time, we'll likely take a closer look at it and see if it's possible to come up with either an extension or a non-intrusive patch.

Thanks, Michael

Charlie Poole (charlie.poole)
Changed in nunitv2:
status: Triaged → Fix Committed
Charlie Poole (charlie.poole)
Changed in nunitv2:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.