hiifong/pmd
1
0
Fork 0
forked from phoedos/pmd
Code Pull Requests Activity

Fix tests

Browse Source
This commit is contained in:
Clément Fournier
2020-02-15 13:20:22 +01:00
parent 7d48a1e111
commit 351e53866c
4 changed files with 4 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
pmd-core/src/main/java/net/sourceforge/pmd/lang/rule/internal/LatticeRelation.java
View File

@@ -202,13 +202,6 @@ class LatticeRelation<T, @NonNull U> {
node.resetFrozenData(); node.resetFrozenData();
} }
/*
We need to do all this because there may be diamonds
in the lattice, in which case some nodes are reachable through
several paths and we would risk combining their proper values
several times.
*/
int n = nodes.size(); int n = nodes.size();
// topological sort // topological sort

3
pmd-core/src/test/resources/net/sourceforge/pmd/TestRuleset1.xml
View File

@@ -42,6 +42,7 @@ Just for test
<properties> <properties>
<property name="test2" type="String" description="test 2" value="foo"/> <property name="test2" type="String" description="test 2" value="foo"/>
<property name="test3" type="String" description="test 3" value="bar"/> <property name="test3" type="String" description="test 3" value="bar"/>
<property name="xpath" value="//dummyNode" />
</properties> </properties>
<example> <example>
<![CDATA[ <![CDATA[
@@ -52,4 +53,4 @@ Just for test
<rule name="TestRuleRef" ref="net/sourceforge/pmd/TestRuleset2.xml/TestRule"/> <rule name="TestRuleRef" ref="net/sourceforge/pmd/TestRuleset2.xml/TestRule"/>
</ruleset> </ruleset>

2
pmd-test/src/main/java/net/sourceforge/pmd/test/lang/DummyLanguageModule.java
View File

@@ -63,7 +63,7 @@ public class DummyLanguageModule extends BaseLanguageModule {
} }
} }
private static class DummyRootNode extends DummyNode implements RootNode { public static class DummyRootNode extends DummyNode implements RootNode {
} }

2
pmd-test/src/test/java/net/sourceforge/pmd/testframework/RuleTstTest.java
View File

@@ -49,7 +49,7 @@ public class RuleTstTest {
verify(rule).start(any(RuleContext.class)); verify(rule).start(any(RuleContext.class));
verify(rule).end(any(RuleContext.class)); verify(rule).end(any(RuleContext.class));
verify(rule).getLanguage(); verify(rule).getLanguage();
verify(rule).getTargetingStrategy(); verify(rule, times(2)).getTargetingStrategy();
verify(rule).getMinimumLanguageVersion(); verify(rule).getMinimumLanguageVersion();
verify(rule).getMaximumLanguageVersion(); verify(rule).getMaximumLanguageVersion();
verify(rule).apply(any(Node.class), any(RuleContext.class)); verify(rule).apply(any(Node.class), any(RuleContext.class));