[Metadata][TCSACR-339] Supports new file formats - WAV, FLAC and OGG 18/238918/2
authorHaesu Gwon <haesu.gwon@samsung.com>
Mon, 20 Jul 2020 08:37:55 +0000 (17:37 +0900)
committerHaesu Gwon <haesu.gwon@samsung.com>
Tue, 21 Jul 2020 08:01:30 +0000 (17:01 +0900)
Change-Id: I8a47141eed355a251d576c9c05eccd3450a9c80a

tct-suite-vs/Tizen.Metadata.Tests/res/test_editor.flac [new file with mode: 0644]
tct-suite-vs/Tizen.Metadata.Tests/res/test_editor.ogg [new file with mode: 0644]
tct-suite-vs/Tizen.Metadata.Tests/res/test_editor.wav [new file with mode: 0644]
tct-suite-vs/Tizen.Metadata.Tests/testcase/MetadataEditor/TSMetadataEditor.cs [changed mode: 0755->0644]

diff --git a/tct-suite-vs/Tizen.Metadata.Tests/res/test_editor.flac b/tct-suite-vs/Tizen.Metadata.Tests/res/test_editor.flac
new file mode 100644 (file)
index 0000000..8fcc481
Binary files /dev/null and b/tct-suite-vs/Tizen.Metadata.Tests/res/test_editor.flac differ
diff --git a/tct-suite-vs/Tizen.Metadata.Tests/res/test_editor.ogg b/tct-suite-vs/Tizen.Metadata.Tests/res/test_editor.ogg
new file mode 100644 (file)
index 0000000..e0020fc
Binary files /dev/null and b/tct-suite-vs/Tizen.Metadata.Tests/res/test_editor.ogg differ
diff --git a/tct-suite-vs/Tizen.Metadata.Tests/res/test_editor.wav b/tct-suite-vs/Tizen.Metadata.Tests/res/test_editor.wav
new file mode 100644 (file)
index 0000000..baab8a1
Binary files /dev/null and b/tct-suite-vs/Tizen.Metadata.Tests/res/test_editor.wav differ
old mode 100755 (executable)
new mode 100644 (file)
index 2844799..999c285
@@ -8,24 +8,30 @@ namespace Tizen.Multimedia.Tests
     [Description("Testing Tizen.Multimedia.MetadataEditor class")]
     public class MetadataEditorTests
     {
-        private const string _mediaPath = "/opt/usr/home/owner/share/res/test_editor.mp3";
-        private const string _artworkPath = "/opt/usr/home/owner/share/res/test_artwork.jpg";
-
-        private MetadataEditor _metadata;
+        private readonly string[] _fileFormat = { ".flac", ".mp3", ".ogg", ".wav" };
+        private readonly string _originPath = "/opt/usr/home/owner/share/res/test_editor";
+        private readonly string _filePath = "/opt/usr/home/owner/share/res/copied";
+        private readonly string _artworkPath = "/opt/usr/home/owner/share/res/test_artwork.jpg";
 
         [SetUp]
         public void SetUp()
         {
-            var copied = "/opt/usr/home/owner/share/res/copied.mp3";
-            File.Copy(_mediaPath, copied, true);
-
-            _metadata = new MetadataEditor(copied);
+            foreach(var format in _fileFormat)
+            {
+                File.Copy(_originPath + format, _filePath + format, true);
+            }
         }
 
         [TearDown]
         public void TearDown()
         {
-            _metadata?.Dispose();
+            foreach (var format in _fileFormat)
+            {
+                if (File.Exists(_filePath + format))
+                {
+                    File.Delete(_filePath + format);
+                }
+            }
         }
 
         [Test]
@@ -35,9 +41,13 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "CONSTR")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void MetadataEditor_CHECK_CREATE_CONSTRUCTOR_WITH_PATH()
         {
-            Assert.That(() => new MetadataEditor(_mediaPath), Throws.Nothing);
+            foreach(var format in _fileFormat)
+            {
+                Assert.That(() => new MetadataEditor(_filePath + format), Throws.Nothing);
+            }
         }
 
         [Test]
