enum constants visibility (IDEADEV-22700)

inspections/impl/com/intellij/codeInspection/visibility/VisibilityInspection.java

@@ -132,6 +132,8 @@ public class VisibilityInspection extends GlobalInspectionTool {
       //ignore implicit constructors. User should not be able to see them.
       if (refElement instanceof RefImplicitConstructor) return null;
 
+      if (refElement instanceof RefField && ((RefField)refElement).getElement() instanceof PsiEnumConstant) return null;
+
       //ignore library override methods.
       if (refElement instanceof RefMethod) {
         RefMethod refMethod = (RefMethod) refElement;

inspections/tests/com/intellij/codeInspection/VisibilityInspectionTest.java

@@ -74,4 +74,11 @@ public class VisibilityInspectionTest extends InspectionTestCase {
     myTool.SUGGEST_PRIVATE_FOR_INNERS = false;
     doTest("visibility/implicitConstructor", myTool, false, true);
   }
+
+  public void testEnumConstants() throws Exception {
+    myTool.SUGGEST_PACKAGE_LOCAL_FOR_MEMBERS = true;
+    myTool.SUGGEST_PACKAGE_LOCAL_FOR_TOP_CLASSES = true;
+    myTool.SUGGEST_PRIVATE_FOR_INNERS = false;
+    doTest("visibility/enumConstantsVisibility", myTool, false, true);
+  }
 }

testData/inspection/visibility/enumConstantsVisibility/expected.xml

@@ -0,0 +1,4 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<problems>  
+</problems>
+

testData/inspection/visibility/enumConstantsVisibility/src/Foo.java

@@ -0,0 +1,7 @@
+public enum Foo {
+  BAR, UNUSED;
+  
+  public static void main(String[] args){
+    System.out.println(BAR);
+  }
+}