From: huiyu.eun Date: Wed, 22 May 2019 04:47:43 +0000 (+0900) Subject: [NUI][Non-ACR] Call VideoView Dispose X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fchanges%2F03%2F206603%2F8;p=test%2Ftct%2Fcsharp%2Fapi.git [NUI][Non-ACR] Call VideoView Dispose Change-Id: I4e2664cced1b9344b00e4cf16d72baed9b5d14fa Signed-off-by: huiyu.eun --- diff --git a/tct-suite-vs/Tizen.NUI.Tests/testcase/TSVideoView.FinishedEventArgs.cs b/tct-suite-vs/Tizen.NUI.Tests/testcase/TSVideoView.FinishedEventArgs.cs index dc66f8d..098b0b1 100755 --- a/tct-suite-vs/Tizen.NUI.Tests/testcase/TSVideoView.FinishedEventArgs.cs +++ b/tct-suite-vs/Tizen.NUI.Tests/testcase/TSVideoView.FinishedEventArgs.cs @@ -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"); + videoView.Dispose(); } } } diff --git a/tct-suite-vs/Tizen.NUI.Tests/testcase/TSVideoView.cs b/tct-suite-vs/Tizen.NUI.Tests/testcase/TSVideoView.cs index 4fefa96..ff41222 100755 --- a/tct-suite-vs/Tizen.NUI.Tests/testcase/TSVideoView.cs +++ b/tct-suite-vs/Tizen.NUI.Tests/testcase/TSVideoView.cs @@ -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; + 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; } [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, "Should return VideoView instance."); + { + /* TEST CODE */ + Assert.IsNotNull(_videoView, "_videoView is null."); + Assert.IsInstanceOf(_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, "Should return VideoView instance."); + var videoView = new VideoView("DALI"); + Assert.IsNotNull(videoView, "videoView is null."); + Assert.IsInstanceOf(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("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)"); } } @@ -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(); - videoView.ResourceUrl = video_path; - Assert.IsInstanceOf(videoView, "Should be an instance of VideoView type."); + /* TEST CODE */ + _videoView.ResourceUrl = _videoPath; + Assert.IsInstanceOf(_videoView, "Should be an instance of VideoView type."); try { _flagOnFinished = false; Assert.False(_flagOnFinished, "_flagOnFinished should false initial"); - videoView.Finished += OnFinished; - videoView.Play(); + _videoView.Finished += OnFinished; + _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 { - videoView.Finished -= OnFinished; + _videoView.Finished -= OnFinished; } } }