@@ -47,10 +57,14 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "CONSTN")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void MetadataEditor_CHECK_CREATE_CONSTRUCTOR_WITH_NONEXISTENT_PATH()
         {
-            Assert.That(() => new MetadataEditor("/opt/usr/home/owner/share/res/no_exists.mp4"),
+            foreach (var format in _fileFormat)
+            {
+                Assert.That(() => new MetadataEditor("/opt/usr/home/owner/share/res/no_exists" + format),
                 Throws.TypeOf<FileNotFoundException>());
+            }
         }
 
         [Test]
@@ -84,11 +98,17 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "PRW")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void Artist_PROPERTY_READ_WRITE()
         {
-            _metadata.Artist = "Artist";
-
-            Assert.That(_metadata.Artist, Is.EqualTo("Artist"));
+            foreach (var format in _fileFormat)
+            {
+                using (var metadata = new MetadataEditor(_filePath + format))
+                {
+                    metadata.Artist = "Artist";
+                    Assert.That(metadata.Artist, Is.EqualTo("Artist"));
+                }
+            }
         }
 
         [Test]
@@ -98,11 +118,17 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "PRW")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void Title_PROPERTY_READ_WRITE()
         {
-            _metadata.Title = "Title";
-
-            Assert.That(_metadata.Title, Is.EqualTo("Title"));
+            foreach (var format in _fileFormat)
+            {
+                using (var metadata = new MetadataEditor(_filePath + format))
+                {
+                    metadata.Title = "Title";
+                    Assert.That(metadata.Title, Is.EqualTo("Title"));
+                }
+            }
         }
 
         [Test]
@@ -112,11 +138,17 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "PRW")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void Album_PROPERTY_READ_WRITE()
         {
-            _metadata.Album = "Album";
-
-            Assert.That(_metadata.Album, Is.EqualTo("Album"));
+            foreach (var format in _fileFormat)
+            {
+                using (var metadata = new MetadataEditor(_filePath + format))
+                {
+                    metadata.Album = "Album";
+                    Assert.That(metadata.Album, Is.EqualTo("Album"));
+                }
+            }
         }
 
         [Test]
@@ -126,11 +158,17 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "PRW")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void Genre_PROPERTY_READ_WRITE()
         {
-            _metadata.Genre = "Genre";
-
-            Assert.That(_metadata.Genre, Is.EqualTo("Genre"));
+            foreach (var format in _fileFormat)
+            {
+                using (var metadata = new MetadataEditor(_filePath + format))
+                {
+                    metadata.Genre = "Genre";
+                    Assert.That(metadata.Genre, Is.EqualTo("Genre"));
+                }
+            }
         }
 
         [Test]
@@ -140,11 +178,17 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "PRW")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void Author_PROPERTY_READ_WRITE()
         {
-            _metadata.Author = "Author";
-
-            Assert.That(_metadata.Author, Is.EqualTo("Author"));
+            foreach (var format in _fileFormat)
+            {
+                using (var metadata = new MetadataEditor(_filePath + format))
+                {
+                    metadata.Author = "Author";
+                    Assert.That(metadata.Author, Is.EqualTo("Author"));
+                }
+            }
         }
 
         [Test]
@@ -154,11 +198,17 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "PRW")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void Copyright_PROPERTY_READ_WRITE()
         {
-            _metadata.Copyright = "Copyright";
-
-            Assert.That(_metadata.Copyright, Is.EqualTo("Copyright"));
+            foreach (var format in _fileFormat)
+            {
+                using (var metadata = new MetadataEditor(_filePath + format))
+                {
+                    metadata.Copyright = "Copyright";
+                    Assert.That(metadata.Copyright, Is.EqualTo("Copyright"));
+                }
+            }
         }
 
         [Test]
@@ -168,11 +218,17 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "PRW")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void Date_PROPERTY_READ_WRITE()
         {
-            _metadata.Date = "2000";
-
-            Assert.That(_metadata.Date, Is.EqualTo("2000"));
+            foreach (var format in _fileFormat)
+            {
+                using (var metadata = new MetadataEditor(_filePath + format))
+                {
+                    metadata.Date = "2000";
+                    Assert.That(metadata.Date, Is.EqualTo("2000"));
+                }
+            }
         }
 
         [Test]
@@ -182,11 +238,17 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "PRW")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void Description_PROPERTY_READ_WRITE()
         {
-            _metadata.Description = "Description";
-
-            Assert.That(_metadata.Description, Is.EqualTo("Description"));
+            foreach (var format in _fileFormat)
+            {
+                using (var metadata = new MetadataEditor(_filePath + format))
+                {
+                    metadata.Description = "Description";
+                    Assert.That(metadata.Description, Is.EqualTo("Description"));
+                }
+            }
         }
 
         [Test]
@@ -196,11 +258,17 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "PRW")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void Comment_PROPERTY_READ_WRITE()
         {
-            _metadata.Comment = "Comment";
-
-            Assert.That(_metadata.Comment, Is.EqualTo("Comment"));
+            foreach (var format in _fileFormat)
+            {
+                using (var metadata = new MetadataEditor(_filePath + format))
+                {
+                    metadata.Comment = "Comment";
+                    Assert.That(metadata.Comment, Is.EqualTo("Comment"));
+                }
+            }
         }
 
         [Test]
@@ -210,11 +278,17 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "PRW")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void TrackNumber_PROPERTY_READ_WRITE()
         {
-            _metadata.TrackNumber = "5";
-
-            Assert.That(_metadata.TrackNumber, Is.EqualTo("5"));
+            foreach (var format in _fileFormat)
+            {
+                using (var metadata = new MetadataEditor(_filePath + format))
+                {
+                    metadata.TrackNumber = "5";
+                    Assert.That(metadata.TrackNumber, Is.EqualTo("5"));
+                }
+            }
         }
 
         [Test]
@@ -224,9 +298,17 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "PRO")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void PictureCount_PROPERTY_READ_ONLY()
         {
-            Assert.That(_metadata.PictureCount, Is.EqualTo(1));
+            foreach (var format in _fileFormat)
+            {
+                using (var metadata = new MetadataEditor(_filePath + format))
+                {
+                    Log.Info("Tizen.Multimedia.Metadata", _filePath + format);
+                    Assert.That(metadata.PictureCount, Is.GreaterThanOrEqualTo(1));
+                }
+            }
         }
 
         [Test]
@@ -236,11 +318,17 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "PRW")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void Conductor_PROPERTY_READ_WRITE()
         {
-            _metadata.Conductor = "Conductor";
-
-            Assert.That(_metadata.Conductor, Is.EqualTo("Conductor"));
+            foreach (var format in _fileFormat)
+            {
+                using (var metadata = new MetadataEditor(_filePath + format))
+                {
+                    metadata.Conductor = "Conductor";
+                    Assert.That(metadata.Conductor, Is.EqualTo("Conductor"));
+                }
+            }
         }
 
         [Test]
@@ -250,11 +338,17 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "PRW")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void UnsyncLyrics_PROPERTY_READ_WRITE()
         {
-            _metadata.UnsyncLyrics = "UnsyncLyrics";
-
-            Assert.That(_metadata.UnsyncLyrics, Is.EqualTo("UnsyncLyrics"));
+            foreach (var format in _fileFormat)
+            {
+                using (var metadata = new MetadataEditor(_filePath + format))
+                {
+                    metadata.UnsyncLyrics = "UnsyncLyrics";
+                    Assert.That(metadata.UnsyncLyrics, Is.EqualTo("UnsyncLyrics"));
+                }
+            }
         }
 
         [Test]
@@ -264,12 +358,19 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "MR")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void GetPicture_CHECK_RESULT()
         {
-            Artwork artwork = _metadata.GetPicture(0);
-
-            Assert.That(artwork, Is.Not.Null);
-            Assert.That(artwork.Data.Length, Is.GreaterThan(0));
+            foreach (var format in _fileFormat)
+            {
+                using (var metadata = new MetadataEditor(_filePath + format))
+                {
+                    Artwork artwork = metadata.GetPicture(0);
+
+                    Assert.That(artwork, Is.Not.Null);
+                    Assert.That(artwork.Data.Length, Is.GreaterThan(0));
+                }
+            }
         }
 
         [Test]
