blob: 22d82209232f97eb1461bcfa36aab26b7c9d1131 [file] [log] [blame]
/*
* Copyright (C) 2012 The Android Open Source Project
*
* Licensed under the Apache License, Version 2.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.apache.org/licenses/LICENSE-2.0
*
* 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.tools.lint.checks;
import static com.android.tools.lint.detector.api.TextFormat.TEXT;
import com.android.annotations.NonNull;
import com.android.annotations.Nullable;
import com.android.tools.lint.detector.api.Context;
import com.android.tools.lint.detector.api.Detector;
import com.android.tools.lint.detector.api.Issue;
import com.android.tools.lint.detector.api.Location;
import com.android.tools.lint.detector.api.Severity;
@SuppressWarnings("javadoc")
public class WrongCallDetectorTest extends AbstractCheckTest {
@Override
protected Detector getDetector() {
return new WrongCallDetector();
}
public void test() throws Exception {
assertEquals(""
+ "src/test/pkg/LayoutTest.java:23: Error: Suspicious method call; should probably call \"layout\" rather than \"onLayout\" [WrongCall]\n"
+ " child.onLayout(changed, left, top, right, bottom); // Not OK\n"
+ " ~~~~~~~~\n"
+ "src/test/pkg/LayoutTest.java:25: Error: Suspicious method call; should probably call \"measure\" rather than \"onMeasure\" [WrongCall]\n"
+ " super.onMeasure(0, 0); // Not OK\n"
+ " ~~~~~~~~~\n"
+ "src/test/pkg/LayoutTest.java:26: Error: Suspicious method call; should probably call \"draw\" rather than \"onDraw\" [WrongCall]\n"
+ " super.onDraw(null); // Not OK\n"
+ " ~~~~~~\n"
+ "src/test/pkg/LayoutTest.java:33: Error: Suspicious method call; should probably call \"layout\" rather than \"onLayout\" [WrongCall]\n"
+ " super.onLayout(false, 0, 0, 0, 0); // Not OK\n"
+ " ~~~~~~~~\n"
+ "src/test/pkg/LayoutTest.java:34: Error: Suspicious method call; should probably call \"measure\" rather than \"onMeasure\" [WrongCall]\n"
+ " child.onMeasure(widthMeasureSpec, heightMeasureSpec); // Not OK\n"
+ " ~~~~~~~~~\n"
+ "src/test/pkg/LayoutTest.java:41: Error: Suspicious method call; should probably call \"draw\" rather than \"onDraw\" [WrongCall]\n"
+ " child.onDraw(canvas); // Not OK\n"
+ " ~~~~~~\n"
+ "6 errors, 0 warnings\n",
lintProject(
"bytecode/.classpath=>.classpath",
"bytecode/AndroidManifest.xml=>AndroidManifest.xml",
"res/layout/onclick.xml=>res/layout/onclick.xml",
"bytecode/LayoutTest.java.txt=>src/test/pkg/LayoutTest.java",
"bytecode/LayoutTest.class.data=>bin/classes/test/pkg/LayoutTest.class"
));
}
public void testGetOldValue() {
assertEquals("onLayout", WrongCallDetector.getOldValue(
"Suspicious method call; should probably call \"layout\" rather than \"onLayout\"",
TEXT));
}
public void testGetNewValue() {
assertEquals("layout", WrongCallDetector.getNewValue(
"Suspicious method call; should probably call \"layout\" rather than \"onLayout\"",
TEXT));
}
@Override
protected void checkReportedError(@NonNull Context context, @NonNull Issue issue,
@NonNull Severity severity, @Nullable Location location, @NonNull String message) {
assertNotNull(message, WrongCallDetector.getOldValue(message, TEXT));
assertNotNull(message, WrongCallDetector.getNewValue(message, TEXT));
}
}