MuranoCLI 3.2.0.beta.1 → 3.2.0.beta.5

data/spec/cmd_common.rb

@@ -16,6 +16,7 @@ require 'tmpdir'
 require 'webmock/rspec'
 
 require 'MrMurano/Config'
+require 'MrMurano/HttpAuthed'
 
 # Prevent Commander from registering its at_exit hook.
 # - Also print warning if spec exits, which rspec doesn't see as wrong.
@@ -50,6 +51,9 @@ at_exit do
     warn('¡Unexpected spec exit killed rspec!')
     warn('!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!')
     # DEVs: If you see this message, the last test exited unexpectedly.
+    warn('')
+    warn("Died on: #{$exited_abnormally}")
+    warn('')
   end
 end
 alias original_at_exit at_exit unless defined?(original_at_exit)
@@ -96,6 +100,8 @@ RSpec.shared_context 'CI_CMD' do
     args.push '-c', 'fullerror'
     # The spinner output would make it hard to write expects().
     args.push '--no-progress'
+    # Save the token.
+    args.push '--cache'
 
     if Gem.win_platform?
       cmd = args.map do |i|
@@ -159,7 +165,7 @@ RSpec.shared_context 'CI_CMD' do
     expect(out).to \
       eq('').or \
         eq("No solutions found\n").or \
-          match(/^Deleted [\d]+ solutions/)
+          match(/^Deleted [\d]+ solutions?/)
     expect(strip_color(err)).to eq('').or eq("No solutions found\n")
     expect(status.exitstatus).to eq(0).or eq(1)
   end
@@ -221,7 +227,7 @@ RSpec.shared_context 'CI_CMD' do
   # Utility fcns: Strings.
 
   # rname makes a random Murano-acceptable Solution name.
-  def rname(name)
+  def rname(name='')
     #"#{name}-#{Random.new.rand.hash.abs.to_s(16)}"
     # MUR-2454: Product name may only contain letters and numbers.
     #"#{name}#{Random.new.rand.hash.abs.to_s(16)}"
@@ -272,7 +278,7 @@ RSpec.shared_context 'CI_CMD' do
 
   def murano_command_runner(cmd, *args, wont_run: false, wont_parse: false)
     # This is a functional test, so tell WebMock to back off.
-    # FIXME: Remember the setting and re-enable.
+    allow_net_conn = WebMock::Config.instance.allow_net_connect != false
     WebMock.allow_net_connect!
 
     capture_stdio
@@ -288,25 +294,25 @@ RSpec.shared_context 'CI_CMD' do
     $stdout = tmpout
     $stderr = tmperr
 
-    $exited_abnormally = true
+    $exited_abnormally = "#{cmd} #{args.join(' ')}"
+
+    MrMurano::HttpAuthed.instance.credentials_reset
 
     # When Commander is loaded, it sets an at_exit hook, which we monkey
     # patch in ReCommander. Since Config.validate_cmd is called before
     # at_exit, it uses runner.command_exit to tell ReCommander's at_exit
     # monkey patch not to call Commander.run!. Via rspec, we don't use the
     # at_exit hook, or ReCommander.
-    if $cfg.nil?
-      $cfg = MrMurano::Config.new(::Commander::Runner.instance)
-      $cfg.load
-    end
+    $cfg = MrMurano::Config.new(::Commander::Runner.instance)
+    $cfg.load
     $cfg['tool.no-progress'] = true
-    $cfg.validate_cmd(cmd)
+    $cfg.validate_cmd
     runner = ::Commander::Runner.instance
     unless defined?(runner.command_exit) && runner.command_exit
-      # Commander's at_exit hook calls runner.run! which runs the command
-      # that was determined with Commander was loaded. I [lb] tried a few
-      # different ways to reset Runner.instance, but nothing worked; our
-      # best bet is to just call the command directly.
+      # (lb): Commander's at_exit hook calls runner.run! which runs the
+      # command that was determined with Commander was loaded. I tried a
+      # few different ways to reset Runner.instance, but nothing worked;
+      # our best bet is to just call the command directly.
       the_cmd = command(cmd.to_sym)
       when_called = the_cmd.peek_when_called.dup
 
