Bug 968781

Summary:	Correct test methods dependency in EjbMethodInterceptorTestCase
Product:	[JBoss] JBoss Enterprise Application Platform 6	Reporter:	Chao Wang <chaowan>
Component:	Testsuite	Assignee:	Chao Wang <chaowan>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	low	Docs Contact:
Priority:	unspecified
Version:	6.1.0	CC:	cdewolf, ochaloup
Target Milestone:	ER1
Target Release:	EAP 6.1.1
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2013-09-16 20:26:50 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Chao Wang 2013-05-30 01:56:53 UTC

Description of problem:

org.jboss.as.test.integration.ejb.interceptor.method.EjbMethodInterceptorTestCase failed intermittently due to dependency of test ordering on https://jenkins.mw.lab.eng.bos.redhat.com/hudson/view/EAP6/view/EAP6-AS-Testsuite/job/eap-60-as-testsuite-RHEL-matrix-openJDK7/lastCompletedBuild/jdk=openjdk-1.7.0-local,label_exp=RHEL6%26%26x86_64/testReport/org.jboss.as.test.integration.ejb.interceptor.method/EjbMethodInterceptorTestCase/testMethodLevelInterceptors/

This test assumes testMethodLevelInterceptors() run at first which does not contain variable TopSecretInterceptor.called reset to false at beginning like second test method does. If test run in different order, then it will fail, should isolate this "connection" between test methods.

Version-Release number of selected component (if applicable):

EAP 6.1.1


How reproducible:

Use @InSequence to run testMethodOverloaded() before testMethodLevelInterceptors(), then test will fail.

Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:

No dependency between test order.


Additional info:

Comment 1 JBoss JIRA Server 2013-05-30 01:59:32 UTC

Chao Wang <chaowan> made a comment on jira WFLY-1407

pull request merged

Comment 2 Rostislav Svoboda 2013-06-07 10:14:48 UTC

QA_ACK granted, AS TS fix.

Comment 3 Ondrej Chaloupka 2013-06-19 22:45:48 UTC

Verified on EAP 6.1.1.ER1.
The test ordering should not be problem in this case any more.