dropcaster 0.0.8 → 1.0.0

data/lib/dropcaster/errors.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Dropcaster
   class ConfigurationError < StandardError
     def initialize(msg)
@@ -6,14 +8,14 @@ module Dropcaster
   end
 
   class MissingAttributeError < ConfigurationError
-    def initialize(missingAttribute)
-      super("#{missingAttribute} is a mandatory channel attribute, but it is missing.")
+    def initialize(missing_attribute)
+      super("#{missing_attribute} is a mandatory channel attribute, but it is missing.")
     end
   end
 
   class AmbiguousSourcesError < ConfigurationError
-    def initialize(ambiguousSources)
-      super("The list of sources is ambiguous. Can't derive common directory from these: #{ambiguousSources.inspect}")
+    def initialize(ambiguous_sources)
+      super("The list of sources is ambiguous. Can't derive common directory from these: #{ambiguous_sources.inspect}")
     end
   end
 

data/lib/dropcaster/item.rb

@@ -1,31 +1,37 @@
+# frozen_string_literal: true
+
+require 'pathname'
 require 'mp3info'
 require 'digest/sha1'
+require 'dropcaster/logging'
 
 module Dropcaster
   class Item
-    attr_reader :file_name, :tag, :tag2, :duration, :file_size, :uuid, :pub_date, :pub_date, :lyrics
+    include Logging
+
+    attr_reader :file_path, :tag, :tag2, :duration, :file_size, :uuid, :pub_date, :lyrics
     attr_accessor :artist, :image_url, :url, :keywords
 
