Uploaded image for project: 'OpenShift GitOps'
  1. OpenShift GitOps
  2. GITOPS-7302

Improve test coverage of community PR for webhook feature in Image Updater

XMLWordPrintable

    • GitOps Tangerine Sprint 18

      Story (Required)

      • The community PR for the webhook feature is currently lacking good test coverage. 
      • Add Unit Tests for code in community PR.

      Background and Approach (Required)

      Out of Scope

      • webhooks from other container registry, which is to be handled in GITOPS-7004

      Dependencies

      • image updater CRD and controller

      Acceptance Criteria (Mandatory)

      • functional impl of webhook processor in image updater to handle webhook request from quay
      • unit tests for new code
      • no regression in existing functionality and tests

      Definition of Done

      • Code Complete:
        • All code has been written, reviewed, and approved.
      • Tested:
        • Unit tests have been written and passed.
        • Ensure code coverage is not reduced with the changes.
        • Integration tests have been automated.
        • System tests have been conducted, and all critical bugs have been fixed.
        • Tested and merged on OpenShift either upstream or downstream on a local build.
      • Documentation:
        • User documentation or release notes have been written (if applicable).
      • Build:
        • Code has been successfully built and integrated into the main repository / project.
        • Midstream changes (if applicable) are done, reviewed, approved and merged.
      • Review:
        • Code has been peer-reviewed and meets coding standards.
        • All acceptance criteria defined in the user story have been met.
        • Tested by reviewer on OpenShift.
      • Deployment:
        • The feature has been deployed on OpenShift cluster for testing.

              rh-ee-ccoco Christopher Coco (Inactive)
              cfang@redhat.com Cheng Fang
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved: