RE-673: bug fixes

cpanel · Sep 18, 2024 · 913aa4d · 913aa4d
1 parent c5938d5
commit 913aa4d
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/lib/Elevate/Components.pm b/lib/Elevate/Components.pm
@@ -219,7 +219,7 @@ sub _check_single_blocker ( $self, $name ) {
 
 sub _get_blocker_for ( $self, $name ) {    # useful for tests
     my $pkg = "Elevate::Components::$name";    # need to be loaded
-    return $pkg->new( blockers => $self );
+    return $pkg->new( components => $self );
 }
 
 1;
diff --git a/lib/Elevate/Components/Base.pm b/lib/Elevate/Components/Base.pm
@@ -20,7 +20,7 @@ use Carp ();
 use Cpanel::JSON ();
 
 use Simple::Accessor qw(
-  cpev
+  components
   rpm
   yum
   dnf
@@ -50,6 +50,17 @@ BEGIN {
     }
 }
 
+sub _build_components {
+    if ( $0 =~ qr{\bt/} ) {
+
+        # only for testing
+        return Elevate::Components->new;
+    }
+
+    # outside unit tests we should always be initialized with an 'Elevate::Components' object
+    Carp::confess(q[Missing components]);
+}
+
 sub _build_rpm ($self) {
     return Elevate::RPM->new( cpev => $self );
 }