-    def initialize(file_path, options = nil)
-      Mp3Info.open(file_path){|mp3info|
-        @file_name = Pathname.new(File.expand_path(file_path)).relative_path_from(Pathname.new(Dir.pwd)).cleanpath.to_s
+    def initialize(file_path, options=nil)
+      Mp3Info.open(file_path) { |mp3info|
+        @file_path = Pathname.new(File.expand_path(file_path)).relative_path_from(Pathname.new(Dir.pwd)).cleanpath
         @tag = mp3info.tag
         @tag2 = mp3info.tag2
         @duration = mp3info.length
-        if @tag2["ULT"]
-          @lyrics = {};
-          @tag2["ULT"].split(/\x00/).drop(1).each_slice(2) { |k, v| @lyrics[k] = v }
+        if @tag2['ULT']
+          @lyrics = {}
+          @tag2['ULT'].split(/\x00/).drop(1).each_slice(2) { |k, v| @lyrics[k] = v }
         end
       }
 
-      @file_size = File.new(@file_name).stat.size
-      @uuid = Digest::SHA1.hexdigest(File.read(file_name))
+      @file_size = File.new(file_path).stat.size
+      @uuid = Digest::SHA1.hexdigest(File.read(file_path))
 
-      unless tag2.TDR.blank?
-        @pub_date = DateTime.parse(tag2.TDR)
+      if tag2.TDR.blank?
+        logger.info("#{file_path} has no pub date set, using the file's modification time")
+        @pub_date = Time.parse(File.new(file_path).mtime.to_s)
       else
-        Dropcaster.logger.info("#{file_path} has no pub date set, using the file's modification time")
-        @pub_date = DateTime.parse(File.new(file_name).mtime.to_s)
+        @pub_date = Time.parse(tag2.TDR)
       end
     end
   end

data/lib/dropcaster/log_formatter.rb

@@ -1,6 +1,16 @@
+# frozen_string_literal: true
+
 require 'logger'
 
 module Dropcaster
+  module Logging
+    attr_writer :logger
+
+    def logger
+      @logger ||= NullLogger.new
+    end
+  end
+
   class LogFormatter < Logger::Formatter
     def call(severity, time, program_name, message)
       "#{severity}: #{message}\n"

data/lib/dropcaster/logging.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+require 'null_logger'
+
+module Dropcaster
+  module Logging
+    attr_writer :logger
+
+    def logger
+      @logger ||= NullLogger.new
+    end
+  end
+end

data/lib/dropcaster/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Dropcaster
-  VERSION = '0.0.8'
+  VERSION = '1.0.0'
 end

data/lib/dropcaster.rb

@@ -1,17 +1,12 @@
-$:.unshift File.dirname(__FILE__)
+# frozen_string_literal: true
 
-require 'rubygems'
-require 'bundler/setup'
+$LOAD_PATH.unshift File.dirname(__FILE__)
 
 require 'yaml'
 require 'active_support/core_ext/date_time/conversions'
 require 'active_support/core_ext/object/blank'
-require 'active_support/core_ext/module/attribute_accessors'
-
-require 'logger'
 
 require 'dropcaster/errors'
-require 'dropcaster/log_formatter'
 require 'dropcaster/channel'
 require 'dropcaster/item'
 require 'dropcaster/channel_file_locator'
@@ -19,12 +14,4 @@ require 'dropcaster/version'
 
 module Dropcaster
   CHANNEL_YML = 'channel.yml'
-
-  mattr_accessor :logger
-
-  unless @@logger
-    @@logger = Logger.new(STDERR)
-    @@logger.level = Logger::WARN
-    @@logger.formatter = LogFormatter.new
-  end
 end

data/templates/channel.html.erb

@@ -26,7 +26,7 @@
   <h1>Episodes</h1>
   <% items.each{|item| %>
     <div class="item" id="<%= h(item.uuid) %>">
-      <h1><%= item.tag.title || item.tag2.TIT2%></h1>
+      <h1><%= h(item.tag.title || item.tag2.TIT2 || item.file_path.to_s) %></h1>
     <% unless item.tag2.SUBTITLE.blank? %>
       <h2><%= h(item.tag2.SUBTITLE) %></h2>
     <% end %>

data/templates/channel.rss.erb

@@ -45,7 +45,7 @@
   <% end %>
   <% items.each{|item| %>
     <item>
-      <title><%= item.tag.title || item.tag2.TIT2 || item.file_name%></title>
+      <title><%= h(item.tag.title || item.tag2.TIT2 || item.file_path.to_s) %></title>
       <itunes:author><%= h(item.tag2.TP1 || item.tag2.TPE1) %></itunes:author>
     <% unless item.tag2.TT3.blank? %>
       <itunes:subtitle><%= h(truncate(item.tag2.TT3, 50)) %></itunes:subtitle>

data/test/Vagrantfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # -*- mode: ruby -*-
 # vi: set ft=ruby :
 
@@ -9,7 +11,7 @@ Vagrant.configure(2) do |config|
     vb.memory = '256'
   end
 
-  %w(1.9.1 2.0 2.1 2.2).each do |version|
+  %w[1.9.1 2.0 2.1 2.2].each do |version|
     ruby = "ruby#{version}"
     name = "ruby-#{version.tr('.', '-')}"
 

data/test/bin/vagrant-status

@@ -1,15 +1,22 @@
 #!/usr/bin/env ruby
-#
+# frozen_string_literal: true
+
 require 'csv'
 
-headers = [
-  'timestamp', # is a Unix timestamp in UTC of when the message was printed.
-  'target',    # is the target of the following output. This is empty if the message is related to Vagrant globally. Otherwise, this is generally a machine name so you can relate output to a specific machine when multi-VM is in use.
-  'type',      # is the type of machine-readable message being outputted. There are a set of standard types which are covered later.
-  'data',      # is zero or more comma-separated values associated with the prior type. The exact amount and meaning of this data is type-dependent, so you must read the documentation associated with the type to understand fully.
-]
+HEADERS = [
+  # Unix timestamp in UTC of when the message was printed.
+  'timestamp',
+  # is the target of the following output. This is empty if the message is related to Vagrant globally. Otherwise, this is generally a machine name so you can relate output to a
+  # specific machine when multi-VM is in use.
+  'target',
+  # is the type of machine-readable message being outputted. There are a set of standard types which are covered later.
+  'type',
+  # is zero or more comma-separated values associated with the prior type. The exact amount and meaning of this data is
+  # type-dependent, so you must read the documentation associated with the type to understand fully.
+  'data'
+].freeze
 
-types = [
+TYPES = [
   'box-name',          # Name of a box installed into Vagrant.
   'box-provider',      # Provider for an installed box.
   'cli-command',       # A subcommand of vagrant that is available.
@@ -18,7 +25,7 @@ types = [
   'state',             # The state ID of the target machine. targeted
   'state-human-long',  # Human-readable description of the state of the machine. This is the long version, and may be a paragraph or longer. targeted
   'state-human-short', # Human-readable description of the state of the machine. This is the short version, limited to at most a sentence. targeted
-]
+].freeze
 
 module Vagrant
   Box = Struct.new(:name, :provider)
@@ -40,31 +47,30 @@ virtual_machines = Hash.new do |vms, target|
   vms[target] = Vagrant::VM.new(target)
 end
 
-CSV.new(ARGF, headers: headers).each do |message|
+CSV.new(ARGF, headers: HEADERS).each do |message|
   vm = virtual_machines[message['target']]
   data = message['data'].split('%!(VAGRANT_COMMA)')
 
   case message['type']
-    when 'box-name'
-      vm.box.name = data
-    when 'box-provider'
-      vm.box.provider = data
-    when 'state'
-      vm.state.id = data
-    when 'state-human-long'
-      vm.state.long = data
-    when 'state-human-short'
-      vm.state.short = data
-    when 'cli-command'
-      vm.command = data
-    when 'error-exit'
-      vm.error = Vagrant::Error.new(data[0], data[1])
-    when 'provider-name'
-      vm.provider = data
-    else
-      raise "Unknown type: #{message['type']}"
+  when 'box-name'
+    vm.box.name = data
+  when 'box-provider'
+    vm.box.provider = data
+  when 'state'
+    vm.state.id = data
+  when 'state-human-long'
+    vm.state.long = data
+  when 'state-human-short'
+    vm.state.short = data
+  when 'cli-command'
+    vm.command = data
+  when 'error-exit'
+    vm.error = Vagrant::Error.new(data[0], data[1])
+  when 'provider-name'
+    vm.provider = data
+  else
+    raise "Unknown type: #{message['type']}"
   end
 end
 
-puts virtual_machines.map{|name, vm| name}
-#puts virtual_machines.map{|name, vm| "#{name}: #{vm.state.short}"}
+puts(virtual_machines.map { |name, _| name })

data/test/extensions/windows.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Returns true if we run on a windows platform
 #
 # Sample:
@@ -7,6 +9,6 @@
 # http://snippets.dzone.com/posts/show/2112
 #
 def Kernel.is_windows?
-  processor, platform, *rest = RUBY_PLATFORM.split("-")
+  _, platform, _ = RUBY_PLATFORM.split('-')
   platform == 'mingw32'
 end

data/test/helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 gem 'minitest'
 require 'minitest/autorun'
 require 'dropcaster'
@@ -7,5 +9,6 @@ require_relative 'extensions/windows'
 module DropcasterTest
   FIXTURES_DIR = File.join(File.dirname(__FILE__), 'fixtures')
   FIXTURE_ITUNES_MP3 = File.join(FIXTURES_DIR, 'iTunes.mp3')
-  NS_ITUNES = "itunes:http://www.itunes.com/dtds/podcast-1.0.dtd"
+  NUMBER_OF_MP3_FILES = 3
+  NS_ITUNES = 'itunes:http://www.itunes.com/dtds/podcast-1.0.dtd'
 end

data/test/unit/test_app.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'helper'
 require 'open3'
 require 'uri'
@@ -14,24 +16,24 @@ class TestApp < TestChannelXML
   APP_SCRIPT = 'ruby bin/dropcaster'
 
   def channel_rss
-    %x[#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3}]
+    `#{APP_SCRIPT} #{FIXTURES_DIR}`
   end
 
   def test_overwrite_title
     test_title = 'Alice and Bob in Wonderland'
-    channel = channel_node(%x[#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3} --title '#{test_title}'])
+    channel = channel_node(`#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3} --title '#{test_title}'`)
     assert_equal(test_title, channel.find('title').first.content)
   end
 
   def test_overwrite_subtitle
     test_subtitle = 'Tales from another world that is upside down'
-    channel = channel_node(%x[#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3} --subtitle '#{test_subtitle}'])
+    channel = channel_node(`#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3} --subtitle '#{test_subtitle}'`)
     assert_equal(test_subtitle, channel.find('itunes:subtitle').first.content)
   end
 
   def test_overwrite_link
     test_link = 'http://www.example.com/foo/bar'
-    channel = channel_node(%x[#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3} --url '#{test_link}'])
+    channel = channel_node(`#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3} --url '#{test_link}'`)
     assert_equal(test_link, channel.find('link').first.content)
   end
 
@@ -39,14 +41,14 @@ class TestApp < TestChannelXML
     test_link_base = 'http://www.dropbox.com/foo/bar/'
     test_link_file = 'index.html'
     test_link = "#{test_link_base}#{test_link_file}"
-    channel = channel_node(%x[#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3} --url '#{test_link}'])
+    channel = channel_node(`#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3} --url '#{test_link}'`)
     assert_equal(test_link, channel.find('link').first.content)
 
     options = YAML.load_file(File.join(FIXTURES_DIR, Dropcaster::CHANNEL_YML))
     assert_equal(URI.join(test_link_base, options[:image_url]).to_s, channel.find('itunes:image').first['href'])
 
     # check that the item URLs are correct, too
-    item = channel.find("item").first
+    item = channel.find('item').first
     assert(item)
 
     # enclosure
@@ -59,42 +61,43 @@ class TestApp < TestChannelXML
   end
 
   def test_dir_only
-    channel = channel_node(%x[#{APP_SCRIPT} #{FIXTURES_DIR}])
-    assert_equal(1, channel.find('item').size)
+    assert_equal(NUMBER_OF_MP3_FILES, @channel.find('item').size)
   end
 
   def test_overwrite_enclosures_url
     test_enclosures_url = 'http://www.example.com/foo/bar/episodes/'
-    channel = channel_node(%x[#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3} --enclosures '#{test_enclosures_url}'])
-    item = channel.find("item").first
+    channel = channel_node(`#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3} --enclosures '#{test_enclosures_url}'`)
+    item = channel.find('item').first
     assert(item)
     enclosure = item.find('enclosure').first
     assert(enclosure)
-    assert_equal(URI.join(test_enclosures_url,'test/fixtures/iTunes.mp3').to_s, enclosure['url'])
+    assert_equal(URI.join(test_enclosures_url, 'test/fixtures/iTunes.mp3').to_s, enclosure['url'])
   end
 
   def test_overwrite_image_url
     test_image_url = 'http://www.example.com/foo/bar/override.gif'
-    channel = channel_node(%x[#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3} --image '#{test_image_url}'])
+    channel = channel_node(`#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3} --image '#{test_image_url}'`)
     assert_equal(test_image_url, channel.find('itunes:image').first['href'])
 
     # Make sure the items pick up this URL, too
-    item = channel.find("item").first
+    item = channel.find('item').first
     assert(item)
     assert_equal(test_image_url, item.find('itunes:image').first['href'])
   end
 
   def test_overwrite_description
     test_description = 'Testing commandline apps is not that hard.'
-    channel = channel_node(%x[#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3} --description '#{test_description}'])
+    channel = channel_node(`#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3} --description '#{test_description}'`)
     assert_equal(test_description, channel.find('description').first.content)
     assert_equal(test_description, channel.find('itunes:summary', NS_ITUNES).first.content)
   end
 
   def test_no_channel_file
-    Open3.popen3(APP_SCRIPT){|stdin, stdout, stderr|
-      assert_match(/ERROR: No channel file found/, stderr.read)
-    } unless Kernel.is_windows?
+    unless Kernel.is_windows?
+      Open3.popen3(APP_SCRIPT) { |stdin, stdout, stderr|
+        assert_match(/ERROR: No channel file found/, stderr.read)
+      }
+    end
   end
 
   def test_overwrite_all
@@ -102,7 +105,7 @@ class TestApp < TestChannelXML
     test_link = 'http://www.example.com/bar/foot'
     test_description = 'Testing commandline apps is really not that hard.'
 
-    channel = channel_node(%x[#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3} --title '#{test_title}' --url '#{test_link}' --description '#{test_description}'])
+    channel = channel_node(`#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3} --title '#{test_title}' --url '#{test_link}' --description '#{test_description}'`)
 
     assert_equal(test_title, channel.find('title').first.content)
     assert_equal(test_link, channel.find('link').first.content)
@@ -111,9 +114,11 @@ class TestApp < TestChannelXML
   end
 
   def test_channel_template_not_found
-    Open3.popen3("#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3} --channel-template foo/bar/42"){|stdin, stdout, stderr|
-      assert_match(/Unable to load template file/, stderr.read)
-    } unless Kernel.is_windows?
+    unless Kernel.is_windows?
+      Open3.popen3("#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3} --channel-template foo/bar/42") { |stdin, stdout, stderr|
+        assert_match(/Unable to load template file/, stderr.read)
+      }
+    end
   end
 
   #
@@ -121,9 +126,9 @@ class TestApp < TestChannelXML
   #
   def test_overwrite_channel_template
     channel_template = File.join(FIXTURES_DIR, 'test_template.json.erb')
-    channel = JSON.parse(%x[#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3} --channel-template #{channel_template}])
-    assert_equal("All About Everything", channel['channel']['title'])
+    channel = JSON.parse(`#{APP_SCRIPT} #{FIXTURE_ITUNES_MP3} --channel-template #{channel_template}`)
+    assert_equal('All About Everything', channel['channel']['title'])
   end
 
-  # TODO --channel
+  # TODO: --channel
 end

data/test/unit/test_channel.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'helper'
 
 class TestChannel < MiniTest::Test
@@ -9,7 +11,7 @@ class TestChannel < MiniTest::Test
   end
 
   def test_item_count
-    assert_equal(1, @channel.items.size)
+    assert_equal(NUMBER_OF_MP3_FILES, @channel.items.size)
   end
 
   def test_channel
@@ -47,7 +49,9 @@ class TestChannel < MiniTest::Test
   end
 
   def test_channel_explicit_nil
-    assert_channel_explicit(nil, nil)
+    @options[:explicit] = nil
+    channel = Dropcaster::Channel.new(FIXTURES_DIR, @options)
+    assert_nil(channel.explicit)
   end
 
   def test_channel_explicit_clean
@@ -62,19 +66,19 @@ class TestChannel < MiniTest::Test
 
   def test_raise_on_missing_title
     assert_raises Dropcaster::MissingAttributeError do
-      Dropcaster::Channel.new(FIXTURES_DIR, {:url => 'bar', :description => 'foobar'})
+      Dropcaster::Channel.new(FIXTURES_DIR, { url: 'bar', description: 'foobar' })
     end
   end
 
   def test_raise_on_missing_url
     assert_raises Dropcaster::MissingAttributeError do
-      Dropcaster::Channel.new(FIXTURES_DIR, {:title => 'foo', :description => 'foobar'})
+      Dropcaster::Channel.new(FIXTURES_DIR, { title: 'foo', description: 'foobar' })
     end
   end
 
   def test_raise_on_missing_description
     assert_raises Dropcaster::MissingAttributeError do
-      Dropcaster::Channel.new(FIXTURES_DIR, {:title => 'foo', :url => 'bar'})
+      Dropcaster::Channel.new(FIXTURES_DIR, { title: 'foo', url: 'bar' })
     end
   end
 end

data/test/unit/test_channel_locator.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'helper'
 require 'tmpdir'
 require 'pry'
@@ -8,6 +10,7 @@ class TestChannelLocator < MiniTest::Test
   def setup
     @temp_dir = Dir.mktmpdir
   end
+
   def teardown
     FileUtils.remove_entry_secure(@temp_dir)
   end
@@ -30,7 +33,7 @@ class TestChannelLocator < MiniTest::Test
   end
 
   def test_array_of_files_same_dir
-    sources = Array.new
+    sources = []
     sources << File.join(@temp_dir, 'file1.mp3')
     sources << File.join(@temp_dir, 'file2.mp3')
     sources << File.join(@temp_dir, 'file3.mp3')
@@ -39,7 +42,7 @@ class TestChannelLocator < MiniTest::Test
   end
 
   def test_array_of_files_different_dir
-    sources = Array.new
+    sources = []
     sources << File.join(@temp_dir, 'foo', 'file1.mp3')
     sources << File.join(@temp_dir, 'bar', 'file1.mp3')
     sources << File.join(@temp_dir, 'baz', 'file1.mp3')
@@ -54,9 +57,9 @@ class TestChannelLocator < MiniTest::Test
   end
 
   def test_array_with_more_than_a_single_directory
-    Dir.mktmpdir{|tmp_dir1|
-      Dir.mktmpdir{|tmp_dir2|
-        sources = Array.new
+    Dir.mktmpdir { |tmp_dir1|
+      Dir.mktmpdir { |tmp_dir2|
+        sources = []
         sources << File.join(tmp_dir1, 'another_dir')
         sources << File.join(tmp_dir2, 'another_dir')
 

data/test/unit/test_channel_xml.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'helper'
 require 'xml/libxml'
 require 'pry'
@@ -14,7 +16,7 @@ class TestChannelXML < MiniTest::Test
   # Returnes the XML node for the channel passed as XML string
   #
   def channel_node(rss)
-    XML::Document.string(rss).find("//rss/channel").first
+    XML::Document.string(rss).find('//rss/channel').first
   end
 
   #
@@ -27,11 +29,10 @@ class TestChannelXML < MiniTest::Test
     Dropcaster::Channel.new(FIXTURES_DIR, @options.dup).to_rss
   end
 
-  def test_item
-    item = @channel.find("item").first
+  def test_itunes_item
+    item = @channel.find('item[title = "iTunes Name"]').first
     assert(item)
 
-    assert_equal('iTunes Name', item.find('title').first.content)
     assert_equal('iTunes Artist', item.find('itunes:author', NS_ITUNES).first.content)
     assert_equal('iTunes Description (Video Pane)', item.find('itunes:summary', NS_ITUNES).first.content)
     assert_equal('http://www.example.com/podcasts/everything/AllAboutEverything.jpg', item.find('itunes:image', NS_ITUNES).first['href'])
@@ -51,12 +52,31 @@ class TestChannelXML < MiniTest::Test
     assert_equal('3', item.find('itunes:duration', NS_ITUNES).first.content)
   end
 
+  def test_special_ampersand_item
+    # in the actual XML, this is "special &amp;.mp3", but it gets interpreted by XML::Document
+    # if it was just "special &.mp3", it would be invalid XML and we wouldn't get this far
+    item = @channel.find('item[title = "test/fixtures/special &.mp3"]').first
+    assert(item)
+
+    enclosure = item.find('enclosure').first
+    assert(enclosure)
+    assert_equal('http://www.example.com/podcasts/everything/test/fixtures/special%20%26.mp3', enclosure['url'])
+  end
+
+  def test_uppercase_extension_item
+    item = @channel.find('item[title = "test/fixtures/extension.MP3"]').first
+    assert(item)
+
+    enclosure = item.find('enclosure').first
+    assert(enclosure)
+    assert_equal('http://www.example.com/podcasts/everything/test/fixtures/extension.MP3', enclosure['url'])
+  end
+
   def test_attributes_mandatory
-    options = {:title => 'Test Channel',
-               :url => 'http://www.example.com/',
-               :description => 'A test channel',
-               :enclosures_url => 'http://www.example.com/foo/bar',
-              }
+    options = { title: 'Test Channel',
+                url: 'http://www.example.com/',
+                description: 'A test channel',
+                enclosures_url: 'http://www.example.com/foo/bar' }
 
     channel = channel_node(Dropcaster::Channel.new(FIXTURES_DIR, options).to_rss)
     assert_equal('Test Channel', channel.find('title').first.content)

data/test/unit/{test_item.rb → test_itunes_item.rb}

@@ -1,6 +1,8 @@
+# frozen_string_literal: true
+
 require 'helper'
 
-class TestItem < MiniTest::Test
+class TestItunesItem < MiniTest::Test
   include DropcasterTest
 
   def setup
@@ -12,7 +14,7 @@ class TestItem < MiniTest::Test
     assert_equal(58119, @item.file_size)
     assert_equal('77bf84447c0f69ce4a33a18b0ae1e030b82010de', @item.uuid)
     assert_equal(File.mtime(FIXTURE_ITUNES_MP3).to_i, @item.pub_date.to_i)
-    assert_equal('test/fixtures/iTunes.mp3', @item.file_name)
+    assert_equal('test/fixtures/iTunes.mp3', @item.file_path.to_s)
   end
 
   def test_tag