Update from code review

src/modules/Squid.rb

@@ -1448,21 +1448,21 @@ def Write
       if !writeAllSettings
         Report.Error(_("Cannot write settings."))
 
-        result &= false
+        result = false
       end
 
       # write firewall settings
       Progress.NextStage
       if !writeFirewallSettings
         Report.Error(_("Cannot write firewall settings."))
 
-        result &= false
+        result = false
       end
 
       # save service status
       Progress.NextStage
-      result &= save_status
-      result &= start_service
+      result = false unless save_status
+      result = false unless start_service
 
       return false if Abort()
 

testsuite/tests/squid_test.rb

@@ -123,7 +123,7 @@
         end
 
         it "returns true" do
-          expect(squid.Write).to be_truthy
+          expect(squid.Write).to eq(true)
         end
       end
 
@@ -133,7 +133,7 @@
         end
 
         it "returns false" do
-          expect(squid.Write).to be_falsey
+          expect(squid.Write).to eq(false)
         end
       end
     end