The following document contains the results of Checkstyle
. 
| Files | Infos  |
Warnings  |
Errors  |
|---|---|---|---|
| 11 | 0 | 0 | 539 |
| Rules | Violations | Severity |
|---|---|---|
| PackageHtml | 1 | Error |
| NewlineAtEndOfFile | 1 | Error |
| Translation | 0 | Error |
| JavadocMethod | 66 | Error |
| JavadocType | 0 | Error |
| JavadocVariable | 14 | Error |
| JavadocStyle | 18 | Error |
| ConstantName | 3 | Error |
| LocalFinalVariableName | 0 | Error |
| LocalVariableName | 0 | Error |
| MemberName | 2 | Error |
| MethodName | 0 | Error |
| PackageName | 0 | Error |
| ParameterName | 0 | Error |
| StaticVariableName | 0 | Error |
| TypeName | 0 | Error |
| AvoidStarImport | 0 | Error |
| IllegalImport | 0 | Error |
| RedundantImport | 0 | Error |
| UnusedImports | 0 | Error |
| FileLength | 0 | Error |
| LineLength | 97 | Error |
| MethodLength | 0 | Error |
| ParameterNumber | 0 | Error |
| EmptyForIteratorPad | 0 | Error |
| MethodParamPad | 0 | Error |
| NoWhitespaceAfter | 0 | Error |
| NoWhitespaceBefore | 0 | Error |
| OperatorWrap | 9 | Error |
| ParenPad | 0 | Error |
| TypecastParenPad | 0 | Error |
| TabCharacter | 0 | Error |
| WhitespaceAfter | 0 | Error |
| WhitespaceAround | 17 | Error |
| ModifierOrder | 0 | Error |
| RedundantModifier | 0 | Error |
| AvoidNestedBlocks | 0 | Error |
| EmptyBlock | 0 | Error |
| LeftCurly | 121 | Error |
| NeedBraces | 3 | Error |
| RightCurly | 5 | Error |
| AvoidInlineConditionals | 0 | Error |
| DoubleCheckedLocking | 0 | Error |
| EmptyStatement | 0 | Error |
| EqualsHashCode | 0 | Error |
| HiddenField | 3 | Error |
| IllegalInstantiation | 0 | Error |
| InnerAssignment | 0 | Error |
| MagicNumber | 3 | Error |
| MissingSwitchDefault | 0 | Error |
| RedundantThrows | 0 | Error |
| SimplifyBooleanExpression | 0 | Error |
| SimplifyBooleanReturn | 0 | Error |
| DesignForExtension | 58 | Error |
| FinalClass | 1 | Error |
| HideUtilityClassConstructor | 1 | Error |
| InterfaceIsType | 0 | Error |
| VisibilityModifier | 5 | Error |
| ArrayTypeStyle | 0 | Error |
| FinalParameters | 105 | Error |
GenericIllegalRegexp
|
6 | Error |
| TodoComment | 0 | Error |
| UpperEll | 0 | Error |
| Violation | Message | Line |
|---|---|---|
|
'{' should be on the previous line. | 13 |
|
Line is longer than 80 characters. | 14 |
|
Missing a Javadoc comment. | 14 |
|
Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 14 |
|
Missing a Javadoc comment. | 16 |
|
Missing a Javadoc comment. | 17 |
|
Missing a Javadoc comment. | 18 |
|
Missing a Javadoc comment. | 20 |
|
Parameter client should be final. | 20 |
|
'client' hides a field. | 20 |
|
Parameter config should be final. | 20 |
|
'{' should be on the previous line. | 21 |
|
Method 'visibleByTimeout' is not designed for extension - needs to be abstract, final or empty. | 27 |
|
Missing a Javadoc comment. | 27 |
|
Parameter locator should be final. | 27 |
|
'{' should be on the previous line. | 28 |
|
Line is longer than 80 characters. | 33 |
|
Method 'visibleByTimeout' is not designed for extension - needs to be abstract, final or empty. | 39 |
|
Parameter locator should be final. | 39 |
|
Parameter maxMillis should be final. | 39 |
|
'{' should be on the previous line. | 40 |
|
Method 'notVisibleByTimeout' is not designed for extension - needs to be abstract, final or empty. | 44 |
|
Missing a Javadoc comment. | 44 |
|
Parameter locator should be final. | 44 |
|
'{' should be on the previous line. | 45 |
|
Method 'notVisibleByTimeout' is not designed for extension - needs to be abstract, final or empty. | 49 |
|
Missing a Javadoc comment. | 49 |
|
Parameter locator should be final. | 49 |
|
Parameter maxMillis should be final. | 49 |
|
'{' should be on the previous line. | 50 |
|
Method 'elementPresentByTimeout' is not designed for extension - needs to be abstract, final or empty. | 54 |
|
Missing a Javadoc comment. | 54 |
|
Parameter locator should be final. | 54 |
|
'{' should be on the previous line. | 55 |
|
Method 'elementPresentByTimeout' is not designed for extension - needs to be abstract, final or empty. | 59 |
|
Missing a Javadoc comment. | 59 |
|
Parameter locator should be final. | 59 |
|
Parameter maxMillis should be final. | 59 |
|
'{' should be on the previous line. | 60 |
|
Method 'elementPresentUntilTimeout' is not designed for extension - needs to be abstract, final or empty. | 64 |
|
Missing a Javadoc comment. | 64 |
|
Parameter locator should be final. | 64 |
|
'{' should be on the previous line. | 65 |
|
Method 'elementPresentUntilTimeout' is not designed for extension - needs to be abstract, final or empty. | 69 |
|
Missing a Javadoc comment. | 69 |
|
Parameter locator should be final. | 69 |
|
Parameter maxMillis should be final. | 69 |
|
'{' should be on the previous line. | 70 |
|
Method 'elementNotPresentByTimeout' is not designed for extension - needs to be abstract, final or empty. | 74 |
|
Missing a Javadoc comment. | 74 |
|
Parameter locator should be final. | 74 |
|
'{' should be on the previous line. | 75 |
|
Method 'elementNotPresentUntilTimeout' is not designed for extension - needs to be abstract, final or empty. | 81 |
|
Missing a Javadoc comment. | 81 |
|
Parameter locator should be final. | 81 |
|
'{' should be on the previous line. | 82 |
|
Method 'elementNotPresentUntilTimeout' is not designed for extension - needs to be abstract, final or empty. | 87 |
|
Missing a Javadoc comment. | 87 |
|
Parameter locator should be final. | 87 |
|
Parameter maxMillis should be final. | 87 |
|
'{' should be on the previous line. | 88 |
|
Method 'textPresentByTimeout' is not designed for extension - needs to be abstract, final or empty. | 94 |
|
Missing a Javadoc comment. | 94 |
|
Parameter text should be final. | 94 |
|
Parameter maxMillis should be final. | 94 |
|
'{' should be on the previous line. | 95 |
|
Method 'textPresentByTimeout' is not designed for extension - needs to be abstract, final or empty. | 99 |
|
Missing a Javadoc comment. | 99 |
|
Parameter text should be final. | 99 |
|
'{' should be on the previous line. | 100 |
|
Method 'textNotPresentByTimeout' is not designed for extension - needs to be abstract, final or empty. | 104 |
|
Missing a Javadoc comment. | 104 |
|
Parameter text should be final. | 104 |
|
Parameter maxMillis should be final. | 104 |
|
'{' should be on the previous line. | 105 |
|
Method 'textNotPresentByTimeout' is not designed for extension - needs to be abstract, final or empty. | 109 |
|
Missing a Javadoc comment. | 109 |
|
Parameter text should be final. | 109 |
|
'{' should be on the previous line. | 110 |
|
Line is longer than 80 characters. | 115 |
|
Method 'elementNotPresentByTimeout' is not designed for extension - needs to be abstract, final or empty. | 121 |
|
Parameter locator should be final. | 121 |
|
Parameter maxMillis should be final. | 121 |
|
'{' should be on the previous line. | 122 |
|
Method 'byTimeout' is not designed for extension - needs to be abstract, final or empty. | 126 |
|
Missing a Javadoc comment. | 126 |
|
Parameter condition should be final. | 126 |
|
'{' should be on the previous line. | 127 |
|
Method 'byTimeout' is not designed for extension - needs to be abstract, final or empty. | 131 |
|
Missing a Javadoc comment. | 131 |
|
Parameter condition should be final. | 131 |
|
Parameter maxWaitTime should be final. | 131 |
|
'{' should be on the previous line. | 132 |
|
'{' should be on the previous line. | 135 |
|
'{' should be on the previous line. | 137 |
|
Line is longer than 80 characters. | 139 |
|
'if' construct must use '{}'s. | 142 |
|
'{' should be on the previous line. | 146 |
|
'}' should be on the same line. | 148 |
|
'{' should be on the previous line. | 150 |
|
Method 'untilTimeout' is not designed for extension - needs to be abstract, final or empty. | 156 |
|
Missing a Javadoc comment. | 156 |
|
Parameter condition should be final. | 156 |
|
'{' should be on the previous line. | 157 |
|
Method 'untilTimeout' is not designed for extension - needs to be abstract, final or empty. | 161 |
|
Missing a Javadoc comment. | 161 |
|
Parameter condition should be final. | 161 |
|
Parameter maxWaitTime should be final. | 161 |
|
'{' should be on the previous line. | 162 |
|
'{' should be on the previous line. | 165 |
|
'{' should be on the previous line. | 167 |
|
'{' should be on the previous line. | 172 |
|
Line is longer than 80 characters. | 174 |
|
'{' should be on the previous line. | 178 |
|
'}' should be on the same line. | 180 |
|
'{' should be on the previous line. | 182 |
|
Method 'textPresent' is not designed for extension - needs to be abstract, final or empty. | 191 |
|
Parameter text should be final. | 191 |
|
'{' should be on the previous line. | 192 |
|
Line is longer than 80 characters. | 193 |
|
Method 'textNotPresent' is not designed for extension - needs to be abstract, final or empty. | 199 |
|
Parameter text should be final. | 199 |
|
'{' should be on the previous line. | 200 |
|
Line is longer than 80 characters. | 201 |
|
First sentence should end with a period. | 204 |
|
Line is longer than 80 characters. | 206 |
|
Method 'formElementEquals' is not designed for extension - needs to be abstract, final or empty. | 209 |
|
Parameter locator should be final. | 209 |
|
Parameter value should be final. | 209 |
|
'{' should be on the previous line. | 210 |
|
Line is longer than 80 characters. | 211 |
|
First sentence should end with a period. | 214 |
|
Line is longer than 80 characters. | 216 |
|
Method 'elementPresent' is not designed for extension - needs to be abstract, final or empty. | 218 |
|
Parameter locator should be final. | 218 |
|
'{' should be on the previous line. | 219 |
|
Line is longer than 80 characters. | 220 |
|
First sentence should end with a period. | 223 |
|
Line is longer than 80 characters. | 225 |
|
Method 'elementNotPresent' is not designed for extension - needs to be abstract, final or empty. | 227 |
|
Parameter locator should be final. | 227 |
|
'{' should be on the previous line. | 228 |
|
Line is longer than 80 characters. | 229 |
|
Line is longer than 80 characters. | 233 |
|
Line is longer than 80 characters. | 235 |
|
Method 'elementVisible' is not designed for extension - needs to be abstract, final or empty. | 237 |
|
Parameter locator should be final. | 237 |
|
'{' should be on the previous line. | 238 |
|
Line is longer than 80 characters. | 239 |
|
Line is longer than 80 characters. | 243 |
|
Line is longer than 80 characters. | 245 |
|
Method 'elementNotVisible' is not designed for extension - needs to be abstract, final or empty. | 247 |
|
Parameter locator should be final. | 247 |
|
'{' should be on the previous line. | 248 |
|
Line is longer than 80 characters. | 249 |
|
Line is longer than 80 characters. | 253 |
|
Line is longer than 80 characters. | 254 |
|
Line is longer than 80 characters. | 255 |
|
Method 'htmlPresent' is not designed for extension - needs to be abstract, final or empty. | 257 |
|
Parameter html should be final. | 257 |
|
'{' should be on the previous line. | 258 |
|
Line is longer than 80 characters. | 259 |
|
Line is longer than 80 characters. | 263 |
|
Line is longer than 80 characters. | 264 |
|
Line is longer than 80 characters. | 265 |
|
Method 'htmlNotPresent' is not designed for extension - needs to be abstract, final or empty. | 267 |
|
Parameter html should be final. | 267 |
|
'{' should be on the previous line. | 268 |
|
Line is longer than 80 characters. | 269 |
|
First sentence should end with a period. | 272 |
|
Line is longer than 80 characters. | 273 |
|
Line is longer than 80 characters. | 275 |
|
Method 'elementHasText' is not designed for extension - needs to be abstract, final or empty. | 277 |
|
Parameter locator should be final. | 277 |
|
Parameter text should be final. | 277 |
|
'{' should be on the previous line. | 278 |
|
Line is longer than 80 characters. | 279 |
|
'+' is not followed by whitespace. | 279 |
|
'+' is not preceded with whitespace. | 279 |
|
First sentence should end with a period. | 282 |
|
Line is longer than 80 characters. | 283 |
|
Line is longer than 80 characters. | 285 |
|
Method 'elementDoesntHaveText' is not designed for extension - needs to be abstract, final or empty. | 287 |
|
Parameter locator should be final. | 287 |
|
Parameter text should be final. | 287 |
|
'{' should be on the previous line. | 288 |
|
Line is longer than 80 characters. | 289 |
|
'+' is not followed by whitespace. | 289 |
|
'+' is not preceded with whitespace. | 289 |
|
Line is longer than 80 characters. | 293 |
|
Line is longer than 80 characters. | 298 |
|
Method 'attributeContainsValue' is not designed for extension - needs to be abstract, final or empty. | 298 |
|
Parameter locator should be final. | 298 |
|
Parameter attribute should be final. | 298 |
|
Parameter value should be final. | 298 |
|
Line is longer than 80 characters. | 301 |
|
Line is longer than 80 characters. | 305 |
|
Line is longer than 80 characters. | 310 |
|
Method 'attributeDoesntContainValue' is not designed for extension - needs to be abstract, final or empty. | 310 |
|
Parameter locator should be final. | 310 |
|
Parameter attribute should be final. | 310 |
|
Parameter value should be final. | 310 |
|
Line is longer than 80 characters. | 313 |
|
First sentence should end with a period. | 316 |
|
Method 'linkPresentWithText' is not designed for extension - needs to be abstract, final or empty. | 321 |
|
Parameter text should be final. | 321 |
|
'{' should be on the previous line. | 322 |
|
Line is longer than 80 characters. | 323 |
|
First sentence should end with a period. | 326 |
|
Method 'linkNotPresentWithText' is not designed for extension - needs to be abstract, final or empty. | 330 |
|
Parameter text should be final. | 330 |
|
'{' should be on the previous line. | 331 |
|
Line is longer than 80 characters. | 332 |
|
Method 'linkVisibleWithText' is not designed for extension - needs to be abstract, final or empty. | 339 |
|
Parameter text should be final. | 339 |
|
'{' should be on the previous line. | 340 |
|
Line is longer than 80 characters. | 342 |
|
Line is longer than 80 characters. | 346 |
|
Line is longer than 80 characters. | 351 |
|
Method 'elementsVerticallyAligned' is not designed for extension - needs to be abstract, final or empty. | 351 |
|
Parameter locator1 should be final. | 351 |
|
Parameter locator2 should be final. | 351 |
|
Parameter deltaPixels should be final. | 351 |
|
Line is longer than 80 characters. | 353 |
|
Line is longer than 80 characters. | 354 |
|
Line is longer than 80 characters. | 355 |
|
'+' should be on a new line. | 355 |
|
Line is longer than 80 characters. | 356 |
|
Line is longer than 80 characters. | 361 |
|
Method 'elementsSameHeight' is not designed for extension - needs to be abstract, final or empty. | 361 |
|
Missing a Javadoc comment. | 361 |
|
Line is longer than 80 characters. | 365 |
|
'+' should be on a new line. | 365 |
|
Line is longer than 80 characters. | 366 |
|
Method 'elementContainsText' is not designed for extension - needs to be abstract, final or empty. | 373 |
|
Parameter locator should be final. | 373 |
|
Expected @param tag for 'locator'. | 373 |
|
Parameter text should be final. | 373 |
|
Expected @param tag for 'text'. | 373 |
|
'{' should be on the previous line. | 374 |
|
Line is longer than 80 characters. | 376 |
|
'+' is not followed by whitespace. | 376 |
|
'+' is not preceded with whitespace. | 376 |
|
Method 'elementDoesNotContainText' is not designed for extension - needs to be abstract, final or empty. | 383 |
|
Parameter locator should be final. | 383 |
|
Expected @param tag for 'locator'. | 383 |
|
Parameter text should be final. | 383 |
|
Expected @param tag for 'text'. | 383 |
|
'{' should be on the previous line. | 384 |
|
Line is longer than 80 characters. | 385 |
|
'+' is not followed by whitespace. | 385 |
|
'+' is not preceded with whitespace. | 385 |
| Violation | Message | Line |
|---|---|---|
|
First sentence should end with a period. | 3 |
|
Line is longer than 80 characters. | 9 |
|
Method 'getActionWait' is not designed for extension - needs to be abstract, final or empty. | 11 |
|
Missing a Javadoc comment. | 11 |
|
'{' should be on the previous line. | 12 |
|
'400' is a magic number. | 13 |
|
Method 'getPageLoadWait' is not designed for extension - needs to be abstract, final or empty. | 16 |
|
Missing a Javadoc comment. | 16 |
|
'{' should be on the previous line. | 17 |
|
'50000' is a magic number. | 18 |
|
Method 'getConditionCheckInterval' is not designed for extension - needs to be abstract, final or empty. | 21 |
|
Missing a Javadoc comment. | 21 |
|
'{' should be on the previous line. | 22 |
|
'100' is a magic number. | 23 |
| Violation | Message | Line |
|---|---|---|
|
Line is longer than 80 characters. | 7 |
|
'{' should be on the previous line. | 13 |
|
Line is longer than 80 characters. | 17 |
|
Line is longer than 80 characters. | 24 |
|
Parameter testResult should be final. | 28 |
|
'{' is not preceded with whitespace. | 28 |
| Violation | Message | Line |
|---|---|---|
|
Missing package documentation file. | 0 |
| Violation | Message | Line |
|---|---|---|
|
Class SeleniumStarter should be declared as final. | 9 |
|
'{' should be on the previous line. | 10 |
|
Missing a Javadoc comment. | 11 |
|
Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 11 |
|
Missing a Javadoc comment. | 13 |
|
Missing a Javadoc comment. | 14 |
|
Missing a Javadoc comment. | 15 |
|
Missing a Javadoc comment. | 16 |
|
Missing a Javadoc comment. | 17 |
|
Missing a Javadoc comment. | 19 |
|
'{' should be on the previous line. | 20 |
|
Missing a Javadoc comment. | 23 |
|
'{' should be on the previous line. | 24 |
|
Line is longer than 80 characters. | 28 |
|
Missing a Javadoc comment. | 28 |
|
Parameter config should be final. | 28 |
|
'{' should be on the previous line. | 31 |
|
Line is longer than 80 characters. | 37 |
|
Missing a Javadoc comment. | 37 |
|
Parameter config should be final. | 37 |
|
'{' should be on the previous line. | 40 |
|
'{' should be on the previous line. | 42 |
|
'{' should be on the previous line. | 45 |
|
Missing a Javadoc comment. | 53 |
|
Parameter config should be final. | 53 |
|
'{' should be on the previous line. | 54 |
|
'{' should be on the previous line. | 58 |
|
'if' is not followed by whitespace. | 59 |
|
'{' should be on the previous line. | 60 |
|
'}' should be on the same line. | 65 |
|
'{' should be on the previous line. | 67 |
|
'{' should be on the previous line. | 76 |
|
Line has trailing spaces. | 79 |
|
Missing a Javadoc comment. | 83 |
|
'{' should be on the previous line. | 84 |
|
'if' is not followed by whitespace. | 86 |
|
'{' should be on the previous line. | 87 |
|
Line is longer than 80 characters. | 89 |
|
Missing a Javadoc comment. | 95 |
|
'{' should be on the previous line. | 96 |
|
Missing a Javadoc comment. | 100 |
|
Parameter manual should be final. | 100 |
|
'manual' hides a field. | 100 |
|
'{' should be on the previous line. | 101 |
|
First sentence should end with a period. | 105 |
|
'{' should be on the previous line. | 118 |
|
Missing a Javadoc comment. | 122 |
|
Parameter userAgent should be final. | 122 |
|
'userAgent' hides a field. | 122 |
|
'{' should be on the previous line. | 123 |
| Violation | Message | Line |
|---|---|---|
|
Line is longer than 80 characters. | 6 |
|
'{' should be on the previous line. | 9 |
|
Expected @param tag for 'selenium'. | 14 |
|
Line has trailing spaces. | 17 |
| Violation | Message | Line |
|---|---|---|
|
'{' should be on the previous line. | 10 |
|
Line is longer than 80 characters. | 11 |
|
Missing a Javadoc comment. | 11 |
|
Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 11 |
|
Line is longer than 80 characters. | 13 |
|
Missing a Javadoc comment. | 13 |
|
Parameter s should be final. | 13 |
|
Parameter i should be final. | 13 |
|
Parameter s1 should be final. | 13 |
|
Parameter s2 should be final. | 13 |
|
Method 'doCommand' is not designed for extension - needs to be abstract, final or empty. | 18 |
|
Missing a Javadoc comment. | 18 |
|
Parameter s should be final. | 18 |
|
Parameter strings should be final. | 18 |
|
'{' should be on the previous line. | 19 |
|
'{' should be on the previous line. | 21 |
|
'}' should be on the same line. | 23 |
|
'{' should be on the previous line. | 25 |
| Violation | Message | Line |
|---|---|---|
|
File does not end with a newline. | 0 |
|
Line is longer than 80 characters. | 6 |
|
Utility classes should not have a public or default constructor. | 10 |
|
Missing a Javadoc comment. | 12 |
|
'{' should be on the previous line. | 13 |
|
Parameter selenium should be final. | 15 |
|
'{' should be on the previous line. | 16 |
|
Line is longer than 80 characters. | 17 |
|
'{' should be on the previous line. | 21 |
|
Line is longer than 80 characters. | 22 |
|
Missing a Javadoc comment. | 27 |
|
'{' should be on the previous line. | 28 |
|
Parameter selenium should be final. | 30 |
|
'{' should be on the previous line. | 31 |
|
'{' should be on the previous line. | 36 |
|
Missing a Javadoc comment. | 42 |
|
'{' should be on the previous line. | 43 |
|
Parameter selenium should be final. | 45 |
|
'{' should be on the previous line. | 46 |
|
'{' should be on the previous line. | 51 |
|
Missing a Javadoc comment. | 57 |
|
'{' should be on the previous line. | 58 |
|
Parameter selenium should be final. | 60 |
|
'{' should be on the previous line. | 61 |
|
Line has trailing spaces. | 64 |
|
'{' should be on the previous line. | 66 |
|
Missing a Javadoc comment. | 72 |
|
'{' should be on the previous line. | 73 |
|
Parameter selenium should be final. | 75 |
|
'{' should be on the previous line. | 76 |
|
'{' should be on the previous line. | 81 |
|
Missing a Javadoc comment. | 87 |
|
'{' should be on the previous line. | 88 |
|
Parameter selenium should be final. | 90 |
|
'{' should be on the previous line. | 91 |
|
'{' should be on the previous line. | 96 |
| Violation | Message | Line |
|---|---|---|
|
First sentence should end with a period. | 5 |
|
'{' should be on the previous line. | 11 |
|
Missing a Javadoc comment. | 12 |
|
Variable 'assertThat' must be private and have accessor methods. | 12 |
|
Missing a Javadoc comment. | 13 |
|
Variable 'client' must be private and have accessor methods. | 13 |
|
Missing a Javadoc comment. | 14 |
|
Variable 'config' must be private and have accessor methods. | 14 |
|
Missing a Javadoc comment. | 16 |
|
First sentence should end with a period. | 18 |
|
Expected @throws tag for 'Exception'. | 23 |
|
'{' should be on the previous line. | 24 |
|
'{' should be on the previous line. | 29 |
|
'+' should be on a new line. | 37 |
|
'+' should be on a new line. | 38 |
|
'+' should be on a new line. | 39 |
|
'+' should be on a new line. | 40 |
|
Line is longer than 80 characters. | 41 |
|
'+' should be on a new line. | 41 |
|
Line is longer than 80 characters. | 42 |
|
'+' should be on a new line. | 42 |
|
'+' should be on a new line. | 43 |
|
Line has trailing spaces. | 45 |
|
Line is longer than 80 characters. | 52 |
|
Expected an @return tag. | 56 |
|
Method 'getSeleniumClient' is not designed for extension - needs to be abstract, final or empty. | 56 |
|
'{' should be on the previous line. | 57 |
|
Line is longer than 80 characters. | 58 |
|
First sentence should end with a period. | 61 |
|
'{' should be on the previous line. | 65 |
|
First sentence should end with a period. | 68 |
|
Expected @throws tag for 'Exception'. | 72 |
|
'{' should be on the previous line. | 73 |
|
'{' should be on the previous line. | 77 |
|
First sentence should end with a period. | 82 |
|
'{' should be on the previous line. | 86 |
| Violation | Message | Line |
|---|---|---|
|
Line is longer than 80 characters. | 4 |
|
'{' should be on the previous line. | 10 |
|
Expected an @return tag. | 16 |
|
First sentence should end with a period. | 18 |
|
Expected an @return tag. | 22 |
|
First sentence should end with a period. | 24 |
|
Line has trailing spaces. | 25 |
|
Expected an @return tag. | 28 |
|
First sentence should end with a period. | 30 |
|
Expected an @return tag. | 34 |
|
First sentence should end with a period. | 36 |
|
Expected an @return tag. | 40 |
|
Line is longer than 80 characters. | 43 |
|
Expected an @return tag. | 47 |
|
Line is longer than 80 characters. | 51 |
|
Expected an @return tag. | 54 |
|
Expected an @return tag. | 60 |
| Violation | Message | Line |
|---|---|---|
|
Line is longer than 80 characters. | 6 |
|
'{' should be on the previous line. | 10 |
|
Name 'PAGE_LOAD_WAIT' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 15 |
|
Variable 'PAGE_LOAD_WAIT' must be private and have accessor methods. | 15 |
|
Line is longer than 80 characters. | 18 |
|
Name 'ACTION_WAIT' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 21 |
|
Variable 'ACTION_WAIT' must be private and have accessor methods. | 21 |
|
Missing a Javadoc comment. | 23 |
|
Parameter config should be final. | 23 |
|
'{' should be on the previous line. | 24 |
|
Line is longer than 80 characters. | 25 |
|
Line is longer than 80 characters. | 32 |
|
Line is longer than 80 characters. | 33 |
|
Method 'open' is not designed for extension - needs to be abstract, final or empty. | 35 |
|
Parameter url should be final. | 35 |
|
Expected @param tag for 'url'. | 35 |
|
'{' should be on the previous line. | 36 |
|
Line is longer than 80 characters. | 41 |
|
Method 'open' is not designed for extension - needs to be abstract, final or empty. | 43 |
|
Parameter url should be final. | 43 |
|
Expected @param tag for 'url'. | 43 |
|
Parameter timeoutMillis should be final. | 43 |
|
Expected @param tag for 'timeoutMillis'. | 43 |
|
'{' should be on the previous line. | 44 |
|
Method 'waitForPageToLoad' is not designed for extension - needs to be abstract, final or empty. | 52 |
|
Parameter timeoutMillis should be final. | 52 |
|
Expected @param tag for 'timeoutMillis'. | 52 |
|
'{' should be on the previous line. | 53 |
|
Line is longer than 80 characters. | 58 |
|
Line is longer than 80 characters. | 60 |
|
Method 'waitForCondition' is not designed for extension - needs to be abstract, final or empty. | 62 |
|
Parameter javascript should be final. | 62 |
|
Expected @param tag for 'javascript'. | 62 |
|
'{' should be on the previous line. | 63 |
|
Line is longer than 80 characters. | 68 |
|
Method 'waitForCondition' is not designed for extension - needs to be abstract, final or empty. | 71 |
|
Parameter javascript should be final. | 71 |
|
Expected @param tag for 'javascript'. | 71 |
|
Parameter timeoutMillis should be final. | 71 |
|
Expected @param tag for 'timeoutMillis'. | 71 |
|
'{' should be on the previous line. | 72 |
|
Line is longer than 80 characters. | 77 |
|
Line is longer than 80 characters. | 79 |
|
Line is longer than 80 characters. | 80 |
|
Method 'click' is not designed for extension - needs to be abstract, final or empty. | 84 |
|
Parameter locator should be final. | 84 |
|
Parameter waitForPageToLoad should be final. | 84 |
|
'{' should be on the previous line. | 85 |
|
'if' construct must use '{}'s. | 87 |
|
Line is longer than 80 characters. | 92 |
|
Line is longer than 80 characters. | 95 |
|
Method 'submit' is not designed for extension - needs to be abstract, final or empty. | 99 |
|
Parameter form should be final. | 99 |
|
Parameter waitForPageToLoad should be final. | 99 |
|
'{' should be on the previous line. | 100 |
|
'if' construct must use '{}'s. | 102 |
|
Line is longer than 80 characters. | 107 |
|
Line is longer than 80 characters. | 111 |
|
Line is longer than 80 characters. | 112 |
|
Method 'click' is not designed for extension - needs to be abstract, final or empty. | 116 |
|
Parameter locator should be final. | 116 |
|
Parameter timeoutMillis should be final. | 116 |
|
'{' should be on the previous line. | 117 |
|
Line is longer than 80 characters. | 123 |
|
Line is longer than 80 characters. | 128 |
|
Method 'submit' is not designed for extension - needs to be abstract, final or empty. | 132 |
|
Parameter form should be final. | 132 |
|
Parameter timeoutMillis should be final. | 132 |
|
'{' should be on the previous line. | 133 |
|
Line is longer than 80 characters. | 139 |
|
Method 'keyPress' is not designed for extension - needs to be abstract, final or empty. | 143 |
|
Parameter locator should be final. | 143 |
|
Parameter key should be final. | 143 |
|
'{' should be on the previous line. | 144 |
|
Line is longer than 80 characters. | 151 |
|
Line is longer than 80 characters. | 158 |
|
Method 'typeWithFullKeyEvents' is not designed for extension - needs to be abstract, final or empty. | 158 |
|
Parameter locator should be final. | 158 |
|
Parameter string should be final. | 158 |
|
Parameter reset should be final. | 158 |
|
'{' should be on the previous line. | 161 |
|
'{' should be on the previous line. | 170 |
|
Line has trailing spaces. | 185 |
|
'try' is not followed by whitespace. | 186 |
|
'{' is not preceded with whitespace. | 186 |
|
'}' should be on the same line. | 188 |
|
'catch' is not followed by whitespace. | 189 |
|
'{' should be on the previous line. | 190 |
|
Line is longer than 80 characters. | 197 |
|
Line is longer than 80 characters. | 198 |
|
Method 'typeWithFullKeyEvents' is not designed for extension - needs to be abstract, final or empty. | 203 |
|
Parameter locator should be final. | 203 |
|
Parameter string should be final. | 203 |
|
'{' should be on the previous line. | 204 |
|
Method 'selectOption' is not designed for extension - needs to be abstract, final or empty. | 208 |
|
Missing a Javadoc comment. | 208 |
|
Parameter selectName should be final. | 208 |
|
Parameter label should be final. | 208 |
|
'{' should be on the previous line. | 209 |
|
'for' is not followed by whitespace. | 215 |
|
'{' should be on the previous line. | 216 |
|
'if' is not followed by whitespace. | 217 |
|
'{' should be on the previous line. | 218 |
|
'if' is not followed by whitespace. | 222 |
|
'{' should be on the previous line. | 223 |
