Uploaded image for project: 'Axiom'
  1. Axiom
  2. AXIOM-486

AxiomTestVerb should not extend TestVerb

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.2.19
    • Fix Version/s: 1.2.20
    • Component/s: None
    • Labels:
      None

      Description

      AxiomTestVerb currently extends TestVerb, presumably to permit someone to only have to statically import AxiomTestVerb.assertThat; However, TestVerb was not built for extension, will be marked final in more upcoming versions of Truth, especially 1.0.

      The fix is simple. Extend AbstractVerb if (and only if) you need its functionality, and otherwise simply statically import both AxiomTestVerb.assertThat and Truth.assertThat since the overloads are not ambiguous in their type signatures (and erasure).

        Attachments

          Activity

            People

            • Assignee:
              veithen Andreas Veithen
              Reporter:
              christiangruber Christian Gruber
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: