Add basic test for nerve reporter

example/nerve_services/serf_service1.json

@@ -0,0 +1,18 @@
+{
+  "host": "1.2.3.4",
+  "port": 3000,
+  "reporter_type": "serf",
+  "check_interval": 2,
+  "zk_hosts": ["localhost:2181"],
+  "zk_path": "/nerve/services/your_http_service/services",
+  "weight": 2,
+  "checks": [
+    {
+      "type": "http",
+      "uri": "/health",
+      "timeout": 0.2,
+      "rise": 3,
+      "fall": 2
+    }
+  ]
+}

lib/nerve/reporter/serf.rb

@@ -13,8 +13,7 @@ def initialize(service)
       # the name won't just be the name you gave but name_port. this allows a same
       # service to be on multiple ports of a same machine.
 
-      # FIXME: support IPv6?
-      @data        = "#{service['host']}:#{service['port']}"
+      @data = parse_data(get_service_data(service))
       @config_file = File.join(@config_dir,"zzz_nerve_#{@name}.json")
       File.unlink @config_file if File.exists? @config_file
     end
@@ -34,7 +33,9 @@ def report_down
 
     def update_data(new_data='')
       @data = new_data if new_data
-      File.write(@config_file, JSON.generate({tags:{"smart:#{@name}"=>@data}}))
+      data = JSON.parse(@data)
+      tag = "#{data['host']}:#{data['port']}"
+      File.write(@config_file, JSON.generate({tags:{"smart:#{@name}" =>tag}}))
       reload_serf
     end
 

spec/configuration_manager_spec.rb

@@ -24,7 +24,7 @@
       expect(config_manager.config.keys()).to include('instance_id', 'services')
       expect(config_manager.config['services'].keys()).to contain_exactly(
         'your_http_service', 'your_tcp_service', 'rabbitmq_service',
-        'etcd_service1', 'zookeeper_service1'
+        'etcd_service1', 'serf_service1',  'zookeeper_service1'
       )
     end
   end