@@ -342,6 +348,8 @@ RSpec.shared_context 'CI_CMD' do
     #   $stdout, $stderr = STDOUT, STDERR
     restore_stdio
 
+    WebMock.disable_net_connect! unless allow_net_conn
+
     [strip_color(tmpout.string), strip_color(tmperr.string)]
   end
 
@@ -376,11 +384,29 @@ RSpec.shared_context 'CI_CMD' do
     require 'byebug/settings/histfile'
     _ignored = ::Byebug::HistfileSetting::DEFAULT
 
+    $exited_abnormally = ex.example.to_s
+
     @testdir = Pathname.new(Dir.pwd).realpath
     Dir.mktmpdir do |hdir|
+      saved_home = Dir.home
       ENV['HOME'] = hdir
       Dir.chdir(hdir) do
         Dir.mkdir('.murano')
+
+        Dir.chdir('.murano') do
+          begin
+            FileUtils.ln_s(File.join(saved_home, '.murano', 'tokens'), 'tokens')
+          rescue NotImplementedError => err
+            require 'rbconfig'
+            unless OS.windows?
+              warn(
+                'Unexpected: ln_s failed on non-Windows machine / ' \
+                "host_os: #{RbConfig::CONFIG['host_os']} / err: #{err}"
+              )
+            end
+          end
+        end
+
         @tmpdir = File.join(hdir, 'project')
         Dir.mkdir(@tmpdir)
         Dir.chdir(@tmpdir) do
@@ -392,7 +418,10 @@ RSpec.shared_context 'CI_CMD' do
           rm_symlink
         end
       end
+      ENV['HOME'] = saved_home
     end
+
+    $exited_abnormally = false
   end
 end
 

data/spec/cmd_content_spec.rb

@@ -15,19 +15,24 @@ RSpec.describe 'murano content', :cmd, :needs_password do
 
   before(:example) do
     @product_name = rname('contestTest')
-    out, err, status = Open3.capture3(capcmd('murano', 'product', 'create', @product_name, '--save'))
+    out, err, status = Open3.capture3(
+      capcmd('murano', 'product', 'create', @product_name, '--save')
+    )
     expect(err).to eq('')
     expect(out.chomp).to match(/^[a-zA-Z0-9]+$/)
     expect(status.exitstatus).to eq(0)
   end
+
   after(:example) do
-    out, err, status = Open3.capture3(capcmd('murano', 'solution', 'delete', '--yes', @product_name))
+    out, err, status = Open3.capture3(
+      capcmd('murano', 'solution', 'delete', '--yes', @product_name)
+    )
     expect(out).to eq('')
     expect(err).to eq('')
     expect(status.exitstatus).to eq(0)
   end
 
-  it 'life cycle' do
+  it 'life cycle', :club_10s do
     out, err, status = Open3.capture3(capcmd('murano', 'content', 'list'))
     #expect(out.lines).to match(
     #  [
@@ -43,7 +48,9 @@ RSpec.describe 'murano content', :cmd, :needs_password do
     expect(status.exitstatus).to eq(0)
 
     FileUtils.copy(File.join(@testdir, 'spec/fixtures/dumped_config'), 'myFile')
-    out, err, status = Open3.capture3(capcmd('murano', 'content', 'upload', 'myFile', '--tags', 'random=junk'))
+    out, err, status = Open3.capture3(
+      capcmd('murano', 'content', 'upload', 'myFile', '--tags', 'random=junk')
+    )
     expect(out).to eq('')
     expect(err).to eq('')
     expect(status.exitstatus).to eq(0)
@@ -75,7 +82,9 @@ RSpec.describe 'murano content', :cmd, :needs_password do
     )
     expect(status.exitstatus).to eq(0)
 
