Attachment 126839 Details for Bug 99399 – Proposed fix + tests

[patch] Proposed fix + tests

assistpatch.txt (text/plain), 29.00 KB, created by Srikanth Sankaran

on 2009-02-26 07:48:38 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Srikanth Sankaran

Created: 2009-02-26 07:48:38 EST

Size: 29.00 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.jdt.core.tests.compiler
>Index: src/org/eclipse/jdt/core/tests/compiler/parser/GenericsCompletionParserTest.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core.tests.compiler/src/org/eclipse/jdt/core/tests/compiler/parser/GenericsCompletionParserTest.java,v
>retrieving revision 1.36
>diff -u -r1.36 GenericsCompletionParserTest.java
>--- src/org/eclipse/jdt/core/tests/compiler/parser/GenericsCompletionParserTest.java	27 Jun 2008 16:04:46 -0000	1.36
>+++ src/org/eclipse/jdt/core/tests/compiler/parser/GenericsCompletionParserTest.java	26 Feb 2009 12:43:16 -0000
>@@ -39,7 +39,7 @@
> 	String completeBehind = "Z<Y>.";
> 	int cursorLocation = str.indexOf("Z<Y>.") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnType:Z<Y>.>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = "T extends <CompleteOnType:Z<Y>.>";
> 	String completionIdentifier = "";
> 	String expectedReplacedSource = "Z<Y>.";
> 	String expectedUnitDisplayString =
>@@ -67,7 +67,7 @@
> 	String completeBehind = "Z<Y>.W";
> 	int cursorLocation = str.indexOf("Z<Y>.W") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnType:Z<Y>.W>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = "T extends <CompleteOnType:Z<Y>.W>";
> 	String completionIdentifier = "W";
> 	String expectedReplacedSource = "Z<Y>.W";
> 	String expectedUnitDisplayString =
>@@ -95,7 +95,7 @@
> 	String completeBehind = "X<Y>.Z";
> 	int cursorLocation = str.indexOf("X<Y>.Z") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnType:test0001.X<Y>.Z>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = "T extends <CompleteOnType:test0001.X<Y>.Z>";
> 	String completionIdentifier = "Z";
> 	String expectedReplacedSource = "test0001.X<Y>.Z";
> 	String expectedUnitDisplayString =
>@@ -428,7 +428,11 @@
> 	String completeBehind = "Y<Z>.";
> 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnClass:Y<Z>.>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = 
>+		"public class X extends <CompleteOnClass:Y<Z>.> {\n" + 
>+		"  public X() {\n" + 
>+		"  }\n" + 
>+		"}";
> 	String completionIdentifier = "";
> 	String expectedReplacedSource = "Y<Z>.";
> 	String expectedUnitDisplayString =
>@@ -623,7 +627,7 @@
> 	String completeBehind = "Y<Z>.";
> 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnType:Y<Z>.>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = "T extends <CompleteOnType:Y<Z>.>";
> 	String completionIdentifier = "";
> 	String expectedReplacedSource = "Y<Z>.";
> 	String expectedUnitDisplayString =
>@@ -654,7 +658,7 @@
> 	String completeBehind = "Y<Z>.";
> 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnType:Y<Z>.>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = "T extends <CompleteOnType:Y<Z>.>";
> 	String completionIdentifier = "";
> 	String expectedReplacedSource = "Y<Z>.";
> 	String expectedUnitDisplayString =
>@@ -685,7 +689,7 @@
> 	String completeBehind = "Y<Z>.";
> 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnType:Y<Z>.>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = "T extends <CompleteOnType:Y<Z>.>";
> 	String completionIdentifier = "";
> 	String expectedReplacedSource = "Y<Z>.";
> 	String expectedUnitDisplayString =
>@@ -715,7 +719,7 @@
> 	String completeBehind = "Y<Z>.";
> 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnType:Y<Z>.>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = "T extends <CompleteOnType:Y<Z>.>";
> 	String completionIdentifier = "";
> 	String expectedReplacedSource = "Y<Z>.";
> 	String expectedUnitDisplayString =
>@@ -745,7 +749,7 @@
> 	String completeBehind = "Y<Z>.";
> 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnType:Y<Z>.>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = "T extends <CompleteOnType:Y<Z>.>";
> 	String completionIdentifier = "";
> 	String expectedReplacedSource = "Y<Z>.";
> 	String expectedUnitDisplayString =
>@@ -777,7 +781,7 @@
> 	String completeBehind = "Y<Z>.";
> 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnType:Y<Z>.>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = "T extends <CompleteOnType:Y<Z>.>";
> 	String completionIdentifier = "";
> 	String expectedReplacedSource = "Y<Z>.";
> 	String expectedUnitDisplayString =
>@@ -809,7 +813,7 @@
> 	String completeBehind = "Y<Z>.";
> 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnType:Y<Z>.>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = "T extends <CompleteOnType:Y<Z>.>";
> 	String completionIdentifier = "";
> 	String expectedReplacedSource = "Y<Z>.";
> 	String expectedUnitDisplayString =
>@@ -841,7 +845,7 @@
> 	String completeBehind = "Y<Z>.";
> 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnType:Y<Z>.>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = "T extends <CompleteOnType:Y<Z>.>";
> 	String completionIdentifier = "";
> 	String expectedReplacedSource = "Y<Z>.";
> 	String expectedUnitDisplayString =
>@@ -872,7 +876,7 @@
> 	String completeBehind = "Y<Z>.";
> 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnType:Y<Z>.>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = "T extends <CompleteOnType:Y<Z>.>";
> 	String completionIdentifier = "";
> 	String expectedReplacedSource = "Y<Z>.";
> 	String expectedUnitDisplayString =
>@@ -902,7 +906,7 @@
> 	String completeBehind = "Y<Z>.";
> 	int cursorLocation = str.indexOf("Y<Z>.") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnType:Y<Z>.>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = "T extends <CompleteOnType:Y<Z>.>";
> 	String completionIdentifier = "";
> 	String expectedReplacedSource = "Y<Z>.";
> 	String expectedUnitDisplayString =
>@@ -7792,7 +7796,7 @@
> 	String completeBehind = "Z";
> 	int cursorLocation = str.indexOf("Z") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnType:Z>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = "T extends <CompleteOnType:Z>";
> 	String completionIdentifier = "Z";
> 	String expectedReplacedSource = "Z";
> 	String expectedUnitDisplayString =
>@@ -7820,7 +7824,7 @@
> 	String completeBehind = "Z";
> 	int cursorLocation = str.indexOf("Z") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnType:X.Z>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = "T extends <CompleteOnType:X.Z>";
> 	String completionIdentifier = "Z";
> 	String expectedReplacedSource = "X.Z";
> 	String expectedUnitDisplayString =
>@@ -7848,7 +7852,7 @@
> 	String completeBehind = "Z";
> 	int cursorLocation = str.indexOf("Z") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnType:X<W>.Z>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = "T extends <CompleteOnType:X<W>.Z>";
> 	String completionIdentifier = "Z";
> 	String expectedReplacedSource = "X<W>.Z";
> 	String expectedUnitDisplayString =
>@@ -7877,7 +7881,7 @@
> 	String completeBehind = "Z";
> 	int cursorLocation = str.indexOf("Z") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnType:X<W>.Z>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = "T extends <CompleteOnType:X<W>.Z>";
> 	String completionIdentifier = "Z";
> 	String expectedReplacedSource = "X<W>.Z";
> 	String expectedUnitDisplayString =
>@@ -9057,7 +9061,7 @@
> 	String completeBehind = "Strin";
> 	int cursorLocation = str.indexOf("Strin") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnType:Strin>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = "? extends <CompleteOnType:Strin>";
> 	String completionIdentifier = "Strin";
> 	String expectedReplacedSource = "String";
> 	String expectedUnitDisplayString =
>@@ -10682,7 +10686,7 @@
> 	String completeBehind = "Obj";
> 	int cursorLocation = str.indexOf("Obj") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnType:Obj>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = "? extends <CompleteOnType:Obj>";
> 	String completionIdentifier = "Obj";
> 	String expectedReplacedSource = "Obj";
> 	String expectedUnitDisplayString =
>Index: src/org/eclipse/jdt/core/tests/compiler/parser/CompletionParserTest2.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core.tests.compiler/src/org/eclipse/jdt/core/tests/compiler/parser/CompletionParserTest2.java,v
>retrieving revision 1.49
>diff -u -r1.49 CompletionParserTest2.java
>--- src/org/eclipse/jdt/core/tests/compiler/parser/CompletionParserTest2.java	27 Jun 2008 16:04:46 -0000	1.49
>+++ src/org/eclipse/jdt/core/tests/compiler/parser/CompletionParserTest2.java	26 Feb 2009 12:43:11 -0000
>@@ -9327,7 +9327,13 @@
> 	String completeBehind = "Z.";
> 	int cursorLocation = str.indexOf("Z.") + completeBehind.length() - 1;
> 	String expectedCompletionNodeToString = "<CompleteOnClass:Z.>";
>-	String expectedParentNodeToString = "<NONE>";
>+	String expectedParentNodeToString = 
>+		"public class X extends <CompleteOnClass:Z.> {\n" + 
>+		"  {\n" + 
>+		"  }\n" + 
>+		"  public X() {\n" + 
>+		"  }\n" + 
>+		"}";
> 	String completionIdentifier = "";
> 	String expectedReplacedSource = "Z.";
> 	String expectedUnitDisplayString =
>#P org.eclipse.jdt.core
>Index: codeassist/org/eclipse/jdt/internal/codeassist/complete/CompletionParser.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core/codeassist/org/eclipse/jdt/internal/codeassist/complete/CompletionParser.java,v
>retrieving revision 1.201
>diff -u -r1.201 CompletionParser.java
>--- codeassist/org/eclipse/jdt/internal/codeassist/complete/CompletionParser.java	16 Jan 2009 14:29:29 -0000	1.201
>+++ codeassist/org/eclipse/jdt/internal/codeassist/complete/CompletionParser.java	26 Feb 2009 12:43:29 -0000
>@@ -2076,6 +2076,11 @@
> protected void consumeClassHeaderExtends() {
> 	pushOnElementStack(K_NEXT_TYPEREF_IS_CLASS);
> 	super.consumeClassHeaderExtends();
>+	if (this.assistNode != null && this.assistNodeParent == null) {
>+		TypeDeclaration typeDecl = (TypeDeclaration) this.astStack[this.astPtr];
>+		if (typeDecl != null && typeDecl.superclass == this.assistNode)
>+			this.assistNodeParent = typeDecl;
>+	}
> 	popElement(K_NEXT_TYPEREF_IS_CLASS);
> 	popElement(K_EXTENDS_KEYWORD);
> 
>@@ -3752,18 +3757,38 @@
> }
> protected void consumeTypeParameterWithExtends() {
> 	super.consumeTypeParameterWithExtends();
>+	if (this.assistNode != null && this.assistNodeParent == null) {
>+		TypeParameter typeParameter = (TypeParameter) this.genericsStack[this.genericsPtr];
>+		if (typeParameter != null && typeParameter.type == this.assistNode)
>+			this.assistNodeParent = typeParameter;
>+	}
> 	popElement(K_EXTENDS_KEYWORD);
> }
> protected void consumeTypeParameterWithExtendsAndBounds() {
> 	super.consumeTypeParameterWithExtendsAndBounds();
>+	if (this.assistNode != null && this.assistNodeParent == null) {
>+		TypeParameter typeParameter = (TypeParameter) this.genericsStack[this.genericsPtr];
>+		if (typeParameter != null && typeParameter.type == this.assistNode)
>+			this.assistNodeParent = typeParameter;
>+	}
> 	popElement(K_EXTENDS_KEYWORD);
> }
> protected void consumeTypeParameter1WithExtends() {
> 	super.consumeTypeParameter1WithExtends();
>+	if (this.assistNode != null && this.assistNodeParent == null) {
>+		TypeParameter typeParameter = (TypeParameter) this.genericsStack[this.genericsPtr];
>+		if (typeParameter != null && typeParameter.type == this.assistNode)
>+			this.assistNodeParent = typeParameter;
>+	}
> 	popElement(K_EXTENDS_KEYWORD);
> }
> protected void consumeTypeParameter1WithExtendsAndBounds() {
> 	super.consumeTypeParameter1WithExtendsAndBounds();
>+	if (this.assistNode != null && this.assistNodeParent == null) {
>+		TypeParameter typeParameter = (TypeParameter) this.genericsStack[this.genericsPtr];
>+		if (typeParameter != null && typeParameter.type == this.assistNode)
>+			this.assistNodeParent = typeParameter;
>+	}
> 	popElement(K_EXTENDS_KEYWORD);
> }
> protected void consumeWildcard() {
>@@ -3808,18 +3833,38 @@
> }
> protected void consumeWildcardBoundsExtends() {
> 	super.consumeWildcardBoundsExtends();
>+	if (this.assistNode != null && this.assistNodeParent == null) {
>+		Wildcard wildcard = (Wildcard) this.genericsStack[this.genericsPtr];
>+		if (wildcard != null && wildcard.bound == this.assistNode)
>+			this.assistNodeParent = wildcard;
>+	}
> 	popElement(K_EXTENDS_KEYWORD);
> }
> protected void consumeWildcardBounds1Extends() {
> 	super.consumeWildcardBounds1Extends();
>+	if (this.assistNode != null && this.assistNodeParent == null) {
>+		Wildcard wildcard = (Wildcard) this.genericsStack[this.genericsPtr];
>+		if (wildcard != null && wildcard.bound == this.assistNode)
>+			this.assistNodeParent = wildcard;
>+	}
> 	popElement(K_EXTENDS_KEYWORD);
> }
> protected void consumeWildcardBounds2Extends() {
> 	super.consumeWildcardBounds2Extends();
>+	if (this.assistNode != null && this.assistNodeParent == null) {
>+		Wildcard wildcard = (Wildcard) this.genericsStack[this.genericsPtr];
>+		if (wildcard != null && wildcard.bound == this.assistNode)
>+			this.assistNodeParent = wildcard;
>+	}
> 	popElement(K_EXTENDS_KEYWORD);
> }
> protected void consumeWildcardBounds3Extends() {
> 	super.consumeWildcardBounds3Extends();
>+	if (this.assistNode != null && this.assistNodeParent == null) {
>+		Wildcard wildcard = (Wildcard) this.genericsStack[this.genericsPtr];
>+		if (wildcard != null && wildcard.bound == this.assistNode)
>+			this.assistNodeParent = wildcard;
>+	}
> 	popElement(K_EXTENDS_KEYWORD);
> }
> protected void consumeUnaryExpression(int op) {
>Index: codeassist/org/eclipse/jdt/internal/codeassist/CompletionEngine.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core/codeassist/org/eclipse/jdt/internal/codeassist/CompletionEngine.java,v
>retrieving revision 1.388
>diff -u -r1.388 CompletionEngine.java
>--- codeassist/org/eclipse/jdt/internal/codeassist/CompletionEngine.java	5 Feb 2009 09:38:12 -0000	1.388
>+++ codeassist/org/eclipse/jdt/internal/codeassist/CompletionEngine.java	26 Feb 2009 12:43:26 -0000
>@@ -482,6 +482,7 @@
> 	private final static int NONE = 0;
> 	private final static int SUPERTYPE = 1;
> 	private final static int SUBTYPE = 2;
>+	private final static int FINALTYPE = 4; // https://bugs.eclipse.org/bugs/show_bug.cgi?id=99399
> 	
> 	int expectedTypesPtr = -1;
> 	TypeBinding[] expectedTypes = new TypeBinding[1];
>@@ -1113,6 +1114,7 @@
> 		this.foundTypesCount++;
> 		
> 		if (this.options.checkDeprecation && (modifiers & ClassFileConstants.AccDeprecated) != 0) return;
>+		if (((this.forbbidenBindingsFilter & FINALTYPE) != 0) && ((modifiers & ClassFileConstants.AccFinal) != 0)) return;
> 
> 		if (this.options.checkVisibility) {
> 			if((modifiers & ClassFileConstants.AccPublic) == 0) {
>@@ -3725,6 +3727,22 @@
> 
> 	private Scope computeForbiddenBindings(ASTNode astNode, ASTNode astNodeParent, Scope scope) {
> 		this.forbbidenBindingsFilter = NONE;
>+		// https://bugs.eclipse.org/bugs/show_bug.cgi?id=99399, don't propose final types for extension.
>+		if (astNodeParent != null) {
>+			if (astNodeParent instanceof TypeDeclaration) {
>+				TypeDeclaration typeDeclaration = (TypeDeclaration) astNodeParent;
>+				if (typeDeclaration.superclass == astNode)
>+					this.forbbidenBindingsFilter = FINALTYPE;	
>+			} else if (astNodeParent instanceof TypeParameter) {
>+				TypeParameter typeParameter = (TypeParameter) astNodeParent;
>+				if (typeParameter.type == astNode)
>+					this.forbbidenBindingsFilter = FINALTYPE;
>+			} else if (astNodeParent instanceof Wildcard) {
>+				Wildcard wildcard = (Wildcard) astNodeParent;
>+				if (wildcard.bound == astNode && wildcard.kind == Wildcard.EXTENDS)
>+					this.forbbidenBindingsFilter = FINALTYPE;
>+			}
>+		}
> 		if(scope instanceof ClassScope) {
> 			TypeDeclaration typeDeclaration = ((ClassScope)scope).referenceContext;
> 			if(typeDeclaration.superclass == astNode) {
>@@ -7319,6 +7337,8 @@
> 
> 			if (this.options.checkDeprecation && memberType.isViewedAsDeprecated()) continue next;
> 
>+			if (isForbidden(memberType)) continue next;
>+
> 			if (this.options.checkVisibility
> 				&& !memberType.canBeSeenBy(this.unitScope.fPackage))
> 				continue next;
>@@ -10271,6 +10291,9 @@
> 						!scope.isDefinedInSameUnit(sourceType))
> 					continue;
> 
>+				if (isForbidden(sourceType))
>+					continue;
>+
> 				int accessibility = IAccessRule.K_ACCESSIBLE;
> 				if(sourceType.hasRestrictedAccess()) {
> 					AccessRestriction accessRestriction = this.lookupEnvironment.getAccessRestriction(sourceType);
>@@ -10414,6 +10437,9 @@
> 							!scope.isDefinedInSameUnit(refBinding))
> 						continue next;
> 
>+					if (isForbidden(refBinding))
>+						continue next;
>+
> 					int accessibility = IAccessRule.K_ACCESSIBLE;
> 					if(refBinding.hasRestrictedAccess()) {
> 						AccessRestriction accessRestriction = this.lookupEnvironment.getAccessRestriction(refBinding);
>@@ -11596,6 +11622,11 @@
> 				}
> 			}
> 		}
>+		if((this.forbbidenBindingsFilter & FINALTYPE) != 0) {
>+			if (binding instanceof ReferenceBinding && ((ReferenceBinding)binding).isFinal()) {
>+				return true;
>+			}
>+		}
> 		return false;
> 	}
> 
>#P org.eclipse.jdt.core.tests.model
>Index: src/org/eclipse/jdt/core/tests/model/CompletionTests.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core.tests.model/src/org/eclipse/jdt/core/tests/model/CompletionTests.java,v
>retrieving revision 1.194
>diff -u -r1.194 CompletionTests.java
>--- src/org/eclipse/jdt/core/tests/model/CompletionTests.java	4 Feb 2009 16:17:14 -0000	1.194
>+++ src/org/eclipse/jdt/core/tests/model/CompletionTests.java	26 Feb 2009 12:43:47 -0000
>@@ -19353,4 +19353,71 @@
> 			"",
> 			requestor.getResults());
> }
>+
>+// https://bugs.eclipse.org/bugs/show_bug.cgi?id=99399 don't propose final types for extends type completion.
>+public void testCompletionOnExtendFinalClass () throws JavaModelException {
>+	this.workingCopies = new ICompilationUnit[1];
>+	this.workingCopies[0] = getWorkingCopy(
>+		"/Completion/src/test/Test.java",
>+		"package test;"+
>+		"class ThisClassIsNotFinal {}" +
>+		"final class ThisClassIsFinal {}" +
>+		"public class Event extends test.ThisClassI {\n" +
>+		"}");
>+	CompletionTestsRequestor2 requestor = new CompletionTestsRequestor2(true);
>+	String str = this.workingCopies[0].getSource();
>+	String completeBehind = "test.ThisClassI";
>+	int cursorLocation = str.lastIndexOf(completeBehind) + completeBehind.length();
>+	this.workingCopies[0].codeComplete(cursorLocation, requestor, this.wcOwner);
>+
>+	assertResults(
>+			"ThisClassIsNotFinal[TYPE_REF]{ThisClassIsNotFinal, test, Ltest.ThisClassIsNotFinal;, null, null, " + (R_DEFAULT + R_RESOLVED + R_INTERESTING + R_CASE + R_NON_RESTRICTED + R_CLASS) + "}",	
>+			requestor.getResults());
>+}
>+
>+//https://bugs.eclipse.org/bugs/show_bug.cgi?id=99399 don't propose final types for extends type completion.
>+public void testCompletionOnExtendFinalClass2() throws JavaModelException {
>+	this.workingCopies = new ICompilationUnit[1];
>+	this.workingCopies[0] = getWorkingCopy(
>+		"/Completion/src/test/Test.java",
>+		"package test;"+
>+		"class ThisClassIsNotFinal {}" +
>+		"final class ThisClassIsFinal {}" +
>+		"public class Event extends ThisClassI {\n" +
>+		"}");
>+	CompletionTestsRequestor2 requestor = new CompletionTestsRequestor2(true);
>+	String str = this.workingCopies[0].getSource();
>+	String completeBehind = "ThisClassI";
>+	int cursorLocation = str.lastIndexOf(completeBehind) + completeBehind.length();
>+	this.workingCopies[0].codeComplete(cursorLocation, requestor, this.wcOwner);
>+
>+	assertResults(
>+			"ThisClassIsNotFinal[TYPE_REF]{ThisClassIsNotFinal, test, Ltest.ThisClassIsNotFinal;, null, null, " + (R_DEFAULT + R_RESOLVED + R_INTERESTING + R_CASE + R_UNQUALIFIED + R_NON_RESTRICTED + R_CLASS) + "}",
>+			requestor.getResults());
>+}
>+
>+// https://bugs.eclipse.org/bugs/show_bug.cgi?id=99399 test to verify that we continue to propose final types
>+// in other (non extends context).
>+public void testCompletionOnExtendFinalClass3() throws JavaModelException {
>+	this.workingCopies = new ICompilationUnit[1];
>+	this.workingCopies[0] = getWorkingCopy(
>+		"/Completion/src/test/Test.java",
>+		"package test;"+
>+		"class ThisClassIsNotFinal {}" +
>+		"final class ThisClassIsFinal {}" +
>+		"public class Event extends ThisClassIsFinal {\n" +
>+		"	void Boo (ThisClassI x) {}\n" +
>+		"}");
>+	CompletionTestsRequestor2 requestor = new CompletionTestsRequestor2(true);
>+	String str = this.workingCopies[0].getSource();
>+	String completeBehind = "ThisClassI";
>+	int cursorLocation = str.lastIndexOf(completeBehind) + completeBehind.length();
>+	this.workingCopies[0].codeComplete(cursorLocation, requestor, this.wcOwner);
>+
>+	assertResults(
>+			"ThisClassIsFinal[TYPE_REF]{ThisClassIsFinal, test, Ltest.ThisClassIsFinal;, null, null, " + (R_DEFAULT + R_RESOLVED + R_INTERESTING + R_CASE + R_UNQUALIFIED + R_NON_RESTRICTED) + "}\n" +
>+			"ThisClassIsNotFinal[TYPE_REF]{ThisClassIsNotFinal, test, Ltest.ThisClassIsNotFinal;, null, null, " + (R_DEFAULT + R_RESOLVED + R_INTERESTING + R_CASE + R_UNQUALIFIED + R_NON_RESTRICTED) + "}",
>+			requestor.getResults());
>+}
>+
> }
>Index: src/org/eclipse/jdt/core/tests/model/CompletionTests_1_5.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core.tests.model/src/org/eclipse/jdt/core/tests/model/CompletionTests_1_5.java,v
>retrieving revision 1.111
>diff -u -r1.111 CompletionTests_1_5.java
>--- src/org/eclipse/jdt/core/tests/model/CompletionTests_1_5.java	30 Jan 2009 15:47:20 -0000	1.111
>+++ src/org/eclipse/jdt/core/tests/model/CompletionTests_1_5.java	26 Feb 2009 12:43:55 -0000
>@@ -323,8 +323,8 @@
>             ".Y001");
> 
>     assertResults(
>-            "expectedTypesSignatures=null\n" +
>-            "expectedTypesKeys=null",
>+            "expectedTypesSignatures={Ltest0011.Z0011<Ljava.lang.Object;>;}\n" +
>+            "expectedTypesKeys={Ltest0011/Test~Z0011<Ljava/lang/Object;>;}",
>             result.context);
> 
>     assertResults(
>@@ -490,8 +490,8 @@
>             ".Y001");
> 
>     assertResults(
>-            "expectedTypesSignatures=null\n" +
>-            "expectedTypesKeys=null",
>+            "expectedTypesSignatures={Ltest0018.Z0018<Ljava.lang.Object;>;}\n" +
>+            "expectedTypesKeys={Ltest0018/Test~Z0018<Ljava/lang/Object;>;}",
>             result.context);
> 
>     assertResults(
>@@ -513,8 +513,8 @@
>             ".Y001");
> 
>     assertResults(
>-            "expectedTypesSignatures=null\n" +
>-            "expectedTypesKeys=null",
>+            "expectedTypesSignatures={Ltest0019.Z0019<Ljava.lang.Object;>;}\n" +
>+            "expectedTypesKeys={Ltest0019/Test~Z0019<Ljava/lang/Object;>;}",
>             result.context);
> 
>     assertResults(
>@@ -13476,4 +13476,105 @@
> 			requestor.getResults());
> }
> 
>+// https://bugs.eclipse.org/bugs/show_bug.cgi?id=99399 test to verify that we continue to propose final
>+// types in extends contexts but where they are not directly extended.
>+public void testCompletionOnExtends() throws JavaModelException {
>+	this.workingCopies = new ICompilationUnit[1];
>+	this.workingCopies[0] = getWorkingCopy(
>+			"/Completion/src3/test/Test.java",
>+			"package test;\n" +
>+			"class X<T> {};\n" +
>+			"final class ThisClassIsFinal {}\n" +
>+			"class ThisClassIsNotFinal {}\n" +
>+			"public class Test extends X<ThisClassI> {}");
>+
>+	CompletionTestsRequestor2 requestor = new CompletionTestsRequestor2(true);
>+
>+	String str = this.workingCopies[0].getSource();
>+	String completeBehind = "ThisClassI";
>+	int cursorLocation = str.lastIndexOf(completeBehind) + completeBehind.length();
>+	this.workingCopies[0].codeComplete(cursorLocation, requestor, this.wcOwner);
>+
>+	assertResults(
>+			"ThisClassIsFinal[TYPE_REF]{ThisClassIsFinal, test, Ltest.ThisClassIsFinal;, null, null, " + (R_DEFAULT + R_RESOLVED + R_INTERESTING + R_CASE + R_UNQUALIFIED + R_NON_RESTRICTED + R_CLASS) + "}\n" +
>+			"ThisClassIsNotFinal[TYPE_REF]{ThisClassIsNotFinal, test, Ltest.ThisClassIsNotFinal;, null, null, " + (R_DEFAULT + R_RESOLVED + R_INTERESTING + R_CASE + R_UNQUALIFIED + R_NON_RESTRICTED + R_CLASS) + "}",
>+			requestor.getResults());
>+}
>+
>+//https://bugs.eclipse.org/bugs/show_bug.cgi?id=99399 test to verify that we don't propose final
>+//types in extends contexts where we should not.
>+public void testCompletionOnExtends2() throws JavaModelException {
>+	this.workingCopies = new ICompilationUnit[1];
>+	this.workingCopies[0] = getWorkingCopy(
>+			"/Completion/src3/test/Test.java",
>+			"package test;\n" +
>+			"final class ThisClassIsFinal {}\n" +
>+			"class ThisClassIsNotFinal {}\n" +
>+			"public class Test <T extends ThisClassI> {}\n");
>+
>+	CompletionTestsRequestor2 requestor = new CompletionTestsRequestor2(true);
>+
>+	String str = this.workingCopies[0].getSource();
>+	String completeBehind = "ThisClassI";
>+	int cursorLocation = str.lastIndexOf(completeBehind) + completeBehind.length();
>+	this.workingCopies[0].codeComplete(cursorLocation, requestor, this.wcOwner);
>+
>+	assertResults(
>+			"ThisClassIsNotFinal[TYPE_REF]{ThisClassIsNotFinal, test, Ltest.ThisClassIsNotFinal;, null, null, " + (R_DEFAULT + R_RESOLVED + R_INTERESTING + R_CASE + R_UNQUALIFIED + R_NON_RESTRICTED) + "}",
>+			requestor.getResults());
>+}
>+
>+//https://bugs.eclipse.org/bugs/show_bug.cgi?id=99399 test to verify that we don't propose final
>+//types in extends contexts where we should not.
>+public void testCompletionOnExtends3() throws JavaModelException {
>+	this.workingCopies = new ICompilationUnit[1];
>+	this.workingCopies[0] = getWorkingCopy(
>+			"/Completion/src3/test/Test.java",
>+			"package test;\n" +
>+			"final class ThisClassIsFinal {}\n" +
>+			"class ThisClassIsNotFinal {}\n" +
>+			"public class Test {\n" +
>+			"    Test(Bag<? extends ThisClassI> p) {}" +
>+			"}\n");
>+
>+	CompletionTestsRequestor2 requestor = new CompletionTestsRequestor2(true);
>+
>+	String str = this.workingCopies[0].getSource();
>+	String completeBehind = "ThisClassI";
>+	int cursorLocation = str.lastIndexOf(completeBehind) + completeBehind.length();
>+	this.workingCopies[0].codeComplete(cursorLocation, requestor, this.wcOwner);
>+
>+	assertResults(
>+			"ThisClassIsNotFinal[TYPE_REF]{ThisClassIsNotFinal, test, Ltest.ThisClassIsNotFinal;, null, null, " + (R_DEFAULT + R_RESOLVED + R_INTERESTING + R_CASE + R_UNQUALIFIED + R_NON_RESTRICTED) + "}",
>+			requestor.getResults());
>+}
>+
>+//https://bugs.eclipse.org/bugs/show_bug.cgi?id=99399 test to verify that we do propose final
>+//types in super contexts where we should.
>+public void testCompletionOnExtends4() throws JavaModelException {
>+	this.workingCopies = new ICompilationUnit[1];
>+	this.workingCopies[0] = getWorkingCopy(
>+			"/Completion/src3/test/Test.java",
>+			"package test;\n" +
>+			"final class ThisClassIsFinal {}\n" +
>+			"class ThisClassIsNotFinal {}\n" +
>+			"public class Test {\n" +
>+			"    void boo() {\n" +
>+			"        Bag<? super ThisClassI> local;\n" +
>+			"    }\n" +
>+			"}\n");
>+
>+	CompletionTestsRequestor2 requestor = new CompletionTestsRequestor2(true);
>+
>+	String str = this.workingCopies[0].getSource();
>+	String completeBehind = "ThisClassI";
>+	int cursorLocation = str.lastIndexOf(completeBehind) + completeBehind.length();
>+	this.workingCopies[0].codeComplete(cursorLocation, requestor, this.wcOwner);
>+
>+	assertResults(
>+			"ThisClassIsFinal[TYPE_REF]{ThisClassIsFinal, test, Ltest.ThisClassIsFinal;, null, null, " + (R_DEFAULT + R_RESOLVED + R_INTERESTING + R_CASE + R_UNQUALIFIED + R_NON_RESTRICTED) + "}\n" +
>+			"ThisClassIsNotFinal[TYPE_REF]{ThisClassIsNotFinal, test, Ltest.ThisClassIsNotFinal;, null, null, " + (R_DEFAULT + R_RESOLVED + R_INTERESTING + R_CASE + R_UNQUALIFIED + R_NON_RESTRICTED) + "}",
>+			requestor.getResults());
>+}
>+
> }

Actions: View | Diff

Attachments on bug 99399: 126839 | 127156 | 127304