brightbox-cli 5.0.0.alpha → 5.0.0.rc1

data/lib/brightbox-cli/load_balancers.rb

@@ -9,26 +9,42 @@ module Brightbox
       new(conn.load_balancers.create(options))
     end
 
-    def acme_domains
-      if attributes["acme"]
-        attributes["acme"]["domains"].map do |domain|
-          [domain["identifier"], domain["status"]].join(":")
-        end.join(",")
+    def acme_cert
+      if attributes[:acme] && attributes[:acme][:certificate]
+        OpenStruct.new(
+          :expires_at => attributes[:acme][:certificate][:expires_at],
+          :fingerprint => attributes[:acme][:certificate][:fingerprint],
+          :issued_at => attributes[:acme][:certificate][:issued_at],
+          :subjects => attributes[:acme][:certificate][:domains].join(",")
+        )
       else
-        []
+        OpenStruct.new(
+          :expires_at => "",
+          :fingerprint => "",
+          :issued_at => "",
+          :subjects => ""
+        )
       end
-    rescue StandardError
-      []
     end
 
-    def attributes
-      fog_model.attributes
+    def formatted_acme_domains
+      return "" unless attributes[:acme]
+
+      attributes[:acme][:domains].map do |domain|
+        [domain[:identifier], domain[:status]].join(":")
+      end.join(",")
+    rescue StandardError
+      ""
     end
 
     def to_row
       attributes.merge(
         :locked => locked?,
-        :acme_domains => acme_domains,
+        :acme_domains => formatted_acme_domains,
+        :acme_cert_expires_at => acme_cert.expires_at,
+        :acme_cert_fingerprint => acme_cert.fingerprint,
+        :acme_cert_issued_at => acme_cert.issued_at,
+        :acme_cert_subjects => acme_cert.subjects,
         :ssl_minimum_version => ssl_minimum_version,
         :ssl_issuer => certificate_issuer,
         :ssl_subject => certificate_subject,

data/lib/brightbox-cli/server_groups.rb

@@ -34,15 +34,11 @@ module Brightbox
       fog_model.destroy
     end
 
-    def attributes
-      fog_model.attributes
-    end
-
     def to_row
-      o = attributes
-      o[:servers] = server_string
-      o[:server_count] = server_count
-      o
+      super.merge(
+        servers: server_string,
+        server_count: server_count
+      ).to_h
     end
 
     def server_ids

data/lib/brightbox-cli/servers.rb

@@ -37,25 +37,31 @@ module Brightbox
     end
 
     def attributes
-      a = fog_model.attributes
-      a[:image] = image_id
-      a[:created_at] = created_at
-      a[:created_on] = created_on
-      a[:type] = server_type["handle"]
-      a[:status] = fog_model.state
-      a[:locked] = locked?
-      a[:zone] = zone && zone["handle"]
-      a[:hostname] = hostname
-      a[:public_hostname] = "public.#{fqdn}" unless cloud_ips.empty?
-      a[:ipv6_hostname] = ipv6_fqdn if interfaces.any? { |i| i["ipv6_address"] }
-      a
+      fog_attributes.tap do |attrs|
+        attrs[:created_at] = created_at
+        attrs[:created_on] = created_on
+        attrs[:hostname] = hostname
+        attrs[:image] = image_id
+        attrs[:locked] = locked?
+        attrs[:status] = fog_model.state
+        attrs[:type] = server_type["handle"]
+        attrs[:zone] = zone && zone["handle"]
+
+        unless cloud_ips.empty?
+          attrs[:public_hostname] = "public.#{fqdn}"
+        end
+
+        if interfaces.any? { |i| i["ipv6_address"] }
+          attrs[:ipv6_hostname] = ipv6_fqdn
+        end
+      end
     end
 
     def to_row
-      o = attributes
-      o[:cloud_ip_ids] = cloud_ips.map { |i| i["id"] }
-      o[:ips] = interfaces.map { |i| i["ipv4_address"] }.join(", ")
-      o
+      attributes.merge(
+        cloud_ips: cloud_ips.map { |i| i["public_ip"] }.join(", "),
+        ips: interfaces.map { |i| i["ipv4_address"] }.join(", ")
+      )
     end
 
     def deleted?

data/lib/brightbox-cli/types.rb

@@ -3,10 +3,10 @@ module Brightbox
     def self.require_account?; true; end
 
     def attributes
-      o = fog_model.attributes
-      o[:ram] = ram
-      o[:disk] = disk
-      o
+      fog_attributes.tap do |attrs|
+        attrs[:ram] = ram
+        attrs[:disk] = disk
+      end
     end
 
     def ram
@@ -17,10 +17,6 @@ module Brightbox
       fog_model.disk.to_i
     end
 
-    def to_row
-      attributes
-    end
-
     def render_cell
       fog_model.handle
     end

data/lib/brightbox-cli/user_collaboration.rb

@@ -69,7 +69,7 @@ module Brightbox
     def to_row
       row_attributes = attributes
       row_attributes[:account] = attributes[:account]["id"]
-      row_attributes
+      row_attributes.to_h
     end
   end
 end

data/lib/brightbox-cli/users.rb

@@ -1,11 +1,9 @@
 module Brightbox
   class User < Api
-    def attributes
-      fog_model.attributes
-    end
-
     def to_row
-      attributes.merge(:accounts => accounts.size)
+      attributes.merge(
+        accounts: accounts.size
+      )
     end
 
     def self.all

data/lib/brightbox-cli/version.rb

@@ -1,3 +1,3 @@
 module Brightbox
-  VERSION = "5.0.0.alpha".freeze unless defined?(Brightbox::VERSION)
+  VERSION = "5.0.0.rc1".freeze unless defined?(Brightbox::VERSION)
 end

data/lib/brightbox-cli/volume.rb

@@ -57,19 +57,15 @@ module Brightbox
     end
 
     def attributes
-      a = fog_model.attributes
-      a[:id] = fog_model.id
-      a[:image] = image_id
-      a[:locked] = locked?
-      a[:server] = server_id
-      a[:status] = state
-      a[:type] = storage_type
-      a[:zone] = zone_id
-      a
-    end
-
-    def to_row
-      attributes
+      super.merge(
+        id: fog_model.id,
+        image: image_id,
+        locked: locked?,
+        server: server_id,
+        status: state,
+        type: storage_type,
+        zone: zone_id
+      )
     end
 
     def update(options)

data/lib/brightbox-cli/zones.rb

@@ -2,10 +2,6 @@ module Brightbox
   class Zone < Api
     def self.require_account?; true; end
 
-    def to_row
-      attributes
-    end
-
     def self.all
       conn.zones
     end

data/lib/brightbox_cli.rb

@@ -71,6 +71,7 @@ require_relative "brightbox/cli/config"
 require_relative "brightbox-cli/connection_manager"
 require_relative "brightbox-cli/tables"
 require_relative "brightbox-cli/logging"
+require_relative "brightbox-cli/indifferent_access_hash"
 require_relative "brightbox-cli/api"
 require_relative "brightbox-cli/config/cache"
 require_relative "brightbox-cli/config/gpg_encrypted_passwords"

data/spec/commands/cloudips/update_spec.rb

@@ -59,7 +59,9 @@ describe "brightbox cloudips update" do
 
       it "puts new name in update" do
         expect(Brightbox::CloudIP.conn).to receive(:update_cloud_ip).with(*expected_args).and_call_original
-        expect(stderr).to eq("")
+
+        expect(stderr).to be_empty unless ENV["DEBUG"]
+
         expect(stdout).to include("cip-12345")
         expect(stdout).to include("New name")
       end
@@ -82,7 +84,7 @@ describe "brightbox cloudips update" do
 
       it "puts new name in update" do
         expect(Brightbox::CloudIP.conn).to receive(:update_cloud_ip).with(*expected_args).and_call_original
-        expect(stderr).to eq("")
+        expect(stderr).to be_empty unless ENV["DEBUG"]
         expect(stdout).to include("cip-12345")
       end
     end
@@ -114,7 +116,7 @@ describe "brightbox cloudips update" do
     end
 
     it "puts new name in update" do
-      expect(stderr).to eq("")
+      expect(stderr).to be_empty unless ENV["DEBUG"]
       expect(stdout).to include("cip-dfsa3")
       expect(stdout).to include("domain.example")
     end
@@ -146,7 +148,7 @@ describe "brightbox cloudips update" do
     end
 
     it "puts new name in update" do
-      expect(stderr).to eq("")
+      expect(stderr).to be_empty unless ENV["DEBUG"]
       expect(stdout).to include("cip-dfsa3")
     end
   end
@@ -155,7 +157,7 @@ describe "brightbox cloudips update" do
     let(:argv) { ["cloudips", "update", "--reverse-dns", "domain.example", "--delete-reverse-dns", "cip-dfsa3"] }
 
     it "puts new name in update" do
-      expect(stderr).to eq("ERROR: You must either specify a reverse DNS record or --delete-reverse-dns\n")
+      expect(stderr).to include("ERROR: You must either specify a reverse DNS record or --delete-reverse-dns\n")
       expect(stdout).to eq("")
     end
   end

data/spec/commands/configmaps/create_spec.rb

@@ -109,7 +109,7 @@ describe "brightbox configmaps create" do
       it "does not error" do
         expect { output }.to_not raise_error
 
-        expect(stderr).to eq("ERROR: Config map data was not valid JSON\n")
+        expect(stderr).to include("ERROR: Config map data was not valid JSON\n")
 
         expect(stdout).to eq("")
       end
@@ -156,7 +156,7 @@ describe "brightbox configmaps create" do
         it "does not error" do
           expect { output }.to_not raise_error
 
-          expect(stderr).to eq("Creating config map\n")
+          expect(stderr).to include("Creating config map\n")
 
           aggregate_failures do
             expect(stdout).to match("cfg-s432l")
@@ -169,7 +169,7 @@ describe "brightbox configmaps create" do
           it "does not error" do
             expect { output }.to_not raise_error
 
-            expect(stderr).to eq("ERROR: Config map data was not valid JSON\n")
+            expect(stderr).to include("ERROR: Config map data was not valid JSON\n")
 
             expect(stdout).to eq("")
           end
@@ -219,7 +219,7 @@ describe "brightbox configmaps create" do
       it "does not error" do
         expect { output }.to_not raise_error
 
-        expect(stderr).to eq("Creating config map\n")
+        expect(stderr).to include("Creating config map\n")
 
         aggregate_failures do
           expect(stdout).to match("cfg-mj53s")
@@ -232,7 +232,7 @@ describe "brightbox configmaps create" do
         it "does not error" do
           expect { output }.to_not raise_error
 
-          expect(stderr).to eq("ERROR: Config map data was not valid JSON\n")
+          expect(stderr).to include("ERROR: Config map data was not valid JSON\n")
 
           expect(stdout).to eq("")
         end
@@ -248,7 +248,7 @@ describe "brightbox configmaps create" do
       it "does not error" do
         expect { output }.to_not raise_error
 
-        expect(stderr).to eq("ERROR: Config map data can only be passed by either 'data' or 'data-file'\n")
+        expect(stderr).to include("ERROR: Config map data can only be passed by either 'data' or 'data-file'\n")
 
         expect(stdout).to eq("")
       end

data/spec/commands/configmaps/destroy_spec.rb

@@ -26,7 +26,7 @@ describe "brightbox configmaps destroy" do
     it "does not error" do
       expect { output }.to_not raise_error
 
-      expect(stderr).to eq("ERROR: You must specify config map IDs as arguments\n")
+      expect(stderr).to include("ERROR: You must specify config map IDs as arguments\n")
 
       expect(stdout).to match("")
     end

data/spec/commands/configmaps/list_spec.rb

@@ -35,7 +35,7 @@ describe "brightbox configmaps list" do
     it "does not error" do
       expect { output }.to_not raise_error
 
-      expect(stderr).to eq("")
+      expect(stderr).to be_empty unless ENV["DEBUG"]
 
       aggregate_failures do
         expect(stdout).to match("id.*name")

data/spec/commands/configmaps/show_spec.rb

@@ -35,7 +35,7 @@ describe "brightbox configmaps show" do
     it "does not error" do
       expect { output }.to_not raise_error
 
-      expect(stderr).to eq("")
+      expect(stderr).to be_empty unless ENV["DEBUG"]
 
       aggregate_failures do
         expect(stdout).to match("id: cfg-12345")
@@ -81,7 +81,7 @@ describe "brightbox configmaps show" do
       it "does not error" do
         expect { output }.to_not raise_error
 
-        expect(stderr).to eq("ERROR: You can only access data for a single config map at a time\n")
+        expect(stderr).to include("ERROR: You can only access data for a single config map at a time\n")
 
         expect(stdout).to eq("")
       end
@@ -137,7 +137,7 @@ describe "brightbox configmaps show" do
       it "does not error" do
         expect { output }.to_not raise_error
 
-        expect(stderr).to eq("")
+        expect(stderr).to be_empty unless ENV["DEBUG"]
 
         expect(stdout).to eq(%({"key":"value","name":"key name"}\n))
       end

data/spec/commands/configmaps/update_spec.rb

@@ -27,7 +27,7 @@ describe "brightbox configmaps update" do
     it "does not error" do
       expect { output }.to_not raise_error
 
-      expect(stderr).to eq("ERROR: You must specify the config map ID as the first argument\n")
+      expect(stderr).to include("ERROR: You must specify the config map ID as the first argument\n")
 
       expect(stdout).to match("")
     end
@@ -73,7 +73,7 @@ describe "brightbox configmaps update" do
     it "does not error" do
       expect { output }.to_not raise_error
 
-      expect(stderr).to eq("Updating cfg-0932s\n")
+      expect(stderr).to include("Updating cfg-0932s\n")
 
       aggregate_failures do
         expect(stdout).to match("cfg-0932s")
@@ -134,7 +134,7 @@ describe "brightbox configmaps update" do
       it "does not error" do
         expect { output }.to_not raise_error
 
-        expect(stderr).to eq("Updating cfg-25hrt\n")
+        expect(stderr).to include("Updating cfg-25hrt\n")
 
         aggregate_failures do
           expect(stdout).to match("cfg-25hrt")
@@ -148,7 +148,7 @@ describe "brightbox configmaps update" do
       it "does not error" do
         expect { output }.to_not raise_error
 
-        expect(stderr).to eq("ERROR: Config map data was not valid JSON\n")
+        expect(stderr).to include("ERROR: Config map data was not valid JSON\n")
 
         expect(stdout).to eq("")
       end
@@ -218,7 +218,7 @@ describe "brightbox configmaps update" do
         it "does not error" do
           expect { output }.to_not raise_error
 
-          expect(stderr).to eq("Updating cfg-gr45a\n")
+          expect(stderr).to include("Updating cfg-gr45a\n")
 
           aggregate_failures do
             expect(stdout).to match("cfg-gr45a")
@@ -231,7 +231,7 @@ describe "brightbox configmaps update" do
           it "does not error" do
             expect { output }.to_not raise_error
 
-            expect(stderr).to eq("ERROR: Config map data was not valid JSON\n")
+            expect(stderr).to include("ERROR: Config map data was not valid JSON\n")
 
             expect(stdout).to eq("")
           end
@@ -303,7 +303,7 @@ describe "brightbox configmaps update" do
         it "does not error" do
           expect { output }.to_not raise_error
 
-          expect(stderr).to eq("Updating cfg-stdin\n")
+          expect(stderr).to include("Updating cfg-stdin\n")
 
           aggregate_failures do
             expect(stdout).to match("cfg-stdin")
@@ -316,7 +316,7 @@ describe "brightbox configmaps update" do
           it "does not error" do
             expect { output }.to_not raise_error
 
-            expect(stderr).to eq("ERROR: Config map data was not valid JSON\n")
+            expect(stderr).to include("ERROR: Config map data was not valid JSON\n")
 
             expect(stdout).to eq("")
           end
@@ -332,7 +332,7 @@ describe "brightbox configmaps update" do
         it "does not error" do
           expect { output }.to_not raise_error
 
-          expect(stderr).to eq("ERROR: Config map data can only be passed by either 'data' or 'data-file'\n")
+          expect(stderr).to include("ERROR: Config map data can only be passed by either 'data' or 'data-file'\n")
 
           expect(stdout).to eq("")
         end

data/spec/commands/lbs/create_spec.rb

@@ -11,48 +11,34 @@ describe "brightbox lbs" do
       cache_access_token(config, "f83da712e6299cda953513ec07f7a754f747d727")
     end
 
-    context "without required nodes arguments" do
+    context "without nodes arguments" do
       let(:argv) { %w[lbs create] }
-
-      it "does not error" do
-        expect { output }.to_not raise_error
-
-        expect(stderr).to eq("ERROR: You must specify which servers to balance connections to\n")
-      end
-    end
-
-    context "with required nodes arguments" do
-      let(:argv) { %w[lbs create srv-12345] }
-      let(:expected_args) { { nodes: [{ node: "srv-12345" }] } }
+      let(:expected_args) { { nodes: [] } }
 
       let(:json_response) do
         <<~EOS
         {
           "id": "lba-12345",
-          "nodes": [
-            {
-              "id": "srv-12345"
-            }
-          ]
+          "nodes": []
         }
         EOS
       end
 
       before do
         stub_request(:post, "http://api.brightbox.localhost/1.0/load_balancers?account_id=acc-12345")
-          .with(:body => hash_including("nodes" => [{ "node" => "srv-12345" }]))
+          .with(:body => hash_including("nodes" => []))
           .to_return(:status => 202, :body => json_response)
       end
 
       it "makes request" do
         expect(Brightbox::LoadBalancer).to receive(:create).with(hash_including(expected_args)).and_call_original
 
-        expect(stderr).to eq("Creating a new load balancer\n")
+        expect(stderr).to include("Creating a new load balancer\n")
         expect(stdout).to include("lba-12345")
       end
     end
 
-    context "with required nodes arguments" do
+    context "with a nodes argument" do
       let(:argv) { %w[lbs create srv-12345] }
       let(:expected_args) { { nodes: [{ node: "srv-12345" }] } }
 
@@ -78,12 +64,12 @@ describe "brightbox lbs" do
       it "makes request" do
         expect(Brightbox::LoadBalancer).to receive(:create).with(hash_including(expected_args)).and_call_original
 
-        expect(stderr).to eq("Creating a new load balancer\n")
+        expect(stderr).to include("Creating a new load balancer\n")
         expect(stdout).to include("lba-12345")
       end
     end
 
-    context "with multiple nodes" do
+    context "with multiple node arguments" do
       let(:argv) { %w[lbs create srv-12345 srv-54321] }
       let(:expected_args) { { nodes: [{ node: "srv-12345" }, { node: "srv-54321" }] } }
 
@@ -112,7 +98,7 @@ describe "brightbox lbs" do
       it "makes request" do
         expect(Brightbox::LoadBalancer).to receive(:create).with(hash_including(expected_args)).and_call_original
 
-        expect(stderr).to eq("Creating a new load balancer\n")
+        expect(stderr).to include("Creating a new load balancer\n")
         expect(stdout).to include("lba-12345")
       end
     end
@@ -139,7 +125,7 @@ describe "brightbox lbs" do
       it "includes buffer_size in response" do
         expect(Brightbox::LoadBalancer).to receive(:create).with(hash_including(expected_args)).and_call_original
 
-        expect(stderr).to eq("Creating a new load balancer\n")
+        expect(stderr).to include("Creating a new load balancer\n")
         expect(stdout).to include("lba-12345")
       end
     end
@@ -165,7 +151,7 @@ describe "brightbox lbs" do
 
       it "includes ssl_minimum_version in response" do
         expect(Brightbox::LoadBalancer).to receive(:create).with(hash_including(expected_args)).and_call_original
-        expect(stderr).to eq("Creating a new load balancer\n")
+        expect(stderr).to include("Creating a new load balancer\n")
         expect(stdout).to include("lba-12345")
       end
     end
@@ -206,7 +192,7 @@ describe "brightbox lbs" do
       end
 
       it "includes acme_certificate_domain in response" do
-        expect(stderr).to eq("Creating a new load balancer\n")
+        expect(stderr).to include("Creating a new load balancer\n")
         expect(stdout).to include("lba-12345")
       end
     end

data/spec/commands/lbs/show_spec.rb

@@ -39,7 +39,7 @@ describe "brightbox lbs" do
 
       it "shows load balancer details" do
         aggregate_failures do
-          expect(stderr).to eq("")
+          expect(stderr).to be_empty unless ENV["DEBUG"]
           expect(stdout).to include("id: lba-12345")
           expect(stdout).to include("status: active")
           expect(stdout).to include("name: app-lb1")
@@ -68,7 +68,15 @@ describe "brightbox lbs" do
                 "identifier": "domain2.test",
                 "status": "verified"
               }
