royw-dvdprofiler2xbmc 0.0.6 → 0.0.8

data/spec/app_spec.rb

@@ -0,0 +1,146 @@
+require File.dirname(__FILE__) + '/spec_helper.rb'
+
+require 'tempfile'
+
+describe "App" do
+
+  before(:all) do
+    logger = Log4r::Logger.new('dvdprofiler2xbmc')
+    logger.outputters = Log4r::StdoutOutputter.new(:console)
+    Log4r::Outputter[:console].formatter  = Log4r::PatternFormatter.new(:pattern => "%m")
+    logger.level = Log4r::WARN
+    AppConfig.default
+    AppConfig[:logger] = logger
+    AppConfig.load
+    File.mkdirs(TMPDIR)
+    AppConfig[:logger].warn { "\nApp Specs" }
+  end
+
+  after(:all) do
+    Dir.glob(File.join(TMPDIR, "app_*")).each do |filename|
+      File.delete(filename) if File.file?(filename)
+      Dir.rmdir(filename) if File.directory?(filename)
+    end
+  end
+
+  it 'should save a file' do
+    # get a temp filename to a non-existing file
+    outfile = Tempfile.new('tmdb_movie_spec_saving', TMPDIR)
+    filespec = outfile.path
+    outfile.unlink
+
+    data = 'Howdy Partner'
+
+    DvdProfiler2Xbmc.save_to_file(filespec, data)
+    open(filespec).read.should == "#{data}\n"
+  end
+
+  it 'should backup the file before saving' do
+    # get a temp filename to a non-existing file
+    outfile = Tempfile.new('tmdb_movie_spec_saving', TMPDIR)
+    filespec = outfile.path
+    backupspec = filespec + AppConfig[:extensions][:backup]
+    outfile.unlink
+
+    data = []
+    data[0] = 'Howdy Partner'
+    data[1] = 'Howdy Ho Neighbor'
+
+    DvdProfiler2Xbmc.save_to_file(filespec, data[0])
+    DvdProfiler2Xbmc.save_to_file(filespec, data[1])
+    open(filespec).read.should == "#{data[1]}\n" &&
+    open(backupspec).read.should == "#{data[0]}\n"
+  end
+
+  it 'should generate .nfo filespec for single part media' do
+    filespec = DvdProfiler2Xbmc.generate_filespec('/a/b/c.m4v', :nfo)
+    filespec.should == '/a/b/c.nfo'
+  end
+
+  it 'should generate .nfo filespec for multiple part media' do
+    filespec = DvdProfiler2Xbmc.generate_filespec('/a/b/c.cd1.m4v', :nfo)
+    filespec.should == '/a/b/c.nfo'
+  end
+
+  it 'should generate .tbn filespec for single part media' do
+    filespec = DvdProfiler2Xbmc.generate_filespec('/a/b/c.m4v', :thumbnail)
+    filespec.should == '/a/b/c.tbn'
+  end
+
+  it 'should generate .tbn filespec for multiple part media' do
+    filespec = DvdProfiler2Xbmc.generate_filespec('/a/b/c.cd1.m4v', :thumbnail)
+    filespec.should == '/a/b/c.tbn'
+  end
+
+  it 'should generate -fanart filespec for single part media' do
+    filespec = DvdProfiler2Xbmc.generate_filespec('/a/b/c.m4v', :fanart)
+    filespec.should == '/a/b/c-fanart'
+  end
+
+  it 'should generate -fanart filespec for multiple part media' do
+    filespec = DvdProfiler2Xbmc.generate_filespec('/a/b/c.cd1.m4v', :fanart)
+    filespec.should == '/a/b/c-fanart'
+  end
+
+  it 'should generate -fanart.jpg filespec for single part media' do
+    filespec = DvdProfiler2Xbmc.generate_filespec('/a/b/c.m4v', :fanart, :extension => File.extname('foo.jpg'))
+    filespec.should == '/a/b/c-fanart.jpg'
+  end
+
+  it 'should generate -fanart.jpg filespec for multiple part media' do
+    filespec = DvdProfiler2Xbmc.generate_filespec('/a/b/c.cd1.m4v', :fanart, :extension => File.extname('foo.jpg'))
+    filespec.should == '/a/b/c-fanart.jpg'
+  end
+
+  it 'should generate .imdb.xml filespec for single part media' do
+    filespec = DvdProfiler2Xbmc.generate_filespec('/a/b/c.m4v', :imdb_xml)
+    filespec.should == '/a/b/c.imdb.xml'
+  end
+
+  it 'should generate .imdb.xml filespec for multiple part media' do
+    filespec = DvdProfiler2Xbmc.generate_filespec('/a/b/c.cd1.m4v', :imdb_xml)
+    filespec.should == '/a/b/c.imdb.xml'
+  end
+
+  it 'should generate .tmdb.xml filespec for single part media' do
+    filespec = DvdProfiler2Xbmc.generate_filespec('/a/b/c.m4v', :tmdb_xml)
+    filespec.should == '/a/b/c.tmdb.xml'
+  end
+
+  it 'should generate .tmdb.xml filespec for multiple part media' do
+    filespec = DvdProfiler2Xbmc.generate_filespec('/a/b/c.cd1.m4v', :tmdb_xml)
+    filespec.should == '/a/b/c.tmdb.xml'
+  end
+
+  it 'should generate .dvdprofiler.xml filespec for single part media' do
+    filespec = DvdProfiler2Xbmc.generate_filespec('/a/b/c.m4v', :dvdprofiler_xml)
+    filespec.should == '/a/b/c.dvdprofiler.xml'
+  end
+
+  it 'should generate .dvdprofiler.xml filespec for multiple part media' do
+    filespec = DvdProfiler2Xbmc.generate_filespec('/a/b/c.cd1.m4v', :dvdprofiler_xml)
+    filespec.should == '/a/b/c.dvdprofiler.xml'
+  end
+
+  it 'should change file permissions' do
+    outfile = Tempfile.new('app_permission_check', TMPDIR)
+    filespec = outfile.path
+    File.chmod(0777, filespec)
+    AppConfig.config[:file_permissions] = 0642.to_s(8)
+    # set_permissions is protected method
+    DvdProfiler2Xbmc.instance.send('set_permissions', TMPDIR)
+    (File.stat(filespec).mode & 07777).should == 0642
+  end
+
+  it 'should change directory permissions' do
+    outdir = Tempfile.new('app_permission_check', TMPDIR)
+    dirspec = outdir.path
+    outdir.unlink
+    Dir.mkdir(dirspec)
+    File.chmod(0777, dirspec)
+    AppConfig.config[:dir_permissions] = 0642.to_s(8)
+    # set_permissions is protected method
+    DvdProfiler2Xbmc.instance.send('set_permissions', TMPDIR)
+    (File.stat(dirspec).mode & 07777).should == 0642
+  end
+end