@@ -279,9 +380,16 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "MEX")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void GetPicture_CHECK_RESULT_WITH_WRONG_INDEX()
         {
-            Assert.That(() => _metadata.GetPicture(-1), Throws.TypeOf<ArgumentOutOfRangeException>());
+            foreach (var format in _fileFormat)
+            {
+                using (var metadata = new MetadataEditor(_filePath + format))
+                {
+                    Assert.That(() => metadata.GetPicture(-1), Throws.TypeOf<ArgumentOutOfRangeException>());
+                }
+            }
         }
 
         [Test]
@@ -291,14 +399,23 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "MCST")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void AddPicture_CHECK_RESULT()
         {
-            int oldValue = _metadata.PictureCount;
+            foreach (var format in _fileFormat)
+            {
+                using (var metadata = new MetadataEditor(_filePath + format))
+                {
+                    int oldValue = metadata.PictureCount;
 
-            _metadata.AddPicture(_artworkPath);
-            _metadata.Commit();
+                    metadata.AddPicture(_artworkPath);
+                    metadata.Commit();
 
-            Assert.That(_metadata.PictureCount, Is.EqualTo(oldValue + 1));
+                    Assert.That(metadata.PictureCount, Is.EqualTo(oldValue + 1));
+
+                    //File.Copy(filePath + format, "/tmp/test_editor"+format);
+                }
+            }
         }
 
         [Test]
@@ -308,9 +425,16 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "MEX")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void AddPicture_CHECK_RESULT_WITH_NULL_PARAM()
         {
-            Assert.That(() => _metadata.AddPicture(null), Throws.ArgumentNullException);
+            foreach (var format in _fileFormat)
+            {
+                using (var metadata = new MetadataEditor(_filePath + format))
+                {
+                    Assert.That(() => metadata.AddPicture(null), Throws.ArgumentNullException);
+                }
+            }
         }
 
         [Test]
@@ -320,14 +444,21 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "MCST")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void RemovePicture_CHECK_RESULT()
         {
-            int oldValue = _metadata.PictureCount;
-
-            _metadata.RemovePicture(0);
-            _metadata.Commit();
-
-            Assert.That(_metadata.PictureCount, Is.EqualTo(oldValue - 1));
+            foreach(var format in _fileFormat)
+            {
+                using (var metadata = new MetadataEditor(_filePath + format))
+                {
+                    int oldValue = metadata.PictureCount;
+
+                    metadata.RemovePicture(0);
+                    metadata.Commit();
+
+                    Assert.That(metadata.PictureCount, Is.EqualTo(oldValue - 1));
+                }
+            }
         }
 
         [Test]
@@ -337,9 +468,16 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "MEX")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void RemovePicture_CHECK_RESULT_WITH_WRONG_INDEX()
         {
-            Assert.That(() => _metadata.RemovePicture(-1), Throws.TypeOf<ArgumentOutOfRangeException>());
+            foreach (var format in _fileFormat)
+            {
+                using (var metadata = new MetadataEditor(_filePath + format))
+                {
+                    Assert.That(() => metadata.RemovePicture(-1), Throws.TypeOf<ArgumentOutOfRangeException>());
+                }
+            }
         }
 
         [Test]
@@ -349,12 +487,19 @@ namespace Tizen.Multimedia.Tests
         [Property("SPEC_URL", "-")]
         [Property("CRITERIA", "MCST")]
         [Property("AUTHOR", "Minje Ahn, minje.ahn@samsung.com")]
+        [Property("AUTHOR", "Haesu Gwon, haesu.gwon@samsung.com")]
         public void Commit_CHECK_RESULT()
         {
-            _metadata.Artist = "Artist";
-            _metadata.Commit();
-
-            Assert.That(_metadata.Artist, Is.EqualTo("Artist"));
+            foreach (var format in _fileFormat)
+            {
+                using (var metadata = new MetadataEditor(_filePath + format))
+                {
+                    metadata.Artist = "Artist";
+                    metadata.Commit();
+
+                    Assert.That(metadata.Artist, Is.EqualTo("Artist"));
+                }
+            }
         }
     }
 }