class SomervilleStarImporters

  def initialize(options = {})
    @school_scope = options["school"]
    @progress_bar = options["progress_bar"]
    @log = options["test_mode"] ? LogHelper::Redirect.instance.file : STDOUT
  end

  def sftp_client
    @client ||= SftpClient.for_star
  end

  def file_importer_classes
    [
      StarReadingImporter,
      StarReadingImporter::HistoricalImporter,
      StarReadingImporter::RecentImporter,

      StarMathImporter,
      StarMathImporter::HistoricalImporter,
      StarMathImporter::RecentImporter,
    ]
  end

  def file_importers
    file_importer_classes.map do |file_importer_class|
      file_importer_class.new(@school_scope, sftp_client, @log, @progress_bar)
    end
  end

end

class SomervilleStarImporters

  def initialize(options = {})
    @school_scope = options["school"]
    @progress_bar = options["progress_bar"]
    @log = options["test_mode"] ? LogHelper::Redirect.instance.file : STDOUT
  end

  def sftp_client
    @client ||= SftpClient.for_star
  end

  def file_importer_classes
    [
      StarReadingImporter::RecentImporter,
      StarMathImporter::RecentImporter,
    ]
  end

  def file_importers
    file_importer_classes.map do |file_importer_class|
      file_importer_class.new(@school_scope, sftp_client, @log, @progress_bar)
    end
  end

end

Stop looking at historial STAR data on every import
Stop looking at historial STAR data on every import

+ According to our partners at STAR, the "Generic SM Extract.csv" file naming convention is the active one
+ So that's the only one we want to be reading when it comes to the regular, nightly data import process
