Uploaded image for project: 'Network Edge'
  1. Network Edge
  2. NE-1182

[Tech Debt] Ingress Operator unit test refactoring and various fixes


    • Icon: Epic Epic
    • Resolution: Done
    • Icon: Minor Minor
    • openshift-4.14
    • None
    • None
    • Ingress Operator unit test refactoring and various fixes
    • False
    • None
    • False
    • Not Selected
    • To Do
    • OCPPLAN-7878 - NetEdge - Maintainability and Debugability & Tech Backlog
    • OCPPLAN-7878NetEdge - Maintainability and Debugability & Tech Backlog
    • 0% To Do, 0% In Progress, 100% Done
    • 0
    • 0

      OCP/Telco Definition of Done
      Epic Template descriptions and documentation.

      <--- Cut-n-Paste the entire contents of this description into your new Epic --->

      Epic Goal

      • To refactor various unit test in cluster-ingress-operator to align with desire unit test standards. The unit tests are in need of various clean up to meet the standards of the network edge such as:
        • Using t.run in all unit tests for sub-test capabilities
        • Removing extraneous test cases
        • Fixing incorrect error messages

      Why is this important?

      • Maintaining standards in unit tests is important for the debug-ability of our code


      1. ...

      Acceptance Criteria

      • Unit tests generally meet our software standards

      Dependencies (internal and external)


      Previous Work (Optional):

      1. For shift week, Miciah provided a handful commits https://github.com/Miciah/cluster-ingress-operator/commits/gateway-api that was the motivation to create this epic. 

      Open questions::

      1. N/A

      Done Checklist

      • CI - CI is running, tests are automated and merged.
      • Release Enablement <link to Feature Enablement Presentation>
      • DEV - Upstream code and tests merged: <link to meaningful PR or GitHub Issue>
      • DEV - Upstream documentation merged: <link to meaningful PR or GitHub Issue>
      • DEV - Downstream build attached to advisory: <link to errata>
      • QE - Test plans in Polarion: <link or reference to Polarion>
      • QE - Automated tests merged: <link or reference to automated tests>
      • DOC - Downstream documentation merged: <link to meaningful PR>

            gspence@redhat.com Grant Spence
            gspence@redhat.com Grant Spence
            0 Vote for this issue
            3 Start watching this issue