[ 
https://issues.apache.org/jira/browse/SQOOP-3308?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16419060#comment-16419060
 ] 

ASF subversion and git services commented on SQOOP-3308:
--------------------------------------------------------

Commit d2c40366a7458b0f4804a382403bb40ac72fbb4f in sqoop's branch 
refs/heads/trunk from [~BoglarkaEgyed]
[ https://git-wip-us.apache.org/repos/asf?p=sqoop.git;h=d2c4036 ]

SQOOP-3308: Mock ConnManager field in TestTableDefWriter

(Szabolcs Vasas via Boglarka Egyed)


> Mock ConnManager field in TestTableDefWriter
> --------------------------------------------
>
>                 Key: SQOOP-3308
>                 URL: https://issues.apache.org/jira/browse/SQOOP-3308
>             Project: Sqoop
>          Issue Type: Sub-task
>    Affects Versions: 1.5.0
>            Reporter: Szabolcs Vasas
>            Assignee: Szabolcs Vasas
>            Priority: Major
>         Attachments: SQOOP-3308.patch, SQOOP-3308.patch, SQOOP-3308.patch, 
> SQOOP-3308.patch
>
>
> TableDefWriter has a dependency on ConnManager to retrieve the names and the 
> types of the table. It also introduces a field called _externalColTypes_ for 
> testing purposes and TestTableDefWriter uses this field to inject the test 
> table column names and types instead of mocking the ConnManager field.
> This setup makes it harder to add test cases to TestTableDefWriter and not a 
> good practice so it should be fixed.



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

Reply via email to