brpm_module_brpm 0.1.18 → 0.1.19

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    ZDdhMjdmZTJjNDZkMjZjMjMwOGM0ZjY0MzIzYWM5YmYyMDdhNzJjYg==
+    NGMzNzQ1YjM3Zjg3YTZhNDk2NjAxNDMwYjc3MTc3NmZiNDliYWM3Zg==
   data.tar.gz: !binary |-
-    ZjJlZDExNmYxYmVlMDE5NzliYzY1YzY5YTM5MjM3OGQzMDAyNDA1ZQ==
+    ZWM1OGM0OGQ0OGY3MmJmMzhlY2E2OTBjZDg5ZThiNGQ1YmZkMGJmZA==
 SHA512:
   metadata.gz: !binary |-
-    N2Q3OWU4NWFiN2FjMWFjNjllOGFhNWI4ZTM4YWNkMTU3MGYwYTFkMzI5ZTI2
-    ZDgzMjViY2I2MDg0NzUxMTk5YzQzNzVhZTFjZjYxOTc4MzA3ODkyYmExNTBk
-    NzQ0MmFjZjQ0ODVkOTE5MDM2ZTlmZmM1ZWY3MzQyMDUzNmY1MDI=
+    OWFjODliNTFhYzllMTYzZWYxNGMyYzhhNDk2YWIyOWQyMGExMjViNTRmMmQ5
+    YTgyY2EyODE2YmFlYjMzZjJhMTcyNmFmMWIwODQ2MDAzZDBmZmZiM2I5Y2Zj
+    ODU0OGVlMGY4OTYyYzEyMjRiYWYxZWUzOTdmMWM4ZmFmNDZkMjI=
   data.tar.gz: !binary |-
-    YWI2ODk2MjFiNzY1YjBhMDQwNTAwYzJlNDIxYmM1ZDk5YWFkMzM1OGQ0MmM1
-    MTU0ZmRiYzUxOTA1NmUyMWM0MmNkZTZiNmMwMzFhZjJmZTFhZmI3YjFmOTE1
-    MTg5NDk0ZTI4NmZmYWEzNzIyN2JlMTdjZGQ3ZTllNGJiZTMzYTA=
+    OWNhMjhhYmU0Mjc4ZTFmMGRlMjlhYmM5MzBjMzMxMTIxZTQ1ZmU4ZTE3YjM3
+    YzcyYTIwYWNjYWQzMDYwMzc1YTIyNmQyMjk3ZTg2NTE4YWFkNWFlOTk0MzUw
+    YWY2MmE5NmVmNWU5NWNmOTYzNTRkNjgxNmJmMWMxMjE1MmFhMWQ=

data/config.yml

@@ -1,6 +1,6 @@
 dependencies:
 
-version: 0.1.18
+version: 0.1.19
 
 author: Niek Bartholomeus
 email: niek.bartholomeus@gmail.com

data/lib/brpm_rest_client.rb

@@ -1514,8 +1514,8 @@ class BrpmRestClient
     result_hash
   end
 
-  def get_list_item_by_name(name)
-    result = brpm_get "v1/list_items?filters[value_text]=#{name}"
+  def get_list_item_by_name(list_name, list_item_name)
+    result = brpm_get "v1/list_items?filters[list_name]=#{list_name}&filters[value_text]=#{list_item_name}"
 
     if result["status"] == "success"
       result_hash = get_list_item_by_id(result["response"].first["id"])
@@ -1572,13 +1572,23 @@ class BrpmRestClient
     result["response"]
   end
 
-  def create_or_update_list_item(list_item)
+  def create_or_update_list_item(list_name, list_item_name)
     BrpmAuto.log "Checking if the corresponding list item already exists ..."
-    existing_list_item = get_list_item_by_name list_item["name"]
+    existing_list_item = get_list_item_by_name(list_name, list_item_name)
 
+    list_item = {}
+    list_item["value_text"] = list_item_name
     if existing_list_item.nil?
       BrpmAuto.log "List item doesn't exist yet."
       list_item_already_exists=false
+
+      list = get_list_by_name(list_name)
+      if list
+        list_item["list_id"] = list["id"]
+      else
+        raise "A list with the name #{list_name} doesn't exist."
+      end
+
     else
       BrpmAuto.log "List item already exists."
       list_item_already_exists=true

data/tests/rest/list_items_spec.rb

@@ -3,15 +3,15 @@ require_relative "../spec_helper"
 describe 'List items REST API' do
   before(:all) do
     setup_brpm_auto
-  end
 
-  it 'should create, read, update and delete a list item' do
     list_items = @brpm_rest_client.get_list_items_by({:value_text => "MyAutomationCategory"})
     list_items.each do |list_item|
       @brpm_rest_client.archive_list_item(list_item["id"])
       @brpm_rest_client.delete_list_item(list_item["id"])
     end
+  end
 
+  it 'should create, read, update and delete a list item' do
     list_id = @brpm_rest_client.get_list_by_name("AutomationCategory")["id"]
 
     list_item = {}
@@ -27,7 +27,7 @@ describe 'List items REST API' do
     expect(list_item["list"]["id"]).to eq(list_id)
     expect(list_item["value_text"]).to eq("MyAutomationCategory")
 
-    list_item = @brpm_rest_client.get_list_item_by_name("MyAutomationCategory")
+    list_item = @brpm_rest_client.get_list_item_by_name("AutomationCategory", "MyAutomationCategory")
 
     expect(list_item["list"]["id"]).to eq(list_id)
     expect(list_item["value_text"]).to eq("MyAutomationCategory")
@@ -42,7 +42,7 @@ describe 'List items REST API' do
 
     @brpm_rest_client.delete_list_item(list_item["id"])
 
-    list_item = @brpm_rest_client.get_list_item_by_name("MyAutomationCategory")
+    list_item = @brpm_rest_client.get_list_item_by_name("AutomationCategory", "MyAutomationCategory")
     expect(list_item).to be_nil
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: brpm_module_brpm
 version: !ruby/object:Gem::Version
-  version: 0.1.18
+  version: 0.1.19
 platform: ruby
 authors:
 - Niek Bartholomeus