[
https://issues.jboss.org/browse/ARQGRA-433?page=com.atlassian.jira.plugin...
]
Lukáš Fryč commented on ARQGRA-433:
-----------------------------------
{quote}
I would personaly remove that FieldAccessEnricherValidator from Graphene completely when
you are considering to remove it in the future, why not to remove it now ...
{quote}
[As I have pointed
out|https://issues.jboss.org/browse/ARQGRA-433?focusedCommentId=12963482&...],
there are two reasons why using publicly accessible fields can't be recommended. We
either need to fix the approach how proxies are used for interception of calls and
injections or we should keep warning user that his usage can has unexpected consequences.
----
One approach that could work without checking test class annotations is leveraging
Arquillian BeforeClass event, that contain a type of the test class. When stored properly,
you can access it during injection time.
FieldAccessValidatorEnricher checks access to fields uniformly
--------------------------------------------------------------
Key: ARQGRA-433
URL:
https://issues.jboss.org/browse/ARQGRA-433
Project: Arquillian Graphene
Issue Type: Bug
Components: core
Affects Versions: 2.0.2.Final
Reporter: Stefan Miklosovic
Priority: Optional
Fix For: 2.0.3.Final
I have this test
{code}
@Rule
public ExpectedException expectedException = ExpectedException.none();
@Test
@InSequence(2)
@ReportMessage("This method should pass.")
public void testWithExpectedExceptionRule() {
expectedException.expect(RuntimeException.class);
throw new RuntimeException("this exception is expected");
}
{code}
Logger writes this out:
WARNING: Public field 'expectedException' found in
org.arquillian.droidium.devconf.AeroGearTestCase. Direct access to fields outside of the
declaring class is not allowed.
Apr 01, 2014 11:50:27 AM
org.jboss.arquillian.graphene.enricher.FieldAccessValidatorEnricher checkFieldValidity
However that @Rule field _has to be_ public
{quote}
A field must be public, not static, and a subtype of TestRule
{quote}
So in this case validation does not make sense.
--
This message was sent by Atlassian JIRA
(v6.2.3#6260)