data/spec/config_editor_spec.rb

@@ -0,0 +1,301 @@
+require File.dirname(__FILE__) + '/spec_helper.rb'
+require File.join(File.dirname(__FILE__), '../lib/dvdprofiler2xbmc/views/config_editor')
+
+require 'tempfile'
+
+describe "ConfigEditor" do
+
+  before(:all) do
+    logger = Log4r::Logger.new('dvdprofiler2xbmc')
+    logger.outputters = Log4r::StdoutOutputter.new(:console)
+    Log4r::Outputter[:console].formatter  = Log4r::PatternFormatter.new(:pattern => "%m")
+    logger.level = Log4r::WARN
+    AppConfig.default
+    AppConfig[:logger] = logger
+    AppConfig.load
+    File.mkdirs(TMPDIR)
+    AppConfig[:logger].warn { "\nConfigEditor Specs" }
+  end
+
+  before(:each) do
+    @input    = StringIO.new
+    @output   = StringIO.new
+    @old_terminal = $terminal
+    $terminal = HighLine.new(@input, @output)
+    @editor = ConfigEditor.new
+  end
+
+  after(:each) do
+    $terminal = @old_terminal
+  end
+
+  it "should return true for boolean data type edits that recieve y" do
+    @input << "y\n"
+    @input.rewind
+    value = @editor.data_type_editor('field_name', :BOOLEAN, false)
+    value.should be_true
+  end
+
+  it "should return false for boolean data type edits that recieve n" do
+    @input << "n\n"
+    @input.rewind
+    value = @editor.data_type_editor('field_name', :BOOLEAN, true)
+    value.should be_false
+  end
+
+  it "should return true for boolean data type edits that recieve yes" do
+    @input << "yes\n"
+    @input.rewind
+    value = @editor.data_type_editor('field_name', :BOOLEAN, false)
+    value.should be_true
+  end
+
+  it "should return false for boolean data type edits that recieve no" do
+    @input << "no\n"
+    @input.rewind
+    value = @editor.data_type_editor('field_name', :BOOLEAN, true)
+    value.should be_false
+  end
+
+  it "should return false for boolean data type edits that default to false" do
+    @input << "\n\n"
+    @input.rewind
+    value = @editor.data_type_editor('field_name', :BOOLEAN, false)
+    value.should be_false
+  end
+
+  it "should return true for boolean data type edits that default to true" do
+    @input << "\n"
+    @input.rewind
+    value = @editor.data_type_editor('field_name', :BOOLEAN, true)
+    value.should be_true
+  end
+
+  it "should handle ask commands" do
+    pathspec = File.expand_path(__FILE__)
+    @input << pathspec << "\n"
+    @input.rewind
+    value = ask('New filespec: ')
+    value.should == pathspec
+  end
+
+  it "should handle ask commands with validation" do
+    pathspec = File.expand_path(__FILE__)
+    @input << pathspec << "\n"
+    @input.rewind
+    value = ask('New filespec: ') do |q|
+      q.validate = lambda { |p| File.exist?(p) && File.file?(p) }
+      q.confirm = false
+    end
+    value.should == pathspec
+  end
+
+  it "should return pathspec" do
+    pathspec = File.expand_path(File.dirname(__FILE__))
+    default_pathspec = File.expand_path(File.join(File.dirname(__FILE__), '..'))
+    @input << pathspec << "\n"
+    @input.rewind
+    value = @editor.data_type_editor('images_dir', :PATHSPEC, default_pathspec)
+    value.should == pathspec
+  end
+
+  it "should return default pathspec" do
+    default_pathspec = File.expand_path(File.join(File.dirname(__FILE__), '..'))
+    @input << "\n"
+    @input.rewind
+    value = @editor.data_type_editor('images_dir', :PATHSPEC, default_pathspec)
+    value.should == default_pathspec
+  end
+
+  it "should reprompt for invalid pathspec" do
+    badpathspec = '/a123/b123/c123'
+    goodpathspec = File.expand_path(File.dirname(__FILE__))
+    default_pathspec = File.expand_path(File.join(File.dirname(__FILE__), '..'))
+    @input << badpathspec << "\n" << goodpathspec << "\n"
+    @input.rewind
+    value = @editor.data_type_editor('images_dir', :PATHSPEC, default_pathspec)
+    value.should == goodpathspec
+  end
+
+  it "should return filespec data type edits" do
+    filespec = File.expand_path(__FILE__)
+    default_filespec = File.expand_path(File.join(File.dirname(__FILE__), 'ruby_dragon'))
+    @input << filespec << "\n"
+    @input.rewind
+    value = @editor.data_type_editor('collection_filespec', :FILESPEC, default_filespec)
+    value.should == filespec
+  end
+
+  it "should reprompt for invalid filespec" do
+    badfilespec = File.expand_path(__FILE__) + 'jaberwooky'
+    goodfilespec = File.expand_path(__FILE__)
+    default_filespec = File.expand_path(File.join(File.dirname(__FILE__), 'ruby_dragon'))
+    @input << badfilespec << "\n" << goodfilespec << "\n"
+    @input.rewind
+    value = @editor.data_type_editor('collection_filespec', :FILESPEC, default_filespec)
+    value.should == goodfilespec
+  end
+
+  it "should accept valid permission 0" do
+    @input << "0" << "\n"
+    @input.rewind
+    value = @editor.data_type_editor('file_permissions', :PERMISSIONS, 0644.to_s(8))
+    value.should == 0.to_s(8)
+  end
+
+  it "should accept valid permission 7777" do
+    @input << "7777" << "\n"
+    @input.rewind
+    value = @editor.data_type_editor('file_permissions', :PERMISSIONS, 0644.to_s(8))
+    value.should == 07777.to_s(8)
+  end
+
+  it "should reject permission > 7777" do
+    @input << "65432" << "\n" << "6543" << "\n"
+    @input.rewind
+    value = @editor.data_type_editor('file_permissions', :PERMISSIONS, 0644.to_s(8))
+    value.should == 06543.to_s(8)
+  end
+
+  it "should accept array of strings" do
+    strings = %w(foo bar howdy)
+    @input << strings.join("\n") << "\n\n"
+    @input.rewind
+    value = @editor.data_type_editor('image_extensions', :ARRAY_OF_STRINGS)
+    value.should == strings
+  end
+
+  it "should accept empty array of strings" do
+    strings = []
+    @input << strings.join("\n") << "\n\n"
+    @input.rewind
+    value = @editor.data_type_editor('image_extensions', :ARRAY_OF_STRINGS)
+    value.should == strings
+  end
+
+  it "should accept array of pathspecs" do
+    cwd = File.expand_path(File.dirname(__FILE__))
+    pathspecs = [cwd, File.join(cwd, '..')]
+    @input << pathspecs.join("\n") << "\n\n\n\n"
+    @input.rewind
+    value = @editor.data_type_editor('directories', :ARRAY_OF_PATHSPECS)
+    value.should == pathspecs
+  end
+
+  it "should accept empty array of pathspecs" do
+    pathspecs = []
+    @input << pathspecs.join("\n") << "\n\n\n"
+    @input.rewind
+    value = @editor.data_type_editor('directories', :ARRAY_OF_PATHSPECS)
+    value.should == pathspecs
+  end
+
+  it "should reject empty directories" do
+    AppConfig.config['directories'] = []
+    AppConfig.validate['directories'].call([]).should be_false
+  end
+
+  it 'should indicate valid AppConfig' do
+    cwd = File.expand_path(File.dirname(__FILE__))
+    AppConfig.default
+    AppConfig.config.directories = [cwd, File.join(cwd, '..')]
+    AppConfig.config.collection_filespec = __FILE__
+    AppConfig.config.images_dir = cwd
+    AppConfig.valid?.should be_true
+  end
+
+  it 'should indicate invalid AppConfig with no directories' do
+    cwd = File.expand_path(File.dirname(__FILE__))
+    AppConfig.default
+    AppConfig.config.directories = []
+    AppConfig.config.collection_filespec = __FILE__
+    AppConfig.config.images_dir = cwd
+    AppConfig.valid?.should be_false
+  end
+
+  it 'should indicate invalid AppConfig with bad directory path' do
+    cwd = File.expand_path(File.dirname(__FILE__))
+    AppConfig.default
+    AppConfig.config.directories = [cwd, cwd + 'jaberwooky']
+    AppConfig.config.collection_filespec = __FILE__
+    AppConfig.config.images_dir = cwd
+    AppConfig.valid?.should be_false
+  end
+
+  it 'should indicate invalid AppConfig with empty collection.xml path' do
+    cwd = File.expand_path(File.dirname(__FILE__))
+    AppConfig.default
+    AppConfig.config.directories = [cwd, File.join(cwd, '..')]
+    AppConfig.config.collection_filespec = ''
+    AppConfig.config.images_dir = cwd
+    AppConfig.valid?.should be_false
+  end
+
+  it 'should indicate invalid AppConfig with bad collection.xml path' do
+    cwd = File.expand_path(File.dirname(__FILE__))
+    AppConfig.default
+    AppConfig.config.directories = [cwd, File.join(cwd, '..')]
+    AppConfig.config.collection_filespec = cwd + 'jaberwooky'
+    AppConfig.config.images_dir = cwd
+    AppConfig.valid?.should be_false
+  end
+
+  it 'should indicate invalid AppConfig with empty images path' do
+    cwd = File.expand_path(File.dirname(__FILE__))
+    AppConfig.default
+    AppConfig.config.directories = [cwd, File.join(cwd, '..')]
+    AppConfig.config.collection_filespec = __FILE__
+    AppConfig.config.images_dir = ''
+    AppConfig.valid?.should be_false
+  end
+
+  it 'should indicate invalid AppConfig with bad images path' do
+    cwd = File.expand_path(File.dirname(__FILE__))
+    AppConfig.default
+    AppConfig.config.directories = [cwd, File.join(cwd, '..')]
+    AppConfig.config.collection_filespec = __FILE__
+    AppConfig.config.images_dir = cwd + 'jaberwooky'
+    AppConfig.valid?.should be_false
+  end
+
+  it 'should indicate invalid AppConfig with no media extensions' do
+    cwd = File.expand_path(File.dirname(__FILE__))
+    AppConfig.default
+    AppConfig.config.directories = [cwd, File.join(cwd, '..')]
+    AppConfig.config.collection_filespec = __FILE__
+    AppConfig.config.images_dir = cwd
+    AppConfig.config.media_extensions = []
+    AppConfig.valid?.should be_false
+  end
+
+  it 'should indicate invalid AppConfig with no image extensions' do
+    cwd = File.expand_path(File.dirname(__FILE__))
+    AppConfig.default
+    AppConfig.config.directories = [cwd, File.join(cwd, '..')]
+    AppConfig.config.collection_filespec = __FILE__
+    AppConfig.config.images_dir = cwd
+    AppConfig.config.image_extensions = []
+    AppConfig.valid?.should be_false
+  end
+
+  it 'should indicate invalid AppConfig with invalid file permissions' do
+    cwd = File.expand_path(File.dirname(__FILE__))
+    AppConfig.default
+    AppConfig.config.directories = [cwd, File.join(cwd, '..')]
+    AppConfig.config.collection_filespec = __FILE__
+    AppConfig.config.images_dir = cwd
+    AppConfig.config.file_permissions = '-2'
+    AppConfig.valid?.should be_false
+  end
+
+  it 'should indicate invalid AppConfig with invalid directory permissions' do
+    cwd = File.expand_path(File.dirname(__FILE__))
+    AppConfig.default
+    AppConfig.config.directories = [cwd, File.join(cwd, '..')]
+    AppConfig.config.collection_filespec = __FILE__
+    AppConfig.config.images_dir = cwd
+    AppConfig.config.dir_permissions = '11111'
+    AppConfig.valid?.should be_false
+  end
+
+end

data/spec/dvdprofiler_profile_spec.rb

@@ -33,7 +33,7 @@ describe "DvdprofilerProfile" do
   end
 
   it "should find by title" do
-    profile = DvdprofilerProfile.first(:titles => ['National Treasure: Book of Secrets'])
+    profile = DvdprofilerProfile.first(:title => 'National Treasure 2: Book of Secrets')
     profile.should_not == nil
   end
 
@@ -48,4 +48,19 @@ describe "DvdprofilerProfile" do
     hash.should_not be_nil
   end
 
+  it "should find multiple movies with the same title" do
+    profiles = DvdprofilerProfile.all(:title => 'Sabrina')
+    profiles.length.should == 2
+  end
+
+  it "should find a single movie using the year when multiple movies have the same title" do
+    profiles = DvdprofilerProfile.all(:title => 'Sabrina', :year => '1995')
+    (profiles.length.should == 1) && (profiles.first.isbn.should == '097363304340')
+  end
+
+  it "should find the other single movie using the year when multiple movies have the same title" do
+    profiles = DvdprofilerProfile.all(:title => 'Sabrina', :year => '1954')
+    (profiles.length.should == 1) && (profiles.first.isbn.should == '097360540246')
+  end
+
 end