Merge pull request #48 from nazwr/15-add---quiet-option-when-running-to-suppress-logging

15 add   quiet option when running to suppress logging

Author: nazwr

README.md

@@ -22,6 +22,7 @@ With Deadlist, you can download audio files in any of the formats available, reg
 | Directory         | -d, --directory | Custom download location. Defaults to ./shows              |
 | Help              | -h, --help      | Show help documentation                                    |
 | Version           | -v, --version   | Show version of DeadList                                   |
+| Quiet             | -q, --quiet     | Suppresses logger output for run                           |
 
 
 ### Advanced Usage

features/cli_integration.feature

@@ -76,3 +76,15 @@ Feature: End-to-end CLI integration
     And it should create the show successfully
     And it should display "Test Download, skipping" message
     And the process should complete without downloads
+
+  Scenario: Run with --quiet flag suppresses info messages
+    Given I have valid arguments "--id gd1977-05-08 --format mp3 --quiet"
+    When I run the DeadList CLI
+    Then it should not display the startup banner
+    And it should not display info messages
+
+  Scenario: Quiet mode still shows errors
+    Given I have arguments with invalid show ID "--id invalid-show --format mp3 --quiet"
+    When I run the DeadList CLI with error handling
+    Then it should display a scraping error message
+    And it should not display the startup banner

features/step_definitions/cli_integration_steps.rb

@@ -293,3 +293,13 @@ def with_argv(args)
   # No download errors, but also no actual downloads
   expect(@output).not_to include("Download failed")
 end
+
+Then('it should not display the startup banner') do
+  expect(@output).not_to include("One man gathers what another man spills")
+  expect(@output).not_to include("=" * 52)
+end
+
+Then('it should not display info messages') do
+  expect(@output).not_to include("tracks found")
+  expect(@output).not_to include("💿")
+end

lib/deadlist.rb

@@ -11,32 +11,37 @@ class DeadList
     attr_reader :current_version
 
     def initialize(logger: Logger.new($stdout))
-        @logger = logger
-        @logger.level = Logger::INFO
-        @logger.formatter = proc do |severity, datetime, progname, msg|
+      @logger = logger
+      @logger.level = Logger::INFO  # Default level
+      @logger.formatter = proc do |severity, datetime, progname, msg|
           "#{msg}\n"
-        end
-        @current_version = VERSION
+      end
+      @current_version = VERSION
     end
 
     # Argument abstraction should probably happen at this level!
 
     def run(argv = ARGV)
-        # Start a new CLI session
-        # In future this could be abstracted to pass the show link vs all args, so a 'session' is started per show.
-        session = CLI.new(@current_version, argv, logger: @logger)
-
-        # Scrape links and metadata for given show
-        session.create_show
-
-        # In future, consider starting multiple downloaders for a list of shows
-        # show_list = session.args[:shows]
-        # show_list.each do |show|
-        #   session.download_show(show)
-        # end
-
-        # Create folder with show date and begin track downloads if format matches
-        session.download_show
+      # Check for --quiet flag and adjust logger level
+      if argv.include?('--quiet') || argv.include?('-q')
+          @logger.level = Logger::ERROR
+      end
+      
+      # Start a new CLI session
+      # In future this could be abstracted to pass the show link vs all args, so a 'session' is started per show.
+      session = CLI.new(@current_version, argv, logger: @logger)
+
+      # Scrape links and metadata for given show
+      session.create_show
+
+      # In future, consider starting multiple downloaders for a list of shows
+      # show_list = session.args[:shows]
+      # show_list.each do |show|
+      #   session.download_show(show)
+      # end
+
+      # Create folder with show date and begin track downloads if format matches
+      session.download_show
     end
 end
 

lib/deadlist/cli/argument_parser.rb

@@ -31,6 +31,10 @@ def self.parse(args, version)
         puts "deadlist v#{version}"
         exit
       end
+
+      opts.on('-q', '--quiet', 'Run silently') do
+        params[:quiet] = true
+      end
     end
 
     parser.parse!(args)