You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by "Stefano Bagnara (JIRA)" <se...@james.apache.org> on 2006/09/25 18:14:51 UTC

[jira] Resolved: (JSPF-29) Simplify DNSService service interface

     [ http://issues.apache.org/jira/browse/JSPF-29?page=all ]

Stefano Bagnara resolved JSPF-29.
---------------------------------

    Resolution: Fixed

Now much code has been moved from the DNSService to the spf library. This mean we have less code in the specific implementation and no business-related code in the services. clover coverage is now 77% (88% for the library classes)

> Simplify DNSService service interface
> -------------------------------------
>
>                 Key: JSPF-29
>                 URL: http://issues.apache.org/jira/browse/JSPF-29
>             Project: jSPF
>          Issue Type: Task
>          Components: Core
>    Affects Versions: 0.9b3
>            Reporter: Stefano Bagnara
>         Assigned To: Stefano Bagnara
>             Fix For: 0.9b4
>
>
> DNSService contains too much logic and we are not able to unit-test this part because of the networking needed.
> So we should try to simplify it as much as possible.
> 1) Remove NoneException and simply return null when no result is found.
> 2) Refactor it to include a generci getRecords(host, recordType)
> 3) Decide wether to introduce a TimeoutException or simply reuse the TempErrorException we use now.
> 4) Move code for specific lookups getARecords/getMXRecords/getPTRRecords/getAAAARecords and so on into the caller terms (or other code). We currenlty have very few callers for that services so it should not be a problem.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org