[Clirr-devel] CVS: clirr/src/test/net/sf/clirr/checks MethodSetCheckTest.java,1.2,1.3

[Lars K?h. <lk...@us...>]

Update of /cvsroot/clirr/clirr/src/test/net/sf/clirr/checks
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv20264/test/net/sf/clirr/checks

Modified Files:
	MethodSetCheckTest.java 
Log Message:
improved the way covariant methods matched when before comparing them

Index: MethodSetCheckTest.java
===================================================================
RCS file: /cvsroot/clirr/clirr/src/test/net/sf/clirr/checks/MethodSetCheckTest.java,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -r1.2 -r1.3
--- MethodSetCheckTest.java	25 Jan 2004 08:16:46 -0000	1.2
+++ MethodSetCheckTest.java	22 May 2004 13:21:44 -0000	1.3
@@ -20,16 +20,8 @@
                     Severity.ERROR, "testlib.MethodsChange", "public void removedMethod(java.lang.String)", null),
             new ApiDifference("Method 'public int getPriv2()' has been removed in testlib.MethodsChange",
                     Severity.ERROR, "testlib.MethodsChange", "public int getPriv2()", null),
-            new ApiDifference("Method 'public void printPriv()' has been removed in testlib.MethodsChange",
-                    Severity.ERROR, "testlib.MethodsChange", "public void printPriv()", null),
-
-            // parameter type changes
-            //new ApiDifference("Parameter type of method 'public void changeParamterType(java.lang.String)' has changed.",
-            //        Severity.ERROR, "testlib.MethodsChange", "public void changeParamterType(java.lang.String)", null),
-            //new ApiDifference("Parameter type of method 'public void weakenParamterType(java.lang.String)' has changed.",
-            //        Severity.ERROR, "testlib.MethodsChange", "public void weakenParamterType(java.lang.String)", null),
-            //new ApiDifference("Parameter type of method 'public void strengthenParamterType(java.lang.String)' has changed.",
-            //        Severity.INFO, "testlib.MethodsChange", "public void strengthenParamterType(java.lang.Object)", null),
+            new ApiDifference("Method 'public java.lang.Long getPrivSquare()' has been added in testlib.MethodsChange",
+                    Severity.INFO, "testlib.MethodsChange", "public java.lang.Long getPrivSquare()", null),
 
             // Constructor changes
             new ApiDifference("Parameter 1 of 'protected MethodsChange(int)' has changed it's type to java.lang.Integer in testlib.MethodsChange",
@@ -47,6 +39,8 @@
             // parameter list changes
             // Note: This is the current behaviour, not necessarily the spec of the desired behaviour
             // TODO: need to check assignability of types (and check if method or class is final?)
+            new ApiDifference("In Method 'public void printPriv()' the number of arguments has changed in testlib.MethodsChange",
+                    Severity.ERROR, "testlib.MethodsChange", "public void printPriv()", null),
             new ApiDifference("Parameter 1 of 'public void strengthenParamType(java.lang.Object)' has changed it's type to java.lang.String in testlib.MethodsChange",
                     Severity.ERROR, "testlib.MethodsChange", "public void strengthenParamType(java.lang.Object)", null),
             new ApiDifference("Parameter 1 of 'public void weakenParamType(java.lang.String)' has changed it's type to java.lang.Object in testlib.MethodsChange",
@@ -54,6 +48,7 @@
             new ApiDifference("Parameter 1 of 'public void changeParamType(java.lang.String)' has changed it's type to java.lang.Integer in testlib.MethodsChange",
                     Severity.ERROR, "testlib.MethodsChange", "public void changeParamType(java.lang.String)", null),
 
+
             // declared exceptions
             // TODO
         };