You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "stack (JIRA)" <ji...@apache.org> on 2018/02/07 23:50:00 UTC

[jira] [Issue Comment Deleted] (HBASE-19951) Cleanup the explicit timeout value for test method

     [ https://issues.apache.org/jira/browse/HBASE-19951?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

stack updated HBASE-19951:
--------------------------
    Comment: was deleted

(was: Here is the hackery I made patch with:

Thanks Appy I don't think Inheritance works for these annotations. Will check in a sec....

Meantime parking some hackery:

Put back in place the @Rule
find hbase-/src/test -name *.java |xargs perl -pi -e 's/(HBaseClassTestRule.forClass(([^.])\.class)/$1\n @Rule\n public final TestRule TIMEOUT =\n CategoryBasedTimeout.forClass($2.class);/g'
Add import for CategoryBasedTimeout
find hbase-*/src/test -name *.java |xargs perl -pi -e 's/(import org\.apache\.hadoop\.hbase\.HBaseClassTestRule/import org.apache.hadoop.hbase.CategoryBasedTimeout;\n$1/g'
Add import for TimeRule...
find hbase-*/src/test -name *.java |xargs perl -pi -e 's/(import org\.junit\.experimental\.categories\.Category/$1\nimport org.junit.rules.TestRule;/g'
Add Rule...
find hbase-*/src/test -name *.java |xargs perl -pi -e 's/(import org\.junit\.ClassRule/$1\nimport org.junit.Rule;/g'

Above might violate CheckStyle for import order in a few cases.)

> Cleanup the explicit timeout value for test method
> --------------------------------------------------
>
>                 Key: HBASE-19951
>                 URL: https://issues.apache.org/jira/browse/HBASE-19951
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Duo Zhang
>            Priority: Major
>         Attachments: HBASE-19951.branch-2.001.patch
>
>
> As said in the parent issue.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)