Skip to content

Commit 816e899

Browse files
authored
Merge pull request #38 from jaypipes/runtime-cleanup
cleanups and DRYing up the reconciler unit tests
2 parents 055b089 + c7e3ac6 commit 816e899

File tree

4 files changed

+89
-239
lines changed

4 files changed

+89
-239
lines changed

mocks/pkg/types/aws_resource_descriptor.go

Lines changed: 0 additions & 21 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

pkg/runtime/reconciler.go

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -369,13 +369,6 @@ func (r *resourceReconciler) patchResourceStatus(
369369
exit := rlog.Trace("r.patchResourceStatus")
370370
defer exit(err)
371371

372-
changedStatus, err := r.rd.UpdateCRStatus(latest)
373-
if err != nil {
374-
return err
375-
}
376-
if !changedStatus {
377-
return nil
378-
}
379372
rlog.Enter("kc.Patch (status)")
380373
// It is necessary to use `DeepCopyObject` versions of `latest` when calling
381374
// `Patch` as this method overrides all values as merged from `desired`.

0 commit comments

Comments
 (0)