Discussion :

[altef quatre]

Bonsoir,

Au niveau du contexte, je souhaite tester une classe imbriquee qui utilise un delegate pour des operations mathematiques (add, sub, mul...).
La classe Nested est imbriquee et en protected car elle n'a pas d'utilite a l'exterieur de MyDelegate. Idem pour le delegate en protected, qui depend de la visibilite de la classe Nested.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
public class MyDelegate
{
	protected class Nested
	{
		protected delegate float MyOpDelegate (Nested x, Nested y);

		// Champs et Constructeurs...

		public static float Add (Nested x, Nested y)
		{
			// des calculs...
		}

		protected float Calculate (MyOpDelegate op)
		{
			// code...
		}
	}
}

Au niveau du test unitaire:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
float res = 0;
MyDelegate_Accessor.Nested nested = new MyDelegate_Accessor.Nested (42f);
 
res = nested.Calculate (MyDelegate_Accessor.Nested.Add);
Assert.AreEqual (res, 51f);

La compilation du code passe nickel, mais en revanche le code du test unitaire plante:

erreur CS1502: La méthode surchargée correspondant le mieux à 'DelegateDummy.MyDelegate_Accessor.Nested.Calculate(DelegateDummy.MyDelegate_Accessor.Nested.MyOpDelegate)' possède des arguments non valides
erreur CS1503: Argument '1'*: impossible de convertir de 'groupe de méthodes' en 'DelegateDummy.MyDelegate_Accessor.Nested.MyOpDelegate'

Le compilateur parle de groupe de methodes, mais ici, Add n'est pas surchargee.
Lorsque je passe tout en public (classe Nested et delegate), le compilateur est gentil et le code du test compile normalement (mais si possible, j'aimerais eviter de tout mettre en public, comme je l'ai enonce plus haut).
Auriez-vous une explication a cela, et comment je peux resoudre mon probleme ?

Autre question: Lorsque je veux creer un test unitaire pour la methode Calculate, Visual Studio ne me propose pas ce choix dans l'assistant... Pourquoi ?

Merci

[tomlev]

Tu utilises quelle version de C# ? Avant la version 2.0, il fallait spécifier explicitement le delegate à utiliser :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

res = nested.Calculate (new MyDelegate_Accessor.Nested.MyOpDelegate(MyDelegate_Accessor.Nested.Add));

[altef quatre]

Si c'est bien comme ca pour obtenir la version de C#, j'ai la version 2.0.50727.4927.

[tomlev]

Bizarre.... et avec le code que je t'ai donné, ça marche pas non plus ?

[altef quatre]

Non c'est pas vraiment mieux, j'obtiens:

erreur CS1503: Argument '1'*: impossible de convertir de 'groupe de méthodes' en 'Microsoft.VisualStudio.TestTools.UnitTesting.PrivateObject'

[tomlev]

Bizarre cette erreur

C'est pas le fait que ta classe Nested et le delegate soient protected qui pose problème ? Parce qu'a priori le code des tests unitaires ne peut pas y accéder si c'est protégé...

[bacelar]

Je crois bien que l'attribut "InternalsVisibleTo" (http://msdn.microsoft.com/fr-fr/library/bb385840.aspx) est une solution de contournement assez moyenne.

[altef quatre]

... ce qui expliquerait le pourquoi ca marche lorsque tout est en public...
Parmi les solutions que j'ai pu voir en googlisant:
* utiliser le preprocesseur pour changer l'attribut des methodes (#ifdef DEBUG public #else protected...) => BEURK
* tout passer en public pour avoir le plaisir de voir les tests marcher => BEURK
* utiliser la reflection dans les tests:
Apres avoir pas mal galere, voila ou j'en suis rendu, et ca a l'air de marcher, meme si le code est tres tire par les cheveux...:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
// On recupere la classe imbrique protegee
Type tNested = new MyDelegate ().GetType ().GetNestedType ("Nested", BindingFlags.NonPublic | BindingFlags.Instance);
 
// On recupere la methode a passer en argument (Add) pour en faire un nouveau delegate de type MyOpDelegate
MethodInfo miAdd = tNested.GetMethod ("Add");
Type tDelegate = tNested.GetNestedType ("MyOpDelegate", BindingFlags.NonPublic | BindingFlags.Public);
Delegate addDelegate = Delegate.CreateDelegate (tDelegate, miAdd);
 
// Enfin, on appelle Calculate a partir d'une instance de Nested.
Object nested = System.Activator.CreateInstance (tNested, new object[] {42f});
MethodInfo miCalculate = tNested.GetMethod ("Calculate", BindingFlags.NonPublic | BindingFlags.Instance);
res = (float) miCalculate.Invoke (nested, new object[] {addDelegate});

En comparaison avec le code initial sans reflection, mais qui ne marche pas...:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
MyDelegate_Accessor.Nested nested = new MyDelegate_Accessor.Nested (42f);
 
res = nested.Calculate (MyDelegate_Accessor.Nested.Add);

Si quelqu'un a une meilleure solution/solution moins alambiquee, je suis preneur...

Merci tomlev pour ta participation