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

Íñigo Goiri resolved HDFS-16265.
--------------------------------
    Fix Version/s: 3.4.0
     Hadoop Flags: Reviewed
       Resolution: Fixed

> Refactor HDFS tool tests for better reuse
> -----------------------------------------
>
>                 Key: HDFS-16265
>                 URL: https://issues.apache.org/jira/browse/HDFS-16265
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: hdfs-client, libhdfs++, tools
>    Affects Versions: 3.4.0
>         Environment: Centos 7, Centos 8, Debian 10, Ubuntu Focal
>            Reporter: Gautham Banasandra
>            Assignee: Gautham Banasandra
>            Priority: Blocker
>              Labels: pull-request-available
>             Fix For: 3.4.0
>
>          Time Spent: 1h
>  Remaining Estimate: 0h
>
> Currently, the test cases written in hdfs-tool-test.h isn't easy to reuse. 
> Primarily because the expectations are different for each HDFS tool. I 
> realized this while I was creating the PR for HDFS-16260. For instance, 
> passing more than one argument is erroneous to hdfs_allowSnapshot while it's 
> the only valid scenario for hdfs_deleteSnapshot.
> Thus, it won't be possible to reuse the test cases without decoupling the 
> expectations from the test case definitions. The solution here is to move the 
> expectations to the corresponding mock classes and invoke the call to set 
> them up in the test cases after the creation of mock instances.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-dev-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-dev-h...@hadoop.apache.org

Reply via email to