spec/etcd/readme_spec.rb in etcd-0.2.0.alpha vs spec/etcd/readme_spec.rb in etcd-0.2.0.beta.1

- old
+ new

@@ -51,14 +51,13 @@ expect(@response.raft_term).to be >= 0 end end context 'set a key named "/message"' do - before(:all) do - @response = Etcd.client.set('/message', 'PinkFloyd') + @response = Etcd.client.set('/message', value: 'PinkFloyd') end it_should_behave_like 'response with valid http headers' it_should_behave_like 'response with valid node data' @@ -68,11 +67,11 @@ end context 'get a key named "/message"' do before(:all) do - Etcd.client.set('/message', 'PinkFloyd') + Etcd.client.set('/message', value: 'PinkFloyd') @response = Etcd.client.get('/message') end it_should_behave_like 'response with valid http headers' it_should_behave_like 'response with valid node data' @@ -83,12 +82,12 @@ end context 'change the value of a key named "/message"' do before(:all) do - Etcd.client.set('/message', 'World') - @response = Etcd.client.set('/message','PinkFloyd') + Etcd.client.set('/message', value: 'World') + @response = Etcd.client.set('/message', value: 'PinkFloyd') end it_should_behave_like 'response with valid http headers' it_should_behave_like 'response with valid node data' @@ -98,12 +97,12 @@ end context 'delete a key named "/message"' do before(:all) do - Etcd.client.set('/message', 'World') - Etcd.client.set('/message','PinkFloyd') + Etcd.client.set('/message', value: 'World') + Etcd.client.set('/message', value: 'PinkFloyd') @response = Etcd.client.delete('/message') end it 'should set the return action to SET' do expect(@response.action).to eq('delete') @@ -114,13 +113,13 @@ end context 'using ttl a key named "/message"' do before(:all) do - Etcd.client.set('/message', 'World') + Etcd.client.set('/message', value: 'World') @set_time = Time.now - @response = Etcd.client.set('/message','PinkFloyd', 5) + @response = Etcd.client.set('/message', value: 'PinkFloyd', ttl: 5) end it_should_behave_like 'response with valid http headers' it_should_behave_like 'response with valid node data' @@ -146,15 +145,15 @@ end context 'waiting for a change against a key named "/message"' do before(:all) do - Etcd.client.set('/message', 'foo') + Etcd.client.set('/message', value: 'foo') thr = Thread.new do @response = Etcd.client.watch('/message') end - Etcd.client.set('/message', 'PinkFloyd') + Etcd.client.set('/message', value: 'PinkFloyd') thr.join end it_should_behave_like 'response with valid http headers' it_should_behave_like 'response with valid node data' @@ -162,20 +161,20 @@ it 'should set the return action to SET' do expect(@response.action).to eq('set') end it 'should get the exact value by specifying a waitIndex' do - client.set('/message', 'someshit') + client.set('/message', value: 'someshit') w_response = client.watch('/message', index: @response.node.modified_index) expect(w_response.node.value).to eq('PinkFloyd') end end context 'atomic in-order keys' do before(:all) do - @response = Etcd.client.create_in_order('/queue', 'PinkFloyd') + @response = Etcd.client.create_in_order('/queue', value: 'PinkFloyd') end it_should_behave_like 'response with valid http headers' it_should_behave_like 'response with valid node data', :create @@ -186,26 +185,26 @@ it 'should have the child key as a positive integer' do expect(@response.key.split('/').last.to_i).to be > 0 end it 'should have the child keys as monotonically increasing' do - first_response = client.create_in_order('/queue', 'The Jimi Hendrix Experience') - second_response = client.create_in_order('/queue', 'The Doors') + first_response = client.create_in_order('/queue', value: 'The Jimi Hendrix Experience') + second_response = client.create_in_order('/queue', value: 'The Doors') first_key = first_response.key.split('/').last.to_i second_key = second_response.key.split('/').last.to_i expect(first_key).to be < second_key end it 'should enlist all children in sorted manner' do responses = [] 10.times do |n| - responses << client.create_in_order('/queue', 'Deep Purple - Track #{n}') + responses << client.create_in_order('/queue', value: 'Deep Purple - Track #{n}') end directory = client.get('/queue', sorted: true) past_index = directory.children.index(responses.first.node) 9.times do |n| - current_index = directory.children.index(responses[n+1].node) + current_index = directory.children.index(responses[n + 1].node) expect(current_index).to be > past_index past_index = current_index end end end @@ -228,22 +227,22 @@ expect(client.get('/directory').node.ttl).to eq(4) end it 'will throw error if updated without setting prevExist' do expect do - client.set('/directory', dir:true, ttl:5) + client.set('/directory', dir: true, ttl: 5) end.to raise_error end it 'can be updated by setting prevExist to true' do - client.set('/directory', prevExist: true, dir:true, ttl:5) + client.set('/directory', prevExist: true, dir: true, ttl: 5) expect(client.get('/directory').node.ttl).to eq(5) end it 'watchers should get expriy notification' do - client.set('/directory/a', 'Test') - client.set('/directory', prevExist: true, dir:true, ttl:2) + client.set('/directory/a', value: 'Test') + client.set('/directory', prevExist: true, dir: true, ttl: 2) response = client.watch('/directory/a', consistent: true, timeout: 3) expect(response.action).to eq('expire') end it 'should be expired after ttl' do sleep 5 @@ -254,41 +253,41 @@ end context 'atomic compare and swap' do it 'should raise error if prevExist is passed a false' do - client.set('/foo', 'one') + client.set('/foo', value: 'one') expect do - client.set('/foo','three', prevExist: false) + client.set('/foo', value: 'three', prevExist: false) end.to raise_error end it 'should raise error is prevValue is wrong' do - client.set('/foo', 'one') + client.set('/foo', value: 'one') expect do - client.set('/foo','three', prevValue: 'two') + client.set('/foo', value: 'three', prevValue: 'two') end.to raise_error end it 'should allow setting the value when prevValue is right' do - client.set('/foo', 'one') - expect(client.set('/foo','three', prevValue: 'one').value).to eq('three') + client.set('/foo', value: 'one') + expect(client.set('/foo', value: 'three', prevValue: 'one').value).to eq('three') end end context 'directory manipulation' do it 'should allow creating directory' do - expect(client.set('/dir',dir:true)).to be_directory + expect(client.set('/dir', dir: true)).to be_directory end it 'should allow listing directory' do - client.set('/foo_dir/foo','bar') + client.set('/foo_dir/foo', value: 'bar') expect(client.get('/').children.map(&:key)).to include('/foo_dir') end it 'should allow recursive directory listing' do response = client.get('/', recursive: true) - expect(response.children.find{|n|n.key=='/foo_dir'}.children).to_not be_empty + expect(response.children.find { |n|n.key == '/foo_dir' }.children).to_not be_empty end it 'should be able to delete empty directory without the recusrive flag' do expect(client.delete('/dir', dir: true).action).to eq('delete') end @@ -299,11 +298,11 @@ end context 'hidden nodes' do before(:all) do - Etcd.client.set('/_message', 'Hello Hidden World') - Etcd.client.set('/message', 'Hello World') + Etcd.client.set('/_message', value: 'Hello Hidden World') + Etcd.client.set('/message', value: 'Hello World') end it 'should not be visible in directory listing' do expect(client.get('/').children.map(&:key)).to_not include('_message') end