The following document contains the results of Checkstyle.
Files | Infos | Warnings | Errors |
---|---|---|---|
20 | 104 | 8 | 157 |
Rules | Violations | Severity |
---|---|---|
JavadocPackage | 3 | Error |
Translation | 0 | Error |
FileLength | 0 | Error |
FileTabCharacter
|
0 | Error |
RegexpSingleline
|
2 | Warning |
AnnotationUseStyle | 0 | Error |
MissingDeprecated | 0 | Error |
MissingOverride
|
0 | Error |
PackageAnnotation | 0 | Error |
JavadocMethod
|
2 | Error |
JavadocType
|
16 | Error |
JavadocVariable
|
11 | Error |
JavadocStyle
|
2 | Error |
ConstantName | 0 | Error |
LocalFinalVariableName | 0 | Error |
LocalVariableName | 19 | Error |
MemberName | 8 | Error |
MethodName | 0 | Error |
PackageName | 0 | Error |
ParameterName | 6 | Error |
StaticVariableName | 0 | Error |
TypeName | 0 | Error |
AvoidStarImport | 0 | Error |
IllegalImport | 0 | Error |
ImportOrder
|
38 | Error |
RedundantImport | 0 | Error |
UnusedImports | 1 | Error |
LineLength
|
6 | Warning |
MethodLength | 0 | Error |
ParameterNumber | 0 | Error |
EmptyForIteratorPad | 0 | Error |
MethodParamPad | 0 | Error |
NoWhitespaceAfter | 0 | Error |
NoWhitespaceBefore | 0 | Error |
OperatorWrap | 2 | Error |
ParenPad | 0 | Error |
TypecastParenPad | 0 | Error |
WhitespaceAfter | 0 | Error |
WhitespaceAround | 2 | Error |
ModifierOrder | 2 | Error |
RedundantModifier | 2 | Error |
AvoidNestedBlocks | 0 | Error |
EmptyBlock | 0 | Error |
LeftCurly
|
3 | Error |
NeedBraces | 4 | Error |
RightCurly
|
2 | Error |
DeclarationOrder | 6 | Error |
DefaultComesLast | 0 | Error |
DoubleCheckedLocking | 0 | Error |
EmptyStatement | 0 | Error |
EqualsHashCode | 0 | Error |
ExplicitInitialization | 0 | Error |
FallThrough | 0 | Error |
HiddenField
|
0 | Error |
IllegalInstantiation | 0 | Error |
InnerAssignment | 0 | Error |
MagicNumber | 12 | Error |
MissingSwitchDefault | 0 | Error |
ParameterAssignment | 0 | Error |
RedundantThrows
|
0 | Error |
SimplifyBooleanExpression | 0 | Error |
SimplifyBooleanReturn | 0 | Error |
StringLiteralEquality | 0 | Error |
DesignForExtension | 104 | Info |
FinalClass | 0 | Error |
HideUtilityClassConstructor | 1 | Error |
InterfaceIsType | 0 | Error |
VisibilityModifier | 11 | Error |
BooleanExpressionComplexity | 0 | Error |
ClassDataAbstractionCoupling | 0 | Error |
ClassFanOutComplexity
|
2 | Error |
CyclomaticComplexity | 0 | Error |
NPathComplexity
|
0 | Error |
JavaNCSS | 0 | Error |
ArrayTypeStyle | 0 | Error |
Indentation | 0 | Error |
TodoComment | 2 | Error |
UpperEll | 0 | Error |
Violation | Message | Line |
---|---|---|
'org.openqa.selenium.By' should be separated from previous imports. | 6 | |
Wrong order for 'javax.inject.Inject' import. | 11 | |
Wrong order for 'java.net.URI' import. | 12 | |
Missing a Javadoc comment. | 14 | |
Missing a Javadoc comment. | 16 | |
Variable 'driver' must be private and have accessor methods. | 17 | |
Missing a Javadoc comment. | 19 | |
Variable 'pageBinder' must be private and have accessor methods. | 20 | |
Missing a Javadoc comment. | 22 | |
Variable 'dialog' must be private and have accessor methods. | 23 | |
'{' is not followed by whitespace. | 25 | |
'}' is not preceded with whitespace. | 25 | |
Missing a Javadoc comment. | 27 | |
Missing a Javadoc comment. | 43 | |
Line is longer than 120 characters. | 56 |
Violation | Message | Line |
---|---|---|
'com.google.common.base.Function' should be separated from previous imports. | 7 | |
'org.openqa.selenium.By' should be separated from previous imports. | 8 | |
Wrong order for 'java.util.ArrayList' import. | 13 | |
'javax.inject.Inject' should be separated from previous imports. | 15 | |
Method 'waitForVisible' is not designed for extension - needs to be abstract, final or empty. | 54 | |
Method 'searchFor' is not designed for extension - needs to be abstract, final or empty. | 60 | |
Method 'waitFor' is not designed for extension - needs to be abstract, final or empty. | 66 | |
Method 'isGadgetVisible' is not designed for extension - needs to be abstract, final or empty. | 72 | |
Method 'getVisibleGadgets' is not designed for extension - needs to be abstract, final or empty. | 77 | |
Method 'setCategory' is not designed for extension - needs to be abstract, final or empty. | 87 | |
':' should be on a new line. | 95 | |
Method 'addGadget' is not designed for extension - needs to be abstract, final or empty. | 104 | |
Line is longer than 120 characters. | 106 | |
Missing a Javadoc comment. | 110 | |
Method 'addGadgetToDirectory' is not designed for extension - needs to be abstract, final or empty. | 118 | |
Method 'finished' is not designed for extension - needs to be abstract, final or empty. | 134 |
Violation | Message | Line |
---|---|---|
Wrong order for 'org.openqa.selenium.By' import. | 10 | |
Wrong order for 'java.util.List' import. | 13 | |
First sentence should end with a period. | 15 | |
Comment matches to-do format 'TODO:'. | 16 | |
Method 'getConfig' is not designed for extension - needs to be abstract, final or empty. | 27 | |
Method 'getConfigTriggerCss' is not designed for extension - needs to be abstract, final or empty. | 33 | |
Method 'getFieldWithName' is not designed for extension - needs to be abstract, final or empty. | 38 | |
Name '$fields' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 41 | |
Name '$field' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 43 | |
Name '$container' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 65 |
Violation | Message | Line |
---|---|---|
Wrong order for 'org.openqa.selenium.By' import. | 8 | |
Wrong order for 'java.util.ArrayList' import. | 11 | |
Missing a Javadoc comment. | 14 | |
Method 'configure' is not designed for extension - needs to be abstract, final or empty. | 22 | |
Method 'submit' is not designed for extension - needs to be abstract, final or empty. | 30 | |
Method 'getConfigTriggerCss' is not designed for extension - needs to be abstract, final or empty. | 37 | |
Method 'getConfig' is not designed for extension - needs to be abstract, final or empty. | 42 | |
Method 'getFields' is not designed for extension - needs to be abstract, final or empty. | 48 | |
Name '$inputs' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 50 | |
Name '$fields' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 53 | |
Name '$input' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 55 | |
Method 'getFieldWithName' is not designed for extension - needs to be abstract, final or empty. | 66 | |
Name '$fields' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 69 | |
Name '$field' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 71 | |
Name '$container' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 89 |
Violation | Message | Line |
---|---|---|
'com.google.common.base.Function' should be separated from previous imports. | 6 | |
'org.openqa.selenium.By' should be separated from previous imports. | 7 | |
Wrong order for 'javax.inject.Inject' import. | 12 | |
Wrong order for 'java.util.List' import. | 13 | |
Missing a Javadoc comment. | 15 | |
Missing a Javadoc comment. | 23 | |
Variable access definition in wrong order. | 23 | |
Variable 'addDialogLink' must be private and have accessor methods. | 24 | |
Method 'getUrl' is not designed for extension - needs to be abstract, final or empty. | 26 | |
Method 'openAddApplicationLinkDialog' is not designed for extension - needs to be abstract, final or empty. | 31 | |
Method 'deleteAppLink' is not designed for extension - needs to be abstract, final or empty. | 37 | |
Missing a Javadoc comment. | 44 | |
Line is longer than 120 characters. | 46 | |
Line is longer than 120 characters. | 47 |
Violation | Message | Line |
---|---|---|
'org.apache.commons.lang.math.RandomUtils' should be separated from previous imports. | 9 | |
Wrong order for 'javax.inject.Inject' import. | 11 | |
Wrong order for 'java.util.Locale' import. | 12 | |
Missing a Javadoc comment. | 14 | |
Static variable definition in wrong order. | 23 | |
'private' modifier out of order with the JLS suggestions. | 23 | |
Static variable definition in wrong order. | 24 | |
'private' modifier out of order with the JLS suggestions. | 24 | |
Method 'logout' is not designed for extension - needs to be abstract, final or empty. | 26 | |
Method 'login' is not designed for extension - needs to be abstract, final or empty. | 33 | |
Method 'getDashboard' is not designed for extension - needs to be abstract, final or empty. | 39 | |
'50000' is a magic number. | 41 | |
Line is longer than 120 characters. | 44 | |
Method 'getDashboard' is not designed for extension - needs to be abstract, final or empty. | 44 | |
Method 'getBlankDashboard' is not designed for extension - needs to be abstract, final or empty. | 50 | |
'998' is a magic number. | 52 | |
Method 'getReadWriteDashboard' is not designed for extension - needs to be abstract, final or empty. | 55 | |
Method 'getReadOnlyDashboard' is not designed for extension - needs to be abstract, final or empty. | 60 | |
'998' is a magic number. | 62 | |
Method 'getUsersOnlyDashboard' is not designed for extension - needs to be abstract, final or empty. | 65 | |
'998' is a magic number. | 67 | |
Method 'getBlankDashboard' is not designed for extension - needs to be abstract, final or empty. | 70 | |
'998' is a magic number. | 72 | |
Method 'switchTo' is not designed for extension - needs to be abstract, final or empty. | 75 | |
Method 'getUrl' is not designed for extension - needs to be abstract, final or empty. | 81 |
Violation | Message | Line |
---|---|---|
'org.openqa.selenium.By' should be separated from previous imports. | 5 | |
Wrong order for 'java.util.ArrayList' import. | 9 | |
'javax.inject.Inject' should be separated from previous imports. | 11 | |
Method 'getToolLinkNames' is not designed for extension - needs to be abstract, final or empty. | 24 | |
Method 'open' is not designed for extension - needs to be abstract, final or empty. | 34 |
Violation | Message | Line |
---|---|---|
'org.openqa.selenium.By' should be separated from previous imports. | 4 | |
Wrong order for 'javax.inject.Inject' import. | 6 | |
Missing a Javadoc comment. | 8 | |
Method 'setLayout' is not designed for extension - needs to be abstract, final or empty. | 14 |
Violation | Message | Line |
---|---|---|
Missing package-info.java file. | 0 | |
'org.openqa.selenium.By' should be separated from previous imports. | 10 | |
Wrong order for 'javax.inject.Inject' import. | 14 | |
Wrong order for 'java.util.List' import. | 15 | |
Class Fan-Out Complexity is 13 (max allowed is 10). | 28 | |
Missing a Javadoc comment. | 30 | |
Variable 'driver' must be private and have accessor methods. | 31 | |
Missing a Javadoc comment. | 33 | |
Variable 'pageBinder' must be private and have accessor methods. | 34 | |
Missing a Javadoc comment. | 36 | |
Variable 'finder' must be private and have accessor methods. | 37 | |
Missing a Javadoc comment. | 39 | |
Variable 'actions' must be private and have accessor methods. | 40 | |
Method 'getChrome' is not designed for extension - needs to be abstract, final or empty. | 49 | |
Method 'getChromeWebElement' is not designed for extension - needs to be abstract, final or empty. | 55 | |
Method 'getMain' is not designed for extension - needs to be abstract, final or empty. | 61 | |
Method 'getMainWebElement' is not designed for extension - needs to be abstract, final or empty. | 67 | |
Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 73 | |
Method 'getTitleBar' is not designed for extension - needs to be abstract, final or empty. | 78 | |
Method 'getTitleBarWebElement' is not designed for extension - needs to be abstract, final or empty. | 83 | |
Method 'minimize' is not designed for extension - needs to be abstract, final or empty. | 88 | |
Method 'openMenu' is not designed for extension - needs to be abstract, final or empty. | 94 | |
Method 'getHtml' is not designed for extension - needs to be abstract, final or empty. | 100 | |
Method 'getText' is not designed for extension - needs to be abstract, final or empty. | 108 | |
Method 'maximize' is not designed for extension - needs to be abstract, final or empty. | 114 | |
Method 'dragTo' is not designed for extension - needs to be abstract, final or empty. | 120 | |
Method 'dragTo' is not designed for extension - needs to be abstract, final or empty. | 125 | |
Method 'dragTo' is not designed for extension - needs to be abstract, final or empty. | 130 | |
'100' is a magic number. | 135 | |
'100' is a magic number. | 135 | |
Method 'isMinimized' is not designed for extension - needs to be abstract, final or empty. | 141 | |
Method 'getMainText' is not designed for extension - needs to be abstract, final or empty. | 146 | |
Method 'changeColor' is not designed for extension - needs to be abstract, final or empty. | 151 | |
Method 'getErrorMsgText' is not designed for extension - needs to be abstract, final or empty. | 157 | |
Method 'getColor' is not designed for extension - needs to be abstract, final or empty. | 163 | |
Method 'switchToIframe' is not designed for extension - needs to be abstract, final or empty. | 168 | |
First sentence should end with a period. | 175 | |
Method 'switchTo' is not designed for extension - needs to be abstract, final or empty. | 180 | |
Method 'switchTo' is not designed for extension - needs to be abstract, final or empty. | 187 | |
Method 'switchBack' is not designed for extension - needs to be abstract, final or empty. | 197 | |
Method 'delete' is not designed for extension - needs to be abstract, final or empty. | 202 | |
Method 'find' is not designed for extension - needs to be abstract, final or empty. | 210 | |
Method 'find' is not designed for extension - needs to be abstract, final or empty. | 216 | |
Method 'findAll' is not designed for extension - needs to be abstract, final or empty. | 222 | |
Method 'findAll' is not designed for extension - needs to be abstract, final or empty. | 228 | |
Method 'find' is not designed for extension - needs to be abstract, final or empty. | 234 | |
Method 'find' is not designed for extension - needs to be abstract, final or empty. | 240 | |
Method 'findAll' is not designed for extension - needs to be abstract, final or empty. | 246 | |
Method 'findAll' is not designed for extension - needs to be abstract, final or empty. | 252 |
Violation | Message | Line |
---|---|---|
Unused import - com.atlassian.webdriver.utils.by.ByJquery. | 4 | |
'org.openqa.selenium.By' should be separated from previous imports. | 5 | |
Wrong order for 'javax.inject.Inject' import. | 9 | |
Missing a Javadoc comment. | 11 | |
Method 'getGadgetMenu' is not designed for extension - needs to be abstract, final or empty. | 24 | |
Method 'getMenuTrigger' is not designed for extension - needs to be abstract, final or empty. | 29 | |
Method 'getTitleBar' is not designed for extension - needs to be abstract, final or empty. | 34 | |
Method 'getContainer' is not designed for extension - needs to be abstract, final or empty. | 39 | |
Line has trailing spaces. | 43 | |
Method 'open' is not designed for extension - needs to be abstract, final or empty. | 44 | |
'-10' is a magic number. | 48 | |
'-10' is a magic number. | 48 | |
'10' is a magic number. | 49 | |
'10' is a magic number. | 49 | |
Method 'get' is not designed for extension - needs to be abstract, final or empty. | 57 | |
Method 'selectByClassName' is not designed for extension - needs to be abstract, final or empty. | 62 | |
Method 'selectByCSSSelector' is not designed for extension - needs to be abstract, final or empty. | 68 |
Violation | Message | Line |
---|---|---|
'it.com.atlassian.gadgets.pages.util.DashboardGadgets' should be separated from previous imports. | 9 | |
Wrong order for 'org.openqa.selenium.By' import. | 10 | |
Wrong order for 'javax.inject.Inject' import. | 14 | |
Wrong order for 'java.util.ArrayList' import. | 15 | |
Missing a Javadoc comment. | 19 | |
Class Fan-Out Complexity is 13 (max allowed is 10). | 19 | |
Missing a Javadoc comment. | 25 | |
Variable access definition in wrong order. | 25 | |
Variable 'atlassianWebDriver' must be private and have accessor methods. | 26 | |
Missing a Javadoc comment. | 28 | |
Variable access definition in wrong order. | 28 | |
Variable 'pageBinder' must be private and have accessor methods. | 29 | |
Comment matches to-do format 'TODO:'. | 34 | |
Missing a Javadoc comment. | 35 | |
Variable access definition in wrong order. | 35 | |
Variable 'dashboardPageElement' must be private and have accessor methods. | 36 | |
'50000' is a magic number. | 53 | |
Method 'getColumn' is not designed for extension - needs to be abstract, final or empty. | 56 | |
'{' should be on a new line. | 60 | |
'}' should be alone on a line. | 62 | |
'{' should be on a new line. | 62 | |
'}' should be alone on a line. | 64 | |
'{' should be on a new line. | 64 | |
Method 'refresh' is not designed for extension - needs to be abstract, final or empty. | 71 | |
Method 'getGadgetsForColumn' is not designed for extension - needs to be abstract, final or empty. | 77 | |
Name '$gadgets' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 80 | |
Name '$gadget' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 82 | |
Method 'getUrl' is not designed for extension - needs to be abstract, final or empty. | 91 | |
Line is longer than 120 characters. | 93 | |
'+' should be on a new line. | 93 | |
Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 97 | |
Method 'openToolsMenu' is not designed for extension - needs to be abstract, final or empty. | 102 | |
Method 'waitUntilDashboardRendered' is not designed for extension - needs to be abstract, final or empty. | 107 | |
Method 'openAddGadgetDialog' is not designed for extension - needs to be abstract, final or empty. | 113 | |
Method 'openEditLayoutDialog' is not designed for extension - needs to be abstract, final or empty. | 119 | |
Method 'getGadgetWithTitle' is not designed for extension - needs to be abstract, final or empty. | 125 | |
Method 'getLayout' is not designed for extension - needs to be abstract, final or empty. | 130 | |
Method 'getGadgetWithTitle' is not designed for extension - needs to be abstract, final or empty. | 137 | |
Method 'canAddGadget' is not designed for extension - needs to be abstract, final or empty. | 142 | |
Line has trailing spaces. | 146 | |
Method 'canEditLayout' is not designed for extension - needs to be abstract, final or empty. | 147 | |
Method 'getGadget' is not designed for extension - needs to be abstract, final or empty. | 153 |
Violation | Message | Line |
---|---|---|
Wrong order for 'java.util.ArrayList' import. | 5 | |
Missing a Javadoc comment. | 10 | |
Name '$fields' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 13 | |
Name '$field' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 15 | |
Name '$field' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 21 | |
Method 'set' is not designed for extension - needs to be abstract, final or empty. | 26 | |
Name '$checkbox' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 30 | |
'if' construct must use '{}'s. | 34 | |
Method 'getValue' is not designed for extension - needs to be abstract, final or empty. | 45 | |
Name '$field' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 49 |
Violation | Message | Line |
---|---|---|
Wrong order for 'java.util.List' import. | 6 | |
Missing a Javadoc comment. | 9 | |
Name '$field' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 12 | |
Name '$field' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 14 | |
Method 'set' is not designed for extension - needs to be abstract, final or empty. | 19 | |
Name '$options' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 21 | |
Name '$option' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 23 | |
'if' construct must use '{}'s. | 27 | |
Method 'getValue' is not designed for extension - needs to be abstract, final or empty. | 37 |
Violation | Message | Line |
---|---|---|
Wrong order for 'java.util.List' import. | 6 | |
Missing a Javadoc comment. | 9 | |
Name '$container' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 13 | |
Name '$inputField' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 14 | |
Name '$submit' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 15 | |
Name '$container' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 17 | |
Method 'clear' is not designed for extension - needs to be abstract, final or empty. | 24 | |
Name '$removeButtons' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 26 | |
Method 'getValue' is not designed for extension - needs to be abstract, final or empty. | 37 | |
Method 'set' is not designed for extension - needs to be abstract, final or empty. | 42 |
Violation | Message | Line |
---|---|---|
Wrong order for 'java.util.Arrays' import. | 6 | |
Missing a Javadoc comment. | 10 | |
Name '$field' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 12 | |
Method 'set' is not designed for extension - needs to be abstract, final or empty. | 21 | |
'if' construct must use '{}'s. | 29 | |
Method 'getValue' is not designed for extension - needs to be abstract, final or empty. | 40 | |
Name '$option' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 44 |
Violation | Message | Line |
---|---|---|
Wrong order for 'java.util.Arrays' import. | 5 | |
Missing a Javadoc comment. | 8 | |
Name '$inputs' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 10 | |
Name '$inputs' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 12 | |
Method 'set' is not designed for extension - needs to be abstract, final or empty. | 17 | |
Name '$radio' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 21 | |
'if' construct must use '{}'s. | 25 | |
Method 'getValue' is not designed for extension - needs to be abstract, final or empty. | 32 | |
Name '$radio' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 36 |
Violation | Message | Line |
---|---|---|
Missing a Javadoc comment. | 6 | |
Name '$field' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 9 | |
Name '$field' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 11 | |
Method 'set' is not designed for extension - needs to be abstract, final or empty. | 16 | |
Method 'getValue' is not designed for extension - needs to be abstract, final or empty. | 23 |
Violation | Message | Line |
---|---|---|
Missing package-info.java file. | 0 | |
Missing a Javadoc comment. | 4 | |
Redundant 'public' modifier. | 7 | |
Redundant 'public' modifier. | 9 |