✨ Allow to upload file contents

Add now accepts file contents

lib/vaulty/cli/add.rb

@@ -4,9 +4,11 @@ class Add < Command
       attr_reader :catacomb, :data
       # @param [Catacomb] catacomb instance
       # @param [Hash] data
-      def initialize(catacomb:, data:)
+      # @param [Hash] files
+      def initialize(catacomb:, data: {}, files: {})
         @catacomb = catacomb
-        @data = data
+        files_with_content = read_file_contents!(files)
+        @data = data.merge(files_with_content)
       end
 
       def call
@@ -27,6 +29,16 @@ def call
       def matching_keys
         @matching_keys ||= catacomb.matching_keys(data.keys)
       end
+
+      def read_file_contents!(files)
+        files.each_with_object({}) do |(key, path), memo|
+          memo.merge!(key => read_file(path))
+        end
+      end
+
+      def read_file(path)
+        File.read(File.expand_path(path))
+      end
     end
   end
 end

lib/vaulty/cli_app.rb

@@ -25,12 +25,16 @@ class CLIApp
     arg(:path)
     command :add do |c|
       c.flag %i(secret s), desc: 'Key/Values to save', type: Array, multiple: true,
-                           required: true, arg_name: 'key:secret'
+        required: true, arg_name: 'key:secret'
+
+      c.flag %i(file f), desc: 'Key/File to be uploaded', type: Array, multiple: true,
+        arg_name: 'key:/path/file.ext'
 
       c.action do |_global_options, options, _args|
         data = options[:secret].reduce({}, :merge)
+        files = options[:file].reduce({}, :merge)
         catacomb = options[:catacomb]
-        Vaulty::CLI::Add.call(catacomb: catacomb, data: data)
+        Vaulty::CLI::Add.call(catacomb: catacomb, data: data, files: files)
       end
     end
 

lib/vaulty/output/table.rb

@@ -29,7 +29,7 @@ def initialize(data, header: DEFAULT_HEADER, highlight: {}, prompt:)
 
       def render
         table = TTY::Table.new(data, header: header, style: :markdown)
-        renderer = table.render(:ascii) do |render|
+        renderer = table.render(:ascii, column_widths: [30, 80]) do |render|
           render.padding = [0, 2, 0, 2]
           render.filter = @filter
         end

spec/cli/add_spec.rb

@@ -1,7 +1,8 @@
 RSpec.describe Vaulty::CLI::Add do
-  let(:instance) { described_class.new(catacomb: catacomb, data: data) }
+  let(:instance) { described_class.new(catacomb: catacomb, data: data, files: files) }
   let(:catacomb) { CatacombMock.new('secret/account') }
   let(:data) { { key: 'value' } }
+  let(:files) { {} }
   let(:confirmation) { true }
   let(:existing_data) { {} }
 
@@ -27,13 +28,8 @@
 
     it 'prints the resulting table' do
       subject
-      expect(output).to include_output <<~TXT
-        +-------+---------+
-        |  Key  |  Value  |
-        +-------+---------+
-        |  key  |  value  |
-        +-------+---------+
-      TXT
+      expect(output).to include_output('key')
+      expect(output).to include_output('value')
     end
 
     context 'when existing data is present' do
@@ -43,13 +39,8 @@
 
       it 'prints out the current table' do
         subject
-        expect(output).to include_output <<~TXT
-          +------------+---------+
-          |  Key       |  Value  |
-          +------------+---------+
-          |  existing  |  value  |
-          +------------+---------+
-        TXT
+        expect(output).to include_output('existing')
+        expect(output).to include_output('value')
       end
 
       context 'when existing data matches the given data' do
@@ -59,13 +50,8 @@
 
         it 'prints out the table with higlights' do
           subject
-          expect(output).to include_output <<~TXT
-            +-------+---------+
-            |  Key  |  Value  |
-            +-------+---------+
-            |  key  |  value  |
-            +-------+---------+
-          TXT
+          expect(output).to include_output('key')
+          expect(output).to include_output('value')
         end
 
         context 'when answering no' do
@@ -93,16 +79,31 @@
 
           it 'prints out the resulting table and higlights the values' do
             subject
-            expect(output).to include_output <<~TXT
-              +-------+---------------+
-              |  Key  |  Value        |
-              +-------+---------------+
-              |  key  |  other value  |
-              +-------+---------------+
-            TXT
+            expect(output).to include_output('key')
+            expect(output).to include_output('other value')
           end
         end
       end
     end
+
+    context 'when files are provided' do
+      let(:files) { { file: [FIXTURE_PATH, 'secret.txt'].join('/') } }
+
+      it 'prints out a banner' do
+        subject
+        expect(output).to include_output('Current value "secret/account"')
+      end
+
+      it 'writes the file contents' do
+        expect(catacomb).to receive(:merge).with(key: 'value', file: "secret\n")
+        subject
+      end
+
+      it 'prints the resulting table' do
+        subject
+        expect(output).to include_output('file')
+        expect(output).to include_output("secret")
+      end
+    end
   end
 end

spec/fixtures/secret.txt

@@ -0,0 +1 @@
+secret

spec/spec_helper.rb

@@ -4,6 +4,8 @@
 
 require 'vaulty'
 
+FIXTURE_PATH = File.join([File.dirname(__FILE__), 'fixtures'])
+
 RSpec.configure do |config|
   # rspec-expectations config goes here. You can use an alternate
   # assertion/expectation library such as wrong or the stdlib/minitest