[visualforce] Cleanup rule test xml files
Avoid unnecessary CDATA for description Use 4 space indentation instead of tabs Remove trailing whitespace End file with a newline
This commit is contained in:
@ -3,35 +3,31 @@
|
|||||||
xmlns="http://pmd.sourceforge.net/rule-tests"
|
xmlns="http://pmd.sourceforge.net/rule-tests"
|
||||||
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
|
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
|
||||||
xsi:schemaLocation="http://pmd.sourceforge.net/rule-tests http://pmd.sourceforge.net/rule-tests_1_0_0.xsd">
|
xsi:schemaLocation="http://pmd.sourceforge.net/rule-tests http://pmd.sourceforge.net/rule-tests_1_0_0.xsd">
|
||||||
<test-code>
|
|
||||||
<description><![CDATA[
|
|
||||||
CSRF by starting a controller with an EL action
|
|
||||||
]]></description>
|
|
||||||
<expected-problems>1</expected-problems>
|
|
||||||
<code><![CDATA[
|
|
||||||
<apex:page controller="AcRestActionsController" action="{!csrfInitMethod}" >
|
|
||||||
]]></code>
|
|
||||||
<source-type>vf</source-type>
|
|
||||||
</test-code>
|
|
||||||
<test-code>
|
|
||||||
<description><![CDATA[
|
|
||||||
Controller without actions is perfectly safe
|
|
||||||
]]></description>
|
|
||||||
<expected-problems>0</expected-problems>
|
|
||||||
<code><![CDATA[
|
|
||||||
<apex:page controller="AcRestActionsController" >
|
|
||||||
]]></code>
|
|
||||||
<source-type>vf</source-type>
|
|
||||||
</test-code>
|
|
||||||
|
|
||||||
<test-code>
|
<test-code>
|
||||||
<description><![CDATA[
|
<description>CSRF by starting a controller with an EL action</description>
|
||||||
JS action on load is perfectly safe
|
<expected-problems>1</expected-problems>
|
||||||
]]></description>
|
<code><![CDATA[
|
||||||
<expected-problems>0</expected-problems>
|
<apex:page controller="AcRestActionsController" action="{!csrfInitMethod}" >
|
||||||
<code><![CDATA[
|
]]></code>
|
||||||
|
<source-type>vf</source-type>
|
||||||
|
</test-code>
|
||||||
|
|
||||||
|
<test-code>
|
||||||
|
<description>Controller without actions is perfectly safe</description>
|
||||||
|
<expected-problems>0</expected-problems>
|
||||||
|
<code><![CDATA[
|
||||||
|
<apex:page controller="AcRestActionsController" >
|
||||||
|
]]></code>
|
||||||
|
<source-type>vf</source-type>
|
||||||
|
</test-code>
|
||||||
|
|
||||||
|
<test-code>
|
||||||
|
<description>JS action on load is perfectly safe</description>
|
||||||
|
<expected-problems>0</expected-problems>
|
||||||
|
<code><![CDATA[
|
||||||
<apex:page controller="AcRestActionsController" action="init()" >
|
<apex:page controller="AcRestActionsController" action="init()" >
|
||||||
]]></code>
|
]]></code>
|
||||||
<source-type>vf</source-type>
|
<source-type>vf</source-type>
|
||||||
</test-code>
|
</test-code>
|
||||||
</test-data>
|
</test-data>
|
||||||
|
File diff suppressed because it is too large
Load Diff
Reference in New Issue
Block a user