-    out, err, status = Open3.capture3(capcmd('murano', 'content', 'download', 'myFile', '-o', 'testDown'))
+    out, err, status = Open3.capture3(
+      capcmd('murano', 'content', 'download', 'myFile', '-o', 'testDown')
+    )
     expect(out).to eq('')
     expect(err).to eq('')
     expect(status.exitstatus).to eq(0)
@@ -84,7 +93,9 @@ RSpec.describe 'murano content', :cmd, :needs_password do
     tdf = IO.read('testDown')
     expect(tdf).to eq(dcf)
 
-    out, err, status = Open3.capture3(capcmd('murano', 'content', 'delete', 'myFile', '-y'))
+    out, err, status = Open3.capture3(
+      capcmd('murano', 'content', 'delete', 'myFile', '-y')
+    )
     expect(out).to eq('')
     expect(err).to eq('')
     expect(status.exitstatus).to eq(0)
@@ -105,4 +116,3 @@ RSpec.describe 'murano content', :cmd, :needs_password do
   end
 end
 
-#  vim: set ai et sw=2 ts=2 :

data/spec/cmd_device_spec.rb

@@ -84,7 +84,7 @@ RSpec.describe 'murano device', :cmd, :needs_password do
     expect(status.exitstatus).to eq(0)
   end
 
