extracting ForFileOutputBuilder for clarity

Author: perryqh

lib/code_ownership/cli.rb

@@ -111,54 +111,7 @@ def self.for_file(argv)
         raise "Please pass in one file. Use `#{EXECUTABLE} for_file --help` for more info"
       end
 
-      if options[:verbose]
-        for_file_verbose(file: files.first, json: options[:json])
-      else
-        for_file_terse(file: files.first, json: options[:json])
-      end
-    end
-
-    def self.for_file_terse(file:, json:)
-      team = CodeOwnership.for_file(file)
-
-      team_name = team&.name || 'Unowned'
-      team_yml = team&.config_yml || 'Unowned'
-
-      if json
-        json_output = {
-          team_name: team_name,
-          team_yml: team_yml
-        }
-
-        puts json_output.to_json
-      else
-        puts <<~MSG
-          Team: #{team_name}
-          Team YML: #{team_yml}
-        MSG
-      end
-    end
-
-    def self.for_file_verbose(file:, json:)
-      verbose = CodeOwnership.for_file_verbose(file) || {team_name: 'Unowned', team_config_yml: 'Unowned', reasons: []}
-
-      if json
-        json = {
-          team_name: verbose[:team_name],
-          team_yml: verbose[:team_config_yml],
-          reasons: verbose[:reasons]
-        }
-
-        puts json.to_json
-      else
-        messages = ["Team: #{verbose[:team_name]}", "Team YML: #{verbose[:team_config_yml]}"]
-        if verbose[:reasons].any?
-          messages << "Reasons:\n- #{verbose[:reasons].join("\n-")}"
-        end
-        messages.last << "\n"
-
-        puts messages.join("\n")
-      end
+      puts CodeOwnership::Private::ForFileOutputBuilder.build(file_path: files.first, json: !!options[:json], verbose: !!options[:verbose])
     end
 
     def self.for_team(argv)