fix syntax errors in test files to make sure we are testing the correct thing

GitOrigin-RevId: 09bf5fde01489c9edbc233e586fe7137f7e3fe9b
This commit is contained in:
Alexey Kudravtsev
2023-09-26 12:59:10 +02:00
committed by intellij-monorepo-bot
parent 6f8350fd7a
commit 67f7db108e
21 changed files with 47 additions and 22 deletions

View File

@@ -2,7 +2,7 @@
@SuppressWarnings("synchronization")
class Main {
final OpenValueBased vb = new OpenValueBased();
final OpenValueBased vb = new OpenValueBased(){};
{
synchronized(vb){ }

View File

@@ -2,7 +2,7 @@
@SuppressWarnings("synchronization")
class Main {
final OpenValueBased vb = new OpenValueBased();
final OpenValueBased vb = new OpenValueBased(){};
void f(){
new OpenValueBased() {

View File

@@ -2,7 +2,7 @@
@SuppressWarnings("synchronization")
class Main {
final OpenValueBased vb = new OpenValueBased();
final OpenValueBased vb = new OpenValueBased(){};
void f(){
var l = ((new OpenValueBased() {

View File

@@ -4,7 +4,7 @@
class Main {
final OpenValueBased vb = new OpenValueBased();
void f{
void f(){
synchronized(vb){ }
}
}

View File

@@ -1,7 +1,7 @@
// "Suppress for statement" "true"
class Main {
final OpenValueBased vb = new OpenValueBased();
final OpenValueBased vb = new OpenValueBased(){};
void f(){
var l = ((new OpenValueBased() {

View File

@@ -1,7 +1,7 @@
// "Suppress for statement" "true"
class Main {
final OpenValueBased vb = new OpenValueBased();
final OpenValueBased vb = new OpenValueBased(){};
void f(){
new OpenValueBased() {

View File

@@ -1,7 +1,7 @@
// "Suppress for method" "true"
class Main {
final OpenValueBased vb = new OpenValueBased();
final OpenValueBased vb = new OpenValueBased(){};
@SuppressWarnings("synchronization")
void f(){

View File

@@ -1,7 +1,7 @@
// "Suppress for method" "true"
class Main {
final OpenValueBased vb = new OpenValueBased();
final OpenValueBased vb = new OpenValueBased(){};
@SuppressWarnings("synchronization")
void f(){

View File

@@ -1,7 +1,7 @@
// "Suppress for method" "true"
class Main {
final OpenValueBased vb = new OpenValueBased();
final OpenValueBased vb = new OpenValueBased(){};
@SuppressWarnings("synchronization")
void f(){

View File

@@ -1,13 +1,17 @@
// "Suppress for method" "true"
class Main {
final OpenValueBased vb = new OpenValueBased();
final OpenValueBased vb = new OpenValueBased(){
@Override
public void g() {
}
};
void f(){
new OpenValueBased() {
@SuppressWarnings("synchronization")
@Override
void g() {
public void g() {
synchronized (vb){ }
}
};

View File

@@ -1,7 +1,7 @@
// "Suppress for class" "true"
class Main {
final OpenValueBased vb = new OpenValueBased();
final OpenValueBased vb = new OpenValueBased(){};
{
synchronized(<caret>vb){ }

View File

@@ -1,7 +1,7 @@
// "Suppress for class" "true"
class Main {
final OpenValueBased vb = new OpenValueBased();
final OpenValueBased vb = new OpenValueBased(){};
void f(){
new OpenValueBased() {

View File

@@ -1,7 +1,7 @@
// "Suppress for class" "true"
class Main {
final OpenValueBased vb = new OpenValueBased();
final OpenValueBased vb = new OpenValueBased(){};
void f(){
var l = ((new OpenValueBased() {

View File

@@ -3,7 +3,7 @@
class Main {
final OpenValueBased vb = new OpenValueBased();
void f{
void f(){
synchronized(<caret>vb){ }
}
}

View File

@@ -1,7 +1,7 @@
// "Suppress for statement" "true"
class Main {
final OpenValueBased vb = new OpenValueBased();
final OpenValueBased vb = new OpenValueBased(){};
void f(){
var l = ((new OpenValueBased() {

View File

@@ -1,7 +1,7 @@
// "Suppress for statement" "true"
class Main {
final OpenValueBased vb = new OpenValueBased();
final OpenValueBased vb = new OpenValueBased(){};
void f(){
new OpenValueBased() {

View File

@@ -1,7 +1,7 @@
// "Suppress for method" "true"
class Main {
final OpenValueBased vb = new OpenValueBased();
final OpenValueBased vb = new OpenValueBased(){};
void f(){
synchronized(<caret>vb){ }

View File

@@ -1,7 +1,7 @@
// "Suppress for method" "true"
class Main {
final OpenValueBased vb = new OpenValueBased();
final OpenValueBased vb = new OpenValueBased(){};
void f(){
new OpenValueBased() {

View File

@@ -1,7 +1,7 @@
// "Suppress for method" "true"
class Main {
final OpenValueBased vb = new OpenValueBased();
final OpenValueBased vb = new OpenValueBased(){};
void f(){
var l = ((new OpenValueBased() {

View File

@@ -1,12 +1,16 @@
// "Suppress for method" "true"
class Main {
final OpenValueBased vb = new OpenValueBased();
final OpenValueBased vb = new OpenValueBased(){
@Override
public void g() {
}
};
void f(){
new OpenValueBased() {
@Override
void g() {
public void g() {
synchronized (<caret>vb){ }
}
};

View File

@@ -4,10 +4,23 @@ package com.intellij.java.codeInsight.daemon.valuebased;
import com.intellij.codeInsight.daemon.quickFix.LightQuickFixParameterizedTestCase;
import com.intellij.codeInspection.LocalInspectionTool;
import com.intellij.codeInspection.valuebased.SynchronizeOnValueBasedClassInspection;
import com.intellij.lang.annotation.HighlightSeverity;
import org.intellij.lang.annotations.Language;
import org.jetbrains.annotations.NonNls;
import org.jetbrains.annotations.NotNull;
public class SynchronizeOnValueBasedClassFixTest extends LightQuickFixParameterizedTestCase {
@Override
protected void setUp() throws Exception {
super.setUp();
@Language("JAVA")
String valueBased = """
package jdk.internal;
public @interface ValueBased {
}
""";
createAndSaveFile("jdk/internal/ValueBased.java", valueBased);
}
@Override
protected @NonNls String getBasePath() {
@@ -19,4 +32,8 @@ public class SynchronizeOnValueBasedClassFixTest extends LightQuickFixParameteri
return new LocalInspectionTool[]{ new SynchronizeOnValueBasedClassInspection() };
}
@Override
protected void beforeActionStarted(String testName, String contents) {
assertEmpty(doHighlighting(HighlightSeverity.ERROR));
}
}