-  it 'writes and reads' do
+  it 'writes and reads', :club_10s do
     FileUtils.mkpath('specs')
     FileUtils.copy(
       File.join(@testdir, 'spec/fixtures/product_spec_files/lightbulb.yaml'),

data/spec/cmd_domain_spec.rb

@@ -35,7 +35,7 @@ RSpec.describe 'murano domain', :cmd, :needs_password do
     expect(status.exitstatus).to eq(0)
   end
 
-  it 'show domain' do
+  it 'show domain', :club_20s, :club_10s do
     out, err, status = Open3.capture3(capcmd('murano', 'domain'))
     # 2017-05-31: Previously, the project could be named whatever and
     #   the URI would start with the same.
@@ -46,7 +46,7 @@ RSpec.describe 'murano domain', :cmd, :needs_password do
     #expect(out.chomp).to end_with('m2.exosite.io')
     out.lines.each do |line|
       expect(line).to match(
-        %r{^(Product|Application): domain[Tt]est[a-z0-9]+ <[a-z0-9]+> https://[.a-z0-9]+$}
+        %r{^(Product|Application): domain[Tt]est[a-z0-9]+ <[a-z0-9]+> https://[.\-a-z0-9]+$}
       )
     end
     expect(err).to eq('')

data/spec/cmd_element_spec.rb

@@ -0,0 +1,400 @@
+# Copyright © 2016-2017 Exosite LLC. All Rights Reserved
+# License: PROPRIETARY. See LICENSE.txt.
+# frozen_string_literal: true
+
+# vim:tw=0:ts=2:sw=2:et:ai
+# Unauthorized copying of this file is strictly prohibited.
+
+# (lb): 2018-05-01: Whoa. Coverage at 303 of 306 lines! 99.02%!!
+
+require 'fileutils'
+require 'json'
+require 'open3'
+require 'pathname'
+require 'tty-editor'
+require 'yaml'
+
+require 'cmd_common'
+require 'MrMurano/Config'
+
+RSpec.describe 'murano element', :cmd, :needs_password do
+  include_context 'CI_CMD'
+
+  context 'without project' do
+    it 'usage' do
+      cmd_verify_help('element')
+    end
+  end
+
+  def expect_exchange_element_table(stdout, stderr, num_cols: nil)
+    expect(stderr).to eq('')
+    lines = stdout.lines
+    # FIXME/2018-04-30: Is this too much detail??
+    #   What about running test once, dumping output to file,
+    #   and expecting same output next time?
+    # Outline of table. n columns. '+-----+-----+---...----+\n'
+    expect(lines[0]).to match(/^(\+-+){#{num_cols}}\+$/)
+    # Header. "... key | value ..."
+    expect(lines[1]).to match(/^\| key \+| value \+|$/)
+    # Separator.
+    expect(lines[2]).to match(/^(\+-+){#{num_cols}}\+$/)
+    # Content. Starts with nested.hash.key.
+    (3..(lines.length - 2)).to_a.each do |line|
+      expect(lines[line]).to match(/^\| [\.0-9a-zA-Z]* +\| /)
+    end
+    expect(lines[-1]).to match(/^(\+-+){#{num_cols}}\+$/)
+  end
+
+  context 'show' do
+    it '--help' do
+      stdout, stderr = murano_command_wont_parse('element show', '--help')
+      expect(stdout).to a_string_starting_with('Usage:')
+      expect(stderr).to eq('')
+    end
+
+    it 'using ambiguous name' do
+      # MEH/2018-04-30: This test is dependent on the platform having
+      # more than one element with the term 'IoT' in its name!
+      #  59bc24836b12c505c98c5a51: ‘Twilio SMS Service’
+      #  5adedb7cdcd2dd1d04082fba: ‘Public application for the twilio’
+      #
+      # FIXME/2018-04-30: Make test-specific elements for this test!
+      stdout, stderr = murano_command_exits('element show', 'twilio')
+      expect(stdout).to eq('')
+      expect(stderr).to a_string_starting_with(
+        'Please be more specific: More than one matching element was found:'
+      )
+    end
+
+    it 'as default-table' do
+      # FIXME/2018-04-30: Make test-specific elements for this test!
+      stdout, stderr = murano_command_run('element show', 'MUR-6407')
+      expect_exchange_element_table(stdout, stderr, num_cols: 2)
+      expect(stderr).to eq('')
+    end
+
+    it 'as table wrap' do
+      # FIXME/2018-04-30: Make test-specific elements for this test!
+      stdout, stderr = murano_command_run('element show', 'MUR-6407', '--wrap')
+      expect_exchange_element_table(stdout, stderr, num_cols: 2)
+      expect(stderr).to eq('')
+    end
+
+    it 'as table truncate' do
+      # FIXME/2018-04-30: Make test-specific elements for this test!
+      stdout, stderr = murano_command_run('element show', 'MUR-6407', '--truncate')
+      expect_exchange_element_table(stdout, stderr, num_cols: 2)
+      expect(stderr).to eq('')
+    end
+
+    # (lb): NOTE: This does not add any additional coverage to element.rb:
+    it 'as json' do
+      # FIXME/2018-04-30: Make test-specific elements for this test!
+      stdout, stderr = murano_command_run('element show', '--json', 'MUR-6407')
+      expect { JSON.parse(stdout) }.to_not raise_error
+      expect(stderr).to eq('')
+    end
+
+    # (lb): NOTE: This does not add any additional coverage to element.rb:
+    it 'as yaml' do
+      # FIXME/2018-04-30: Make test-specific elements for this test!
+      stdout, stderr = murano_command_run('element show', 'MUR-6407', '--yaml')
+      expect { YAML.parse(stdout) }.to_not raise_error
+      expect(stderr).to eq('')
+    end
+
+    # (lb): This adds 1 line of coverage.
+    it 'as csv' do
+      # FIXME/2018-04-30: Make test-specific elements for this test!
+      stdout, stderr = murano_command_run('element show', 'MUR-6407', '--csv')
+      expect(stdout).to_not eq('')
+      expect(stderr).to eq('')
+    end
+  end
+
+  context 'edit' do
+    before(:example) do
+      @json_input_file = 'element-show.json'
+      @spec_swagger_20k = 'swagger-mur-6407__20k.yaml'
+      @spec_swagger_10k = 'swagger-mur-6407__10k.yaml'
+      [
+        @json_input_file,
+        @spec_swagger_20k,
+        @spec_swagger_10k,
+      ].each do |path|
+        FileUtils.copy(
+          File.join(@testdir, "spec/fixtures/exchange_element/#{path}"), path
+        )
+      end
+    end
+
+    def element_editor_expect_edit_json_full
+      # Mock the $EDITOR session.
+      expect(TTY::Editor).to receive(:open) do |path|
+        edit_json_make_unique_contact(path)
+      end
+    end
+
+    def edit_json_make_unique_contact(path)
+      pname = path.is_a?(Pathname) && path || Pathname.new(path)
+      pname.write pname.read.gsub(
+        /\n  "contact": "[a-f0-9]+",\n/,
+        %(\n  "contact": "#{rname}",\n)
+      )
+    end
+
+    def edit_specs_yaml_make_unique_contact(path)
+      pname = path.is_a?(Pathname) && path || Pathname.new(path)
+      rname7 = rname[0..6]
+      pname.write pname.read.gsub(
+        /\n    name: Exosite Exosite Exosite Exosite\n/,
+        %(\n    name: #{rname7} #{rname7} #{rname7} #{rname7}\n)
+      )
+    end
+
+    def element_editor_expect_edit_plain_contact
+      # Mock the $EDITOR session.
+      expect(TTY::Editor).to receive(:open) do |path|
+        pname = path.is_a?(Pathname) && path || Pathname.new(path)
+        pname.write(rname)
+      end
+    end
+
+# DEL:
+    def element_editor_expect_edit_json_from_string
+      expect(TTY::Editor).to receive(:open) do |path|
+        pname = path.is_a?(Pathname) && path || Pathname.new(path)
+        pname.write(
+          %({"tiers":[],"bizid":"55cop31gs89","access":"private","attachment":{},"contact":"#{rname}","description":"For testing Murano CLI element-edit.\\n\\nTEST2222\\n\\nTEST MORE","image":{"detail":{"color":"#224123","filename":"","url":""},"thumbnail":{"color":"#224655","filename":"xxx","url":""}},"markdown":"AxxxxAA2333BBBCCC\\n","name":"MUR-6407 Test Element","source":{"from":"service","name":"mur6407testelement","url":"https://testtesttest.apps.exosite-dev.io/swagger-mur-6407.yaml"},"specs":"","type":"service","tags":[],"active":true,"approval":"approved","elementId":"5ae3600e313d01708c2a6e0f"})
+        )
+      end
+    end
+
+    it '--help' do
+      stdout, stderr = murano_command_wont_parse('element edit', '--help')
+      expect(stdout).to a_string_starting_with('Usage:')
+      expect(stderr).to eq('')
+    end
+
+    it 'as static field option' do
+      stdout, stderr = murano_command_run(
+        'element edit', '--contact', rname, 'MUR-6407'
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq('')
+    end
+
+    it 'as dynamic field option' do
+      stdout, stderr = murano_command_run(
+        'element edit', '-e', "contact=#{rname}", 'MUR-6407'
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq('')
+    end
+
+    it 'as dynamic edit empty string without update' do
+      # I.e., @edit_fields[''] = nil, meaning should bring up editor.
+      # FIXME: Requires (lb)'s fixed commander library.
+
+      # If we don't change the file, Murano CLI will not PUT to BizAPI.
+      expect(TTY::Editor).to receive(:open).once
+      stdout, stderr = murano_command_exits(
+        'element edit', '-e', '', 'MUR-6407'
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq("No new field values specified to update.\n")
+    end
+
+    it 'as dynamic edit empty string with update' do
+      element_editor_expect_edit_json_full
+      stdout, stderr = murano_command_run(
+        'element edit', '-e', '', 'MUR-6407'
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq('')
+    end
+
+    it 'as dynamic edit no field with update' do
+      element_editor_expect_edit_json_full
+      stdout, stderr = murano_command_run(
+        'element edit', '-e', '--', 'MUR-6407'
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq('')
+    end
+
+    it 'as dynamic edit set empty string' do
+      # I.e., @edit_fields[''] = ''
+      stdout, stderr = murano_command_run(
+        #'element edit', '-e', "''=", 'MUR-6407'
+        'element edit', '-e', '=', 'MUR-6407'
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq(
+        %(Request Failed: 400: [400] child "type" fails because ["type" is required]\n)
+      )
+    end
+
+    it 'as edit with too many fields specified' do
+      stdout, stderr = murano_command_exits(
+        'element edit', '-e', "contact=#{rname}", '-e', 'markdown', 'MUR-6407'
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq(
+        %(Please specify at most one field when not specifing all field values.\n)
+      )
+    end
+
+    it 'as edit with no action specified' do
+      stdout, stderr = murano_command_exits(
+        'element edit', 'MUR-6407'
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq(
+        %(Please specify one or more -e/--edit options, or an input file.\n)
+      )
+    end
+
+    it 'as edit with input file not found' do
+      bad_path = "/path/to/nowhere/#{rname}.json"
+      stdout, stderr = murano_command_exits(
+        'element edit', 'MUR-6407', bad_path
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq(
+        %(Input file not found: #{bad_path}\n)
+      )
+    end
+
+    it 'as edit with input file found and valid' do
+      path = Pathname.new(@json_input_file)
+      path.write path.read.gsub(
+        /,"contact":"[a-f0-9]+",/,
+        %(,"contact":"#{rname}",)
+      )
+      stdout, stderr = murano_command_run(
+        'element edit', 'MUR-6407', @json_input_file
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq('')
+    end
+
+    it 'as edit with input file and too many fields specified' do
+      stdout, stderr = murano_command_exits(
+        'element edit', '-e', 'contact', '-e', 'markdown', 'MUR-6407', @spec_swagger_20k
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq(
+        %(Please specify at most a single field when specifing an input file.\n)
+      )
+    end
+
+    it 'as edit with input file and value specified' do
+      stdout, stderr = murano_command_exits(
+        'element edit', '-e', 'contact=XYZ', 'MUR-6407', @spec_swagger_20k
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq(
+        %(Please do not specify a value when specifing an input file.\n)
+      )
+    end
+
+    it 'as edit with input file that is too large and unrecognized field' do
+      stdout, stderr = murano_command_run(
+        'element edit', '--edit', "spec=@#{@spec_swagger_20k}", 'MUR-6407'
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq(
+        %(Request Failed: 400: [400] "spec" is not allowed\n)
+      )
+    end
+
+    it 'as edit with input file that should not be parsed' do
+      stdout, stderr = murano_command_run(
+        'element edit', '--edit', "specs=@#{@spec_swagger_20k}", 'MUR-6407'
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq(
+        %(Request Failed: 400: [400] child "specs" fails because ["specs" must be a string]\n)
+      )
+    end
+
+    # FIXME/2018-05-01 00:10: How does Tilstra circumvent 20k limit?
+    it 'as edit with input file that is too large' do
+      stdout, stderr = murano_command_run(
+        'element edit', '--edit', 'specs', '--plain', '--', 'MUR-6407', @spec_swagger_20k
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq(
+        %(Request Failed: 400: [400] child "specs" fails because ["specs" length must be less than or equal to 10000 characters long]\n)
+      )
+    end
+
+    it 'as edit with input file that is just right' do
+      edit_specs_yaml_make_unique_contact(@spec_swagger_10k)
+      stdout, stderr = murano_command_run(
+        'element edit', '--edit', 'specs', '--plain', '--', 'MUR-6407', @spec_swagger_10k
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq('')
+
+      # Send same file again.
+      stdout, stderr = murano_command_exits(
+        'element edit', '--edit', 'specs', '--plain', '--', 'MUR-6407', @spec_swagger_10k
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq("No new field values specified to update.\n")
+    end
+
+    it 'as dynamic edit single field no value uses EDITOR' do
+      element_editor_expect_edit_plain_contact
+      stdout, stderr = murano_command_run(
+        'element edit', '-e', 'contact', '--', 'MUR-6407'
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq('')
+    end
+
+    it 'as dynamic edit nested field change value' do
+      stdout, stderr = murano_command_run(
+        'element edit', '-e', "image.detail.color=0x#{rname[0..5]}", 'MUR-6407'
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq('')
+    end
+
+    # (lb): Get 2 more lines of coverage.
+    it 'as dynamic edit no field with update from string' do
+      stdout, stderr = murano_command_run(
+        'element edit',
+        '-e',
+        %(={"bizid":"55cop31gs89","access":"private","attachment":{},"contact":"#{rname}","description":"For testing Murano CLI element-edit.\\n\\nTEST2222\\n\\nTEST MORE","image":{"detail":{"color":"#224123","filename":"","url":""},"thumbnail":{"color":"#224655","filename":"xxx","url":""}},"markdown":"AxxxxAA2333BBBCCC\\n","name":"MUR-6407 Test Element","source":{"from":"service","name":"mur6407testelement","url":"https://testtesttest.apps.exosite-dev.io/swagger-mur-6407.yaml"},"specs":"","type":"service","tags":[],"active":true,"elementId":"5ae3600e313d01708c2a6e0f"}),
+        'MUR-6407'
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq('')
+    end
+
+    # (lb): Get 2 more lines of coverage.
+    it 'as dynamic edit no field with update from string not valid Hash' do
+      stdout, stderr = murano_command_exits(
+        'element edit', '-e', %(={), 'MUR-6407'
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq %(The document object is not a Hash: {\n765: unexpected token at '{'\n)
+    end
+
+    # (lb): No additional coverage, other than integration.
+    it 'as dynamic edit no field with update from string fields not allowed' do
+      stdout, stderr = murano_command_run(
+        'element edit',
+        '-e',
+        %(={"tiers":[],"bizid":"55cop31gs89","access":"private","attachment":{},"contact":"#{rname}","description":"For testing Murano CLI element-edit.\\n\\nTEST2222\\n\\nTEST MORE","image":{"detail":{"color":"#224123","filename":"","url":""},"thumbnail":{"color":"#224655","filename":"xxx","url":""}},"markdown":"AxxxxAA2333BBBCCC\\n","name":"MUR-6407 Test Element","source":{"from":"service","name":"mur6407testelement","url":"https://testtesttest.apps.exosite-dev.io/swagger-mur-6407.yaml"},"specs":"","type":"service","tags":[],"active":true,"approval":"approved","elementId":"5ae3600e313d01708c2a6e0f"}),
+        'MUR-6407'
+      )
+      expect(stdout).to eq('')
+      expect(stderr).to eq %(Request Failed: 400: [400] "tiers" is not allowed. "approval" is not allowed\n)
+    end
+  end
+end
+

data/spec/cmd_exchange_spec.rb

@@ -39,7 +39,7 @@ RSpec.describe 'murano exchange', :cmd, :needs_password do
       expect(lines[3]).to match(/^(\+-+){#{num_cols}}\+$/)
       # Content. Starts with elementId.
       (4..(lines.length - 2)).to_a.each do |line|
-        expect(lines[line]).to match(/^\| [0-9a-f]+ \| /)
+        expect(lines[line]).to match(/^\| [0-9a-f]+ +\| /)
       end
       expect(lines[-1]).to match(/^(\+-+){#{num_cols}}\+$/)
     end
@@ -51,7 +51,7 @@ RSpec.describe 'murano exchange', :cmd, :needs_password do
         expect_exchange_element_table(stdout, stderr, num_cols: 4)
       end
 
-      it 'only ids' do
+      it 'only ids', :club_20s, :club_10s do
         stdout, stderr = murano_command_run('exchange list', '--idonly')
         expect(stderr).to eq('')
         stdout.lines.each do |line|