This is an automated email from the ASF dual-hosted git repository.
joshtynjala pushed a change to branch develop
in repository https://gitbox.apache.org/repos/asf/royale-compiler.git
from b7f3c3d05 linter: fix FieldNameRule allowing uppercase letter after _
at start
new b46727727 linter: start adding some tests
new 58fb870f8 linter: add formatting/whitespace tokens to TokenQuery to
support formatting rules
new ced2dde29 SwitchNode: fix getDefaultNode() failing to get default node
from inside an IBlockNode
new 3fd41ac49 linter: more tests
new 2b5660010 linter: EmptyStatementRule should handle block open before
semicolon
new f084dfaf1 linter: fix null results that should have been a token in
TokenQuery classes
new 4dafdbcef linter: LineCommentPositionRule fixes
new 8aafd5bfa linter: more tests
new f04e9788b linter: more tests
The 9 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
.../apache/royale/compiler/utils/ASNodeUtils.java | 1 -
.../compiler/internal/tree/as/SwitchNode.java | 13 +-
linter/build.xml | 2 +-
.../java/org/apache/royale/linter/ASLinter.java | 52 ++++-
.../java/org/apache/royale/linter/MXMLLinter.java | 37 +++-
.../org/apache/royale/linter/MXMLTokenQuery.java | 119 +++++++++--
.../java/org/apache/royale/linter/TokenQuery.java | 179 ++++++++++++++---
.../royale/linter/rules/EmptyStatementRule.java | 17 +-
.../linter/rules/LineCommentPositionRule.java | 14 +-
.../royale/linter/rules/MissingASDocRule.java | 2 +-
.../org/apache/royale/linter/rules/NoWithRule.java | 1 -
.../royale/linter/rules/StrictEqualityRule.java | 1 -
{formatter => linter}/src/test/build.xml | 42 ++--
.../royale/linter/rules/TestClassNameRule.java | 111 +++++++++++
.../royale/linter/rules/TestConstantNameRule.java | 85 ++++++++
.../royale/linter/rules/TestEmptyCommentRule.java | 110 ++++++++++
.../linter/rules/TestEmptyFunctionBodyRule.java | 111 +++++++++++
.../linter/rules/TestEmptyNestedBlockRule.java | 109 ++++++++++
.../linter/rules/TestEmptyStatementRule.java | 98 +++++++++
.../royale/linter/rules/TestFieldNameRule.java | 123 ++++++++++++
.../royale/linter/rules/TestFunctionNameRule.java | 221 +++++++++++++++++++++
.../royale/linter/rules/TestInterfaceNameRule.java | 111 +++++++++++
.../linter/rules/TestLineCommentPositionRule.java | 93 +++++++++
.../rules/TestLocalVarAndParameterNameRule.java | 201 +++++++++++++++++++
.../linter/rules/TestLocalVarShadowsFieldRule.java | 45 ++---
.../apache/royale/linter/rules/TestMXMLIDRule.java | 124 ++++++++++++
.../royale/linter/rules/TestNoAnyTypeRule.java | 109 ++++++++++
.../linter/rules/TestNoBooleanEqualityRule.java | 98 +++++++++
.../rules/TestNoConstructorDispatchEventRule.java | 59 ++++++
.../rules/TestNoConstructorReturnTypeRule.java | 59 ++++++
.../rules/TestNoDuplicateObjectKeysRule.java | 59 ++++++
.../linter/rules/TestNoDynamicClassRule.java | 59 ++++++
.../linter/rules/TestNoIfBooleanLiteralRule.java | 96 +++++++++
.../linter/rules/TestNoLeadingZeroesRule.java | 71 +++++++
.../royale/linter/rules/TestNoSparseArrayRule.java | 71 +++++++
.../linter/rules/TestNoStringEventNameRule.java | 73 +++++++
.../linter/rules/TestNoThisInClosureRule.java | 59 ++++++
.../royale/linter/rules/TestNoTraceRule.java} | 43 ++--
.../linter/rules/TestNoVoidOperatorRule.java | 60 ++++++
.../linter/rules/TestNoWildcardImportRule.java | 59 ++++++
.../royale/linter/rules/TestNoWithRule.java} | 43 ++--
.../TestOverrideContainsOnlySuperCallRule.java | 59 ++++++
.../royale/linter/rules/TestPackageNameRule.java | 148 ++++++++++++++
.../linter/rules/TestStaticConstantsRule.java | 72 +++++++
.../linter/rules/TestStrictEqualityRule.java | 84 ++++++++
.../linter/rules/TestSwitchWithoutDefaultRule.java | 59 ++++++
.../linter/rules/TestUnsafeNegationRule.java | 109 ++++++++++
.../royale/linter/rules/TestValidTypeofRule.java | 131 ++++++++++++
.../linter/rules/TestVariablesOnTopRule.java | 84 ++++++++
49 files changed, 3630 insertions(+), 156 deletions(-)
copy {formatter => linter}/src/test/build.xml (64%)
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestClassNameRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestConstantNameRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestEmptyCommentRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestEmptyFunctionBodyRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestEmptyNestedBlockRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestEmptyStatementRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestFieldNameRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestFunctionNameRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestInterfaceNameRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestLineCommentPositionRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestLocalVarAndParameterNameRule.java
copy
formatter/src/test/java/org/apache/royale/formatter/BaseFormatterTests.java =>
linter/src/test/java/org/apache/royale/linter/rules/TestLocalVarShadowsFieldRule.java
(55%)
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestMXMLIDRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestNoAnyTypeRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestNoBooleanEqualityRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestNoConstructorDispatchEventRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestNoConstructorReturnTypeRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestNoDuplicateObjectKeysRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestNoDynamicClassRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestNoIfBooleanLiteralRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestNoLeadingZeroesRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestNoSparseArrayRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestNoStringEventNameRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestNoThisInClosureRule.java
copy linter/src/{main/java/org/apache/royale/linter/internal/BaseLinter.java
=> test/java/org/apache/royale/linter/rules/TestNoTraceRule.java} (57%)
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestNoVoidOperatorRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestNoWildcardImportRule.java
copy linter/src/{main/java/org/apache/royale/linter/internal/BaseLinter.java
=> test/java/org/apache/royale/linter/rules/TestNoWithRule.java} (58%)
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestOverrideContainsOnlySuperCallRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestPackageNameRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestStaticConstantsRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestStrictEqualityRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestSwitchWithoutDefaultRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestUnsafeNegationRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestValidTypeofRule.java
create mode 100644
linter/src/test/java/org/apache/royale/linter/rules/TestVariablesOnTopRule.java
