gofogo / k8s-sigs-external-dns-fork / 17971651842

81%

Build Type

push

github

Committed by web-flow

Commit Message

refactor(service): reduce cyclomatic complexity of extractHeadlessEndpoints (#5822)

* feat: reduce cyclomatic complexity of service_test

* style: indention added

* style: tab

* refactor: address PR feedback, improve tests, and reduce complexity

* fix(service): address PR feedback and fix linting

* Revert "fix(service): address PR feedback and fix linting"

This reverts commit 4cba488dc.

* refactor: address all PR feedback - improve method naming, add test coverage, fix parameter ordering

* refactor: address latest PR feedback - convert to method, use testutils, add test coverage

* test: add coverage for pod hostname scenario

Addresses 4th to last PR comment about missing test coverage for
the case where pod.Spec.Hostname is set, which creates additional
headless domains (pod-specific hostname + base hostname)

* style: remove extra line

Co-authored-by: Michel Loiseleur <97035654+mloiseleur@users.noreply.github.com>

* style: remove extra line

Co-authored-by: Michel Loiseleur <97035654+mloiseleur@users.noreply.github.com>

---------

Co-authored-by: Michel Loiseleur <97035654+mloiseleur@users.noreply.github.com>

Coverage Stats

65 of 71 new or added lines in 1 file covered. (91.55%)

44 existing lines in 1 file now uncovered.

15765 of 20076 relevant lines covered (78.53%)

753.3 hits per line