blob: 3f029b93b4a97af11c1bedddee8f05545bf99aa1 [file] [log] [blame]
/*
* Copyright (C) 2012 The Android Open Source Project
*
* Licensed under the Eclipse Public License, Version 1.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.eclipse.org/org/documents/epl-v10.php
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package com.android.ide.eclipse.adt.internal.refactorings.core;
import com.android.annotations.NonNull;
import com.android.ide.common.resources.ResourceUrl;
import com.android.ide.eclipse.adt.internal.project.BaseProjectHelper;
import com.android.resources.ResourceType;
import org.eclipse.core.resources.IFile;
import org.eclipse.core.resources.IProject;
import org.eclipse.core.resources.IResource;
import org.eclipse.jdt.core.IField;
import org.eclipse.jdt.core.IJavaProject;
import org.eclipse.jdt.core.IType;
import org.eclipse.jdt.internal.corext.refactoring.rename.RenameFieldProcessor;
import org.eclipse.ltk.core.refactoring.participants.RenameProcessor;
import org.eclipse.ltk.core.refactoring.participants.RenameRefactoring;
@SuppressWarnings({"javadoc", "restriction"})
public class RenameResourceParticipantTest extends RefactoringTestBase {
public void testRefactor1() throws Exception {
renameResource(
TEST_PROJECT,
"@string/app_name",
true /*updateReferences*/,
"myname",
"CHANGES:\n" +
"-------\n" +
"[x] strings.xml - /testRefactor1/res/values/strings.xml\n" +
" @@ -4 +4\n" +
" - <string name=\"app_name\">RefactoringTest</string>\n" +
" + <string name=\"myname\">RefactoringTest</string>\n" +
"\n" +
"\n" +
"[ ] R.java - /testRefactor1/gen/com/example/refactoringtest/R.java\n" +
" @@ -29 +29\n" +
" - public static final int app_name=0x7f040000;\n" +
" + public static final int myname=0x7f040000;\n" +
"\n" +
"\n" +
"[x] AndroidManifest.xml - /testRefactor1/AndroidManifest.xml\n" +
" @@ -13 +13\n" +
" - android:label=\"@string/app_name\"\n" +
" + android:label=\"@string/myname\"\n" +
" @@ -17 +17\n" +
" - android:label=\"@string/app_name\" >\n" +
" + android:label=\"@string/myname\" >");
}
public void testRefactor2() throws Exception {
renameResource(
TEST_PROJECT,
"@+id/menu_settings",
true /*updateReferences*/,
"new_id_for_the_action_bar",
"CHANGES:\n" +
"-------\n" +
"[x] activity_main.xml - /testRefactor2/res/menu/activity_main.xml\n" +
" @@ -4 +4\n" +
" - android:id=\"@+id/menu_settings\"\n" +
" + android:id=\"@+id/new_id_for_the_action_bar\"\n" +
"\n" +
"\n" +
"[ ] R.java - /testRefactor2/gen/com/example/refactoringtest/R.java\n" +
" @@ -19 +19\n" +
" - public static final int menu_settings=0x7f070003;\n" +
" + public static final int new_id_for_the_action_bar=0x7f070003;");
}
public void testRefactor3() throws Exception {
renameResource(
TEST_PROJECT,
"@+id/textView1",
true /*updateReferences*/,
"output",
"CHANGES:\n" +
"-------\n" +
"[x] activity_main.xml - /testRefactor3/res/layout/activity_main.xml\n" +
" @@ -8 +8\n" +
" - android:id=\"@+id/textView1\"\n" +
" + android:id=\"@+id/output\"\n" +
" @@ -19 +19\n" +
" - android:layout_alignLeft=\"@+id/textView1\"\n" +
" - android:layout_below=\"@+id/textView1\"\n" +
" + android:layout_alignLeft=\"@+id/output\"\n" +
" + android:layout_below=\"@+id/output\"\n" +
"\n" +
"\n" +
"[x] MainActivity.java - /testRefactor3/src/com/example/refactoringtest/MainActivity.java\n" +
" @@ -14 +14\n" +
" - View view1 = findViewById(R.id.textView1);\n" +
" + View view1 = findViewById(R.id.output);\n" +
"\n" +
"\n" +
"[ ] R.java - /testRefactor3/gen/com/example/refactoringtest/R.java\n" +
" @@ -20 +20\n" +
" - public static final int textView1=0x7f070000;\n" +
" + public static final int output=0x7f070000;");
}
public void testRefactor4() throws Exception {
renameResource(
TEST_PROJECT,
// same as testRefactor3, but use @id rather than @+id even though @+id is in file
"@id/textView1",
true /*updateReferences*/,
"output",
"CHANGES:\n" +
"-------\n" +
"[x] activity_main.xml - /testRefactor4/res/layout/activity_main.xml\n" +
" @@ -8 +8\n" +
" - android:id=\"@+id/textView1\"\n" +
" + android:id=\"@+id/output\"\n" +
" @@ -19 +19\n" +
" - android:layout_alignLeft=\"@+id/textView1\"\n" +
" - android:layout_below=\"@+id/textView1\"\n" +
" + android:layout_alignLeft=\"@+id/output\"\n" +
" + android:layout_below=\"@+id/output\"\n" +
"\n" +
"\n" +
"[x] MainActivity.java - /testRefactor4/src/com/example/refactoringtest/MainActivity.java\n" +
" @@ -14 +14\n" +
" - View view1 = findViewById(R.id.textView1);\n" +
" + View view1 = findViewById(R.id.output);\n" +
"\n" +
"\n" +
"[ ] R.java - /testRefactor4/gen/com/example/refactoringtest/R.java\n" +
" @@ -20 +20\n" +
" - public static final int textView1=0x7f070000;\n" +
" + public static final int output=0x7f070000;");
}
public void testRefactor5() throws Exception {
renameResource(
TEST_PROJECT,
"@layout/activity_main",
true /*updateReferences*/,
"newlayout",
"CHANGES:\n" +
"-------\n" +
"[x] MainActivity.java - /testRefactor5/src/com/example/refactoringtest/MainActivity.java\n" +
" @@ -13 +13\n" +
" - setContentView(R.layout.activity_main);\n" +
" + setContentView(R.layout.newlayout);\n" +
"\n" +
"\n" +
"[ ] R.java - /testRefactor5/gen/com/example/refactoringtest/R.java\n" +
" @@ -23 +23\n" +
" - public static final int activity_main=0x7f030000;\n" +
" + public static final int newlayout=0x7f030000;\n" +
"\n" +
"\n" +
"[x] Rename 'testRefactor5/res/layout/activity_main.xml' to 'newlayout.xml'\n" +
"\n" +
"[x] Rename 'testRefactor5/res/layout-land/activity_main.xml' to 'newlayout.xml'");
}
public void testRefactor6() throws Exception {
renameResource(
TEST_PROJECT,
"@drawable/ic_launcher",
true /*updateReferences*/,
"newlauncher",
"CHANGES:\n" +
"-------\n" +
"[ ] R.java - /testRefactor6/gen/com/example/refactoringtest/R.java\n" +
" @@ -14 +14\n" +
" - public static final int ic_launcher=0x7f020000;\n" +
" + public static final int newlauncher=0x7f020000;\n" +
"\n" +
"\n" +
"[x] Rename 'testRefactor6/res/drawable-xhdpi/ic_launcher.png' to 'newlauncher.png'\n" +
"\n" +
"[x] Rename 'testRefactor6/res/drawable-mdpi/ic_launcher.png' to 'newlauncher.png'\n" +
"\n" +
"[x] Rename 'testRefactor6/res/drawable-ldpi/ic_launcher.png' to 'newlauncher.png'\n" +
"\n" +
"[x] Rename 'testRefactor6/res/drawable-hdpi/ic_launcher.png' to 'newlauncher.png'\n" +
"\n" +
"[x] AndroidManifest.xml - /testRefactor6/AndroidManifest.xml\n" +
" @@ -12 +12\n" +
" - android:icon=\"@drawable/ic_launcher\"\n" +
" + android:icon=\"@drawable/newlauncher\"");
}
public void testRefactor7() throws Exception {
// Test refactoring initiated on a file rename
IProject project = createProject(TEST_PROJECT);
IFile file = project.getFile("res/layout/activity_main.xml");
renameResource(
project,
file,
true /*updateReferences*/,
"newlayout",
"CHANGES:\n" +
"-------\n" +
"[x] MainActivity.java - /testRefactor7/src/com/example/refactoringtest/MainActivity.java\n" +
" @@ -13 +13\n" +
" - setContentView(R.layout.activity_main);\n" +
" + setContentView(R.layout.newlayout);\n" +
"\n" +
"\n" +
"[ ] R.java - /testRefactor7/gen/com/example/refactoringtest/R.java\n" +
" @@ -23 +23\n" +
" - public static final int activity_main=0x7f030000;\n" +
" + public static final int newlayout=0x7f030000;\n" +
"\n" +
"\n" +
"[x] Rename 'testRefactor7/res/layout-land/activity_main.xml' to 'newlayout.xml'\n" +
"\n" +
"[x] Rename 'testRefactor7/res/layout/activity_main.xml' to 'newlayout.xml'",
null);
}
public void testRefactor8() throws Exception {
// Test refactoring initiated on a Java field rename
IProject project = createProject(TEST_PROJECT);
IJavaProject javaProject = BaseProjectHelper.getJavaProject(project);
assertNotNull(javaProject);
IType type = javaProject.findType("com.example.refactoringtest.R.layout");
if (type == null || !type.exists()) {
type = javaProject.findType("com.example.refactoringtest.R$layout");
System.out.println("Had to switch to $ notation");
}
assertNotNull(type);
assertTrue(type.exists());
IField field = type.getField("activity_main");
assertNotNull(field);
assertTrue(field.exists());
renameResource(
project,
field,
true /*updateReferences*/,
"newlauncher",
"CHANGES:\n" +
"-------\n" +
"[x] Rename 'testRefactor8/res/layout/activity_main.xml' to 'newlauncher.xml'\n" +
"\n" +
"[x] Rename 'testRefactor8/res/layout-land/activity_main.xml' to 'newlauncher.xml'\n" +
"\n" +
"[x] MainActivity.java - /testRefactor8/src/com/example/refactoringtest/MainActivity.java\n" +
" @@ -13 +13\n" +
" - setContentView(R.layout.activity_main);\n" +
" + setContentView(R.layout.newlauncher);\n" +
"\n" +
"\n" +
"[ ] R.java - /testRefactor8/gen/com/example/refactoringtest/R.java\n" +
" @@ -23 +23\n" +
" - public static final int activity_main=0x7f030000;\n" +
" + public static final int newlauncher=0x7f030000;",
null);
}
public void testInvalidName() throws Exception {
renameResource(
TEST_PROJECT,
"@drawable/ic_launcher",
true /*updateReferences*/,
"Newlauncher",
"",
"<ERROR\n" +
"\t\n" +
"ERROR: File-based resource names must start with a lowercase letter.\n" +
"Context: <Unspecified context>\n" +
"code: none\n" +
"Data: null\n" +
">");
}
public void testRefactor9() throws Exception {
// same as testRefactor4, but not updating references
renameResource(
TEST_PROJECT,
"@id/textView1",
false /*updateReferences*/,
"output",
"CHANGES:\n" +
"-------\n" +
"[x] activity_main.xml - /testRefactor9/res/layout/activity_main.xml\n" +
" @@ -8 +8\n" +
" - android:id=\"@+id/textView1\"\n" +
" + android:id=\"@+id/output\"\n" +
"\n" +
"\n" +
"[ ] R.java - /testRefactor9/gen/com/example/refactoringtest/R.java\n" +
" @@ -20 +20\n" +
" - public static final int textView1=0x7f070000;\n" +
" + public static final int output=0x7f070000;");
}
public void testRefactor10() throws Exception {
// Check updating tools: attributes
renameResource(
TEST_PROJECT,
"@layout/preview",
true /*updateReferences*/,
"newlayout",
"CHANGES:\n" +
"-------\n" +
"[x] activity_main.xml - /testRefactor10/res/layout-land/activity_main.xml\n" +
" @@ -10 +10\n" +
" - tools:listitem=\"@layout/preview\" >\n" +
" + tools:listitem=\"@layout/newlayout\" >\n" +
" @@ -17 +17\n" +
" - tools:layout=\"@layout/preview\" />\n" +
" + tools:layout=\"@layout/newlayout\" />");
}
// ---- Test infrastructure ----
protected void renameResource(
@NonNull Object[] testData,
@NonNull Object resource,
boolean updateReferences,
@NonNull String newName,
@NonNull String expected) throws Exception {
renameResource(testData, resource, updateReferences, newName, expected, null);
}
protected void renameResource(
@NonNull Object[] testData,
@NonNull Object resource,
boolean updateReferences,
@NonNull String newName,
@NonNull String expected,
@NonNull String expectedWarnings) throws Exception {
IProject project = createProject(testData);
renameResource(project, resource, updateReferences, newName, expected, expectedWarnings);
}
protected void renameResource(
@NonNull IProject project,
@NonNull Object resource,
boolean updateReferences,
@NonNull String newName,
@NonNull String expected,
@NonNull String expectedWarnings) throws Exception {
RenameProcessor processor = null;
if (resource instanceof String) {
String url = (String) resource;
assert url.startsWith("@") : resource;
ResourceUrl parsedUrl = ResourceUrl.parse(url);
assertNotNull(url, parsedUrl);
assert parsedUrl != null; // null analysis doesn't recognize assertNotNull
ResourceType type = parsedUrl.type;
String currentName = parsedUrl.name;
RenameResourceProcessor p;
p = new RenameResourceProcessor(project, type, currentName, newName);
p.setUpdateReferences(updateReferences);
processor = p;
} else if (resource instanceof IResource) {
IResource r = (IResource) resource;
org.eclipse.ltk.internal.core.refactoring.resource.RenameResourceProcessor p;
p = new org.eclipse.ltk.internal.core.refactoring.resource.RenameResourceProcessor(r);
String fileName = r.getName();
int dot = fileName.indexOf('.');
String extension = (dot != -1) ? fileName.substring(dot) : "";
p.setNewResourceName(newName + extension);
p.setUpdateReferences(updateReferences);
processor = p;
} else if (resource instanceof IField) {
RenameFieldProcessor p = new RenameFieldProcessor((IField) resource);
p.setNewElementName(newName);
p.setUpdateReferences(updateReferences);
processor = p;
} else {
fail("Unsupported resource element in tests: " + resource);
}
assertNotNull(processor);
RenameRefactoring refactoring = new RenameRefactoring(processor);
checkRefactoring(refactoring, expected, expectedWarnings);
}
}