Expose BrewServices.start to fix #1115

Missed this addition in #1115, actually implementing BrewServices.start!
Homebrew · Oct 3, 2022 · fb15302 · fb15302
1 parent 3b4d678
commit fb15302
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/lib/bundle/brew_services.rb b/lib/bundle/brew_services.rb
@@ -17,6 +17,13 @@ def stop(name, verbose: false)
       true
     end
 
+    def start(name, verbose: false)
+      return unless Bundle.system HOMEBREW_BREW_FILE, "services", "start", name, verbose: verbose
+
+      started_services << name
+      true
+    end
+
     def restart(name, verbose: false)
       return unless Bundle.system HOMEBREW_BREW_FILE, "services", "restart", name, verbose: verbose
 

diff --git a/spec/bundle/brew_services_spec.rb b/spec/bundle/brew_services_spec.rb
@@ -42,6 +42,14 @@
       end
     end
 
+    it "starts the service" do
+      allow(described_class).to receive(:started_services).and_return([])
+      expect(Bundle).to receive(:system).with(HOMEBREW_BREW_FILE, "services", "start", "nginx",
+                                              verbose: false).and_return(true)
+      expect(described_class.start("nginx")).to be(true)
+      expect(described_class.started_services).to include("nginx")
+    end
+
     it "restarts the service" do
       allow(described_class).to receive(:started_services).and_return([])
       expect(Bundle).to receive(:system).with(HOMEBREW_BREW_FILE, "services", "restart", "nginx",