Apache Whirr (retired)
  1. Apache Whirr (retired)
  2. WHIRR-440

Unit tests improvements (less bound to external services)

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.7.0
    • Component/s: None
    • Labels:
      None

      Description

      In this patch I have made the unit tests less dependent on having the ability to perform reverse DNS
      resolution and I have also switched some tests to use a memory based state store. We should set a
      goal for 0.8.0 to be able to run the unit tests without having an internet connection.

      1. WHIRR-440.patch
        65 kB
        Andrei Savu
      2. WHIRR-440.patch
        74 kB
        Andrei Savu

        Activity

        Hide
        Andrei Savu added a comment -

        Please review.

        Show
        Andrei Savu added a comment - Please review.
        Hide
        Tom White added a comment -

        +1

        There's a checkstyle warning in FastDnsResolver. After I fixed that all the unit tests passed for me.

        Also, how about changing o.a.whirr.service.state to o.a.whirr.state, and o.a.whirr.util.dns to o.a.whirr.net?

        Show
        Tom White added a comment - +1 There's a checkstyle warning in FastDnsResolver. After I fixed that all the unit tests passed for me. Also, how about changing o.a.whirr.service.state to o.a.whirr.state, and o.a.whirr.util.dns to o.a.whirr.net?
        Hide
        Andrei Savu added a comment -

        Here is an updated version of the patch - changed package names and fixed checkstyle. I'm going to commit this now.

        Show
        Andrei Savu added a comment - Here is an updated version of the patch - changed package names and fixed checkstyle. I'm going to commit this now.
        Hide
        Andrei Savu added a comment -

        Committed to trunk. Thanks Tom for reviewing.

        Show
        Andrei Savu added a comment - Committed to trunk. Thanks Tom for reviewing.

          People

          • Assignee:
            Andrei Savu
            Reporter:
            Andrei Savu
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development