-            ]
+            ],
+            "certificate": {
+              "domains": [
+                "domain.test"
+              ],
+              "expires_at": "2025-12-31T23:59:59Z",
+              "fingerprint": "fingerprint",
+              "issued_at": "2025-01-01T00:00:00Z"
+            }
           },
           "nodes":[
             {
@@ -87,13 +95,17 @@ describe "brightbox lbs" do
 
       it "shows load balancer details" do
         aggregate_failures do
-          expect(stderr).to eq("")
+          expect(stderr).to be_empty unless ENV["DEBUG"]
           expect(stdout).to include("id: lba-12345")
           expect(stdout).to include("status: active")
           expect(stdout).to include("name: app-lb1")
           expect(stdout).to include("created_at: 2012-03-05T12:00Z")
           expect(stdout).to include("nodes: srv-12345")
           expect(stdout).to include("acme_domains: domain.test:verified,domain2.test:verified")
+          expect(stdout).to include("acme_cert_subjects: domain.test")
+          expect(stdout).to include("acme_cert_expires_at: 2025-12-31T23:59:59Z")
+          expect(stdout).to include("acme_cert_fingerprint: fingerprint")
+          expect(stdout).to include("acme_cert_issued_at: 2025-01-01T00:00:00Z")
         end
       end
     end

data/spec/commands/lbs/update_spec.rb

@@ -41,7 +41,7 @@ describe "brightbox lbs" do
       end
 
       it "includes ssl_minimum_version in response" do
-        expect(stderr).to eq("Updating load balancer lba-12345\n")
+        expect(stderr).to include("Updating load balancer lba-12345\n")
         expect(stdout).to include("lba-12345")
       end
     end
@@ -68,7 +68,7 @@ describe "brightbox lbs" do
       end
 
       it "includes ssl_minimum_version in response" do
-        expect(stderr).to eq("Updating load balancer lba-grt24\n")
+        expect(stderr).to include("Updating load balancer lba-grt24\n")
         expect(stdout).to include("lba-grt24")
       end
     end
@@ -95,7 +95,7 @@ describe "brightbox lbs" do
       end
 
       it "includes ssl_minimum_version in response" do
-        expect(stderr).to eq("Updating load balancer lba-kl432\n")
+        expect(stderr).to include("Updating load balancer lba-kl432\n")
         expect(stdout).to include("lba-kl432")
       end
     end
@@ -139,7 +139,7 @@ describe "brightbox lbs" do
       end
 
       it "includes acme_certificate_domain in response" do
-        expect(stderr).to eq("Updating load balancer lba-12345\n")
+        expect(stderr).to include("Updating load balancer lba-12345\n")
         expect(stdout).to include("lba-12345")
       end
     end