Update the rake tasks

Author: blocknotes

README.md

@@ -20,6 +20,11 @@ db:
   service: DB
 ```
 
+## Misc
+Some rake tasks are available:
+- `asdb:list`: list the stored attachments
+- `asdb:get`: download an attachment (ex. `bin/rails "asdb:get[ruby-logo.png,/tmp]"`)
+
 ## Do you like it? Star it!
 If you use this component just star it. A developer is more motivated to improve a project when there is some interest.
 

lib/tasks/active_storage_db_tasks.rake

@@ -2,7 +2,7 @@
 
 namespace :asdb do
   desc 'ActiveStorageDB: list attachments'
-  task ls: [:environment] do |_t, _args|
+  task list: [:environment] do |_t, _args|
     ::ActiveStorage::Blob.order(:filename).pluck(:byte_size, :created_at, :filename).each do |size, dt, filename|
       size_k = (size / 1024).to_s.rjust(7)
       date = dt.strftime('%Y-%m-%d %H:%M')
@@ -11,13 +11,14 @@ namespace :asdb do
   end
 
   desc 'ActiveStorageDB: download attachment'
-  task :cp, [:src, :dst] => [:environment] do |_t, args|
+  task :get, [:src, :dst] => [:environment] do |_t, args|
     src = args[:src]&.strip
     dst = args[:dst]&.strip
     abort('Required arguments: source file, destination file') if src.blank? || dst.blank?
 
     dst = "#{dst}/#{src}" if Dir.exist?(dst)
-    abort("Can't write on: #{dst}") unless File.writable?(dst)
+    dir = File.dirname(dst)
+    abort("Can't write on: #{dir}") unless File.writable?(dir)
 
     blob = ::ActiveStorage::Blob.order(created_at: :desc).find_by(filename: src)
     abort('Source file not found') unless blob

spec/tasks/active_storage_db_tasks_spec.rb

@@ -3,8 +3,8 @@
 RSpec.describe 'ActiveStorageDB tasks' do # rubocop:disable RSpec/DescribeClass
   include_context 'with rake tasks'
 
-  describe 'asdb:ls' do
-    subject(:task) { execute_task('asdb:ls') }
+  describe 'asdb:list' do
+    subject(:task) { execute_task('asdb:list') }
 
     let(:file1) { create(:active_storage_blob, filename: 'file1', created_at: Time.now - 1.hour) }
     let(:file2) { create(:active_storage_blob, filename: 'file2', created_at: Time.now - 5.hour) }
@@ -21,8 +21,8 @@
     end
   end
 
-  describe 'asdb:cp' do
-    subject(:task) { execute_task('asdb:cp', options) }
+  describe 'asdb:get' do
+    subject(:task) { execute_task('asdb:get', options) }
 
     let(:options) {}