Skip to content

Commit

Permalink
fix: remove empty implementation of TestStateListener from AbstractJD…
Browse files Browse the repository at this point in the history
…BCTestElement

TestStateListener + TestBean cause issues like in apache#6174,
so we should remove empty implementations
  • Loading branch information
vlsi committed Dec 19, 2023
1 parent 362f3b1 commit 538584e
Showing 1 changed file with 1 addition and 37 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@
* A base class for all JDBC test elements handling the basics of a SQL request.
*
*/
public abstract class AbstractJDBCTestElement extends AbstractTestElement implements TestStateListener{
public abstract class AbstractJDBCTestElement extends AbstractTestElement {
private static final long serialVersionUID = 235L;

private static final Logger log = LoggerFactory.getLogger(AbstractJDBCTestElement.class);
Expand Down Expand Up @@ -819,40 +819,4 @@ public String getResultVariable() {
public void setResultVariable(String resultVariable) {
this.resultVariable = resultVariable;
}


/**
* {@inheritDoc}
* @see org.apache.jmeter.testelement.TestStateListener#testStarted()
*/
@Override
public void testStarted() {
testStarted("");
}

/**
* {@inheritDoc}
* @see org.apache.jmeter.testelement.TestStateListener#testStarted(java.lang.String)
*/
@Override
public void testStarted(String host) {
}

/**
* {@inheritDoc}
* @see org.apache.jmeter.testelement.TestStateListener#testEnded()
*/
@Override
public void testEnded() {
testEnded("");
}

/**
* {@inheritDoc}
* @see org.apache.jmeter.testelement.TestStateListener#testEnded(java.lang.String)
*/
@Override
public void testEnded(String host) {
}

}

0 comments on commit 538584e

Please sign in to comment.