[NUI][Non-ACR] Call VideoView Dispose 03/206603/8
authorhuiyu.eun <huiyu.eun@samsung.com>
Wed, 22 May 2019 04:47:43 +0000 (13:47 +0900)
committerhuiyu.eun <huiyu.eun@samsung.com>
Thu, 23 May 2019 06:32:07 +0000 (15:32 +0900)
Change-Id: I4e2664cced1b9344b00e4cf16d72baed9b5d14fa
Signed-off-by: huiyu.eun <huiyu.eun@samsung.com>
tct-suite-vs/Tizen.NUI.Tests/testcase/TSVideoView.FinishedEventArgs.cs
tct-suite-vs/Tizen.NUI.Tests/testcase/TSVideoView.cs

index dc66f8d..098b0b1 100755 (executable)
@@ -56,7 +56,8 @@ namespace Tizen.NUI.Tests
             VideoView.FinishedEventArgs finishedEventArgs = new VideoView.FinishedEventArgs();
             VideoView videoView = new VideoView();
             finishedEventArgs.VideoView = videoView;
-            Assert.AreEqual(videoView, finishedEventArgs.VideoView, "Retrieved VideoView should be equal to set value");
+            Assert.AreEqual(videoView, finishedEventArgs.VideoView, "Retrieved VideoView should be equal to set value");\r
+            videoView.Dispose();
         }
     }
 }
index 4fefa96..ff41222 100755 (executable)
@@ -16,8 +16,9 @@ namespace Tizen.NUI.Tests
     public class VideoViewTests
     {
         private string TAG = "NUI";
-        private string video_path = Tizen.Applications.Application.Current.DirectoryInfo.Resource + "demoVideo.mp4";
-        private bool _flagOnFinished;
+        private string _videoPath = Tizen.Applications.Application.Current.DirectoryInfo.Resource + "demoVideo.mp4";
+        private bool _flagOnFinished;\r
+        private VideoView _videoView;
 
         private void OnFinished(object obj, VideoView.FinishedEventArgs args)
         {
@@ -30,12 +31,15 @@ namespace Tizen.NUI.Tests
             Tizen.Log.Info(TAG, "Init() is called!");
             App.MainTitleChangeText("VideoViewTests");
             App.MainTitleChangeBackgroundColor(null);
+            _videoView = new VideoView();
         }
 
         [TearDown]
         public void Destroy()
         {
             Tizen.Log.Info(TAG, "Destroy() is called!");
+            _videoView.Dispose();
+            _videoView = null;
         }\r
 
         [Test]
@@ -47,10 +51,10 @@ namespace Tizen.NUI.Tests
         [Property("AUTHOR", "Zaijuan Sui, z6177.sui@samsung.com")]
         [Property("COVPARAM", "")]
         public void VideoView_INIT_WITH_NULL()
-        {
-            /* TEST CODE */
-            var videoview = new VideoView();
-            Assert.IsInstanceOf<VideoView>(videoview, "Should return VideoView instance.");
+        {\r
+            /* TEST CODE */\r
+            Assert.IsNotNull(_videoView, "_videoView is null.");
+            Assert.IsInstanceOf<VideoView>(_videoView, "Should return VideoView instance.");
         }
 
         [Test]
@@ -64,8 +68,10 @@ namespace Tizen.NUI.Tests
         public void VideoView_INIT_WITH_URL()
         {
             /* TEST CODE */
-            var videoview = new VideoView("DALI");
-            Assert.IsInstanceOf<VideoView>(videoview, "Should return VideoView instance.");
+            var videoView = new VideoView("DALI");\r
+            Assert.IsNotNull(videoView, "videoView is null.");
+            Assert.IsInstanceOf<VideoView>(videoView, "Should return VideoView instance.");
+            videoView.Dispose();
         }
 
         [Test]
@@ -78,9 +84,8 @@ namespace Tizen.NUI.Tests
         public void ResourceUrl_SET_GET_VALUE()
         {
             /* TEST CODE */
-            var videoview = new VideoView();
-            videoview.ResourceUrl = video_path;
-            Assert.AreEqual(video_path, videoview.ResourceUrl, "videoview.ResourceUrl should equal to the setted value.");
+            _videoView.ResourceUrl = _videoPath;
+            Assert.AreEqual(_videoPath, _videoView.ResourceUrl, "videoview.ResourceUrl should equal to the setted value.");
         }
 
         [Test]
@@ -95,8 +100,7 @@ namespace Tizen.NUI.Tests
             /* TEST CODE */
             try
             {
-                VideoView videoview = new VideoView();
-                videoview.Play();
+                _videoView.Play();
             }
             catch (Exception e)
             {
@@ -118,8 +122,7 @@ namespace Tizen.NUI.Tests
             /* TEST CODE */
             try
             {
-                VideoView videoview = new VideoView();
-                videoview.Pause();
+                _videoView.Pause();
             }
             catch (Exception e)
             {
@@ -141,8 +144,7 @@ namespace Tizen.NUI.Tests
             /* TEST CODE */
             try
             {
-                VideoView videoview = new VideoView();
-                videoview.Stop();
+                _videoView.Stop();
             }
             catch (Exception e)
             {
@@ -164,8 +166,7 @@ namespace Tizen.NUI.Tests
             /* TEST CODE */
             try
             {
-                VideoView videoview = new VideoView();
-                videoview.Forward(1000);
+                _videoView.Forward(1000);
             }
             catch (Exception e)
             {
@@ -187,8 +188,7 @@ namespace Tizen.NUI.Tests
             /* TEST CODE */
             try
             {
-                VideoView videoview = new VideoView();
-                videoview.Backward(1000);
+                _videoView.Backward(1000);
             }
             catch (Exception e)
             {
@@ -211,9 +211,8 @@ namespace Tizen.NUI.Tests
             PropertyMap propertyMap = new PropertyMap();
             propertyMap.Insert(Tizen.NUI.Visual.Property.Type, new PropertyValue((int)Tizen.NUI.Visual.Type.Text));
             propertyMap.Insert(Tizen.NUI.TextVisualProperty.Text, new PropertyValue("Hello Goodbye"));
-            VideoView videoview = new VideoView();
-            videoview.Video = propertyMap;
-            PropertyValue propertyvalue3 = videoview.Video.GetValue(1);
+            _videoView.Video = propertyMap;
+            PropertyValue propertyvalue3 = _videoView.Video.GetValue(1);
             string tempvalue = "";
             propertyvalue3.Get(out tempvalue);
             Assert.AreEqual("Hello Goodbye", tempvalue, "Video function does not work");
@@ -229,14 +228,13 @@ namespace Tizen.NUI.Tests
         public void Looping_SET_GET_VALUE()
         {
             /* TEST CODE */
-            VideoView videoview = new VideoView();
-            Window.Instance.GetDefaultLayer().Add(videoview);
+            Window.Instance.GetDefaultLayer().Add(_videoView);
 
-            videoview.Looping = true;
-            Assert.AreEqual(true, videoview.Looping, "Looping function does not work");
+            _videoView.Looping = true;
+            Assert.AreEqual(true, _videoView.Looping, "Looping function does not work");
 
-            videoview.Looping = false;
-            Assert.AreEqual(false, videoview.Looping, "Looping function does not work");
+            _videoView.Looping = false;
+            Assert.AreEqual(false, _videoView.Looping, "Looping function does not work");
         }
 
         [Test]
@@ -249,26 +247,25 @@ namespace Tizen.NUI.Tests
         public void Underlay_SET_GET_VALUE()
         {
             /* TEST CODE */
-            VideoView videoview = new VideoView();
-            Window.Instance.GetDefaultLayer().Add(videoview);
+            Window.Instance.GetDefaultLayer().Add(_videoView);
 
             bool isSupportRawVideo = false;
             if (Tizen.System.Information.TryGetValue<bool>("tizen.org/feature/multimedia.raw_video", out isSupportRawVideo))
             {
                 if (isSupportRawVideo)
                 {
-                    videoview.Underlay = false;
-                    Assert.AreEqual(false, videoview.Underlay, "Underlay function does not work");
+                    _videoView.Underlay = false;
+                    Assert.AreEqual(false, _videoView.Underlay, "Underlay function does not work");
                 }
                 else
                 {
-                    videoview.Underlay = true;
-                    Assert.AreEqual(true, videoview.Underlay, "Underlay function does not work");
+                    _videoView.Underlay = true;
+                    Assert.AreEqual(true, _videoView.Underlay, "Underlay function does not work");
                 }
             }
            else
            {
-               Tizen.Log.Error(TAG, "Error checking if raw_video is supported(systeminfo)");
+               Tizen.Log.Error(TAG, "Error checking if raw_video is supported(systeminfo)");\r
            }
         }
 
@@ -282,14 +279,13 @@ namespace Tizen.NUI.Tests
         public void Muted_SET_GET_VALUE()
         {
             /* TEST CODE */
-            VideoView videoview = new VideoView();
-            Window.Instance.GetDefaultLayer().Add(videoview);
+            Window.Instance.GetDefaultLayer().Add(_videoView);
 
-            videoview.Muted = true;
-            Assert.AreEqual(true, videoview.Muted, "Muted function does not work");
+            _videoView.Muted = true;
+            Assert.AreEqual(true, _videoView.Muted, "Muted function does not work");
 
-            videoview.Muted = false;
-            Assert.AreEqual(false, videoview.Muted, "Muted function does not work");
+            _videoView.Muted = false;
+            Assert.AreEqual(false, _videoView.Muted, "Muted function does not work");
         }
 
         [Test]
@@ -305,13 +301,12 @@ namespace Tizen.NUI.Tests
             PropertyMap propertyMap = new PropertyMap();
             propertyMap.Add("volumeLeft", new PropertyValue(0.0f));
             propertyMap.Add("volumeRight", new PropertyValue(5.0f));
-            VideoView videoview = new VideoView();
 
-            videoview.Volume = propertyMap;
+            _videoView.Volume = propertyMap;
             float left = 0.0f;
             float right = 0.0f;
-            videoview.Volume.Find(0, "volumeLeft").Get(out left);
-            videoview.Volume.Find(1, "volumeRight").Get(out right);
+            _videoView.Volume.Find(0, "volumeLeft").Get(out left);
+            _videoView.Volume.Find(1, "volumeRight").Get(out right);
         }
 
         [Test]
@@ -346,16 +341,15 @@ namespace Tizen.NUI.Tests
         [Property("AUTHOR", "Xianbing Teng, xb.teng@samsung.com")]
         public async Task Finished_CHECK_EVENT()
         {
-            /* TEST CODE */
-            var videoView = new VideoView();\r
-            videoView.ResourceUrl = video_path;\r
-            Assert.IsInstanceOf<VideoView>(videoView, "Should be an instance of VideoView type.");
+            /* TEST CODE */\r
+            _videoView.ResourceUrl = _videoPath;\r
+            Assert.IsInstanceOf<VideoView>(_videoView, "Should be an instance of VideoView type.");
             try
             {
                 _flagOnFinished = false;\r
                 Assert.False(_flagOnFinished, "_flagOnFinished should false initial");
-                videoView.Finished += OnFinished;\r
-                videoView.Play();
+                _videoView.Finished += OnFinished;\r
+                _videoView.Play();
                 await Task.Delay(1000);
                 Assert.True(_flagOnFinished, "_flagOnFinished should be true after Finished triggered");
             }
@@ -367,7 +361,7 @@ namespace Tizen.NUI.Tests
             }
             finally
             {\r
-                videoView.Finished -= OnFinished;\r
+                _videoView.Finished -= OnFinished;\r
             }
         }
     }