zi peng created HBASE-22814:
-------------------------------

             Summary: Insufficient set up in TestSecureWALReplay  because of 
test fixture inheritance
                 Key: HBASE-22814
                 URL: https://issues.apache.org/jira/browse/HBASE-22814
             Project: HBase
          Issue Type: Test
          Components: regionserver
    Affects Versions: 2.1.5
            Reporter: zi peng


Description: 

In _TestSecureWALReplay.java,_ it does not call the test fixture method from 
the base class _TestWALReplay.java_ while the test fixture in the all other 
sibling test cases do (e.g., _TestWALReplayBoundedLogWriterCreation.java_)

Suggestion:

should call *TestWALReplay.setUpBeforeClass()* in @BeforeClass

or use test utilities to remove dependencies of the tests.
{code:java}
public class TestSecureWALReplay extends TestWALReplay {
    @BeforeClass
    public static void setUpBeforeClass() throws Exception {
        Configuration conf = AbstractTestWALReplay.TEST_UTIL.getConfiguration();
        conf.set(HConstants.CRYPTO_KEYPROVIDER_CONF_KEY, 
KeyProviderForTesting.class.getName());
        conf.set(HConstants.CRYPTO_MASTERKEY_NAME_CONF_KEY, "hbase");
        conf.setClass("hbase.regionserver.hlog.reader.impl", 
SecureProtobufLogReader.class,
        Reader.class);
        conf.setClass("hbase.regionserver.hlog.writer.impl", 
SecureProtobufLogWriter.class,
        Writer.class);
        conf.setBoolean(HConstants.ENABLE_WAL_ENCRYPTION, true);
        AbstractTestWALReplay.setUpBeforeClass();
    }
}{code}



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)

Reply via email to