fix removing hostname with test

src/lib/cfa/hosts.rb

@@ -140,12 +140,23 @@ def delete_hostname(hostname)
           if aliases.empty?
             delete_host(entry["ipaddr"])
           else
-            set_host(entry["ipaddr"], aliases.first, aliases[1..-1])
+            entry["canonical"] = aliases.first
+            entry.delete("alias")
+            entry.delete("alias[]")
+            aliases_col = entry.collection("alias")
+            aliases[1..-1].each do |a|
+              aliases_col.add(a)
+            end
           end
         else
           reduced_aliases = aliases_for(entry)
           reduced_aliases.delete(hostname)
-          set_host(entry["ipaddr"], entry["canonical"], reduced_aliases)
+          entry.delete("alias")
+          entry.delete("alias[]")
+          aliases_col = entry.collection("alias")
+          aliases[1..-1].each do |a|
+            aliases_col.add(a)
+          end
         end
       end
     end

test/host_test.rb

@@ -168,5 +168,16 @@
       tested_ip = "10.0.0.42"
       expect(Yast::Host.name_map[tested_ip]).to eql ["newname.suse.cz newname"]
     end
+
+    it "deletes old hostnames passed as first parameter" do
+      Yast::Host.Read
+      Yast::Host.Update("pepa.labs.suse.cz", "newname.suse.cz", "10.0.0.42")
+      Yast::Host.Write
+
+      content = file.content
+
+      expect(content.lines).to include("10.100.128.72   pepa pepa2\n")
+      expect(content.lines).to include("10.0.0.42\tnewname.suse.cz newname\n")
+    end
   end
 end