Merge pull request #119 from svdasein/master

Add spec to ensure that Client.logout is callable

spec/zabbixapi/client_spec.rb

@@ -79,6 +79,30 @@
     end
   end
 
+  describe '.logout' do
+    subject { client_mock.logout() }
+    before do
+      allow_any_instance_of(ZabbixApi::Client).to receive(:api_version).and_return('5.2.2')
+      allow_any_instance_of(ZabbixApi::Client).to receive(:api_request).with(
+        method: 'user.login',
+        params: {
+          user: nil,
+          password: nil
+        }
+      )
+      allow_any_instance_of(ZabbixApi::Client).to receive(:api_request).with(
+        method: 'user.logout'
+      )
+    end
+    it 'revokes auth using api request' do
+      expect_any_instance_of(ZabbixApi::Client).to receive(:api_request).with(
+        method: 'user.logout',
+        params: []
+      )
+      subject
+    end
+  end
+
   describe '.initialize' do
     subject { client_mock }
 
@@ -92,6 +116,10 @@
           password: nil
         }
       )
+      allow_any_instance_of(ZabbixApi::Client).to receive(:api_request).with(
+        method: 'user.logout',
+        params: {}
+      )
     end
 
     context 'when proxy is provided and no_proxy flag is false' do
@@ -228,7 +256,7 @@
       end
     end
 
-    context 'when method is not `apiinfo.version` or `user.login`' do
+    context 'when method is not `apiinfo.version` or `user.login` or `user.logout`' do
       let(:method) { 'fakemethod' }
       let(:message) do
         {