Harmonize Animation API parameter checking and add test cases for them

[platform/core/uifw/dali-core.git] / dali / internal / event / animation / key-frames-impl.cpp

diff --git a/dali/internal/event/animation/key-frames-impl.cpp b/dali/internal/event/animation/key-frames-impl.cpp
index 969d8f3..ac1bd93 100644 (file)
--- a/dali/internal/event/animation/key-frames-impl.cpp
+++ b/dali/internal/event/animation/key-frames-impl.cpp
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2014 Samsung Electronics Co., Ltd.
+ * Copyright (c) 2018 Samsung Electronics Co., Ltd.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -46,26 +46,45 @@ void KeyFrames::CreateKeyFramesSpec(Property::Type type)
   switch(type)
   {
     case Property::BOOLEAN:
+    {
       mKeyFrames = Internal::KeyFrameBoolean::New();
       break;
+    }
+    case Property::INTEGER:
+    {
+      mKeyFrames = Internal::KeyFrameInteger::New();
+      break;
+    }
     case Property::FLOAT:
+    {
       mKeyFrames = Internal::KeyFrameNumber::New();
       break;
+    }
     case Property::VECTOR2:
+    {
       mKeyFrames = Internal::KeyFrameVector2::New();
       break;
+    }
     case Property::VECTOR3:
+    {
       mKeyFrames = Internal::KeyFrameVector3::New();
       break;
+    }
     case Property::VECTOR4:
+    {
       mKeyFrames = Internal::KeyFrameVector4::New();
       break;
+    }
     case Property::ROTATION:
+    {
       mKeyFrames = Internal::KeyFrameQuaternion::New();
       break;
+    }
     default:
-      DALI_ASSERT_DEBUG(!"Type not supported");
+    {
+      DALI_ABORT( "Property type is not animatable" );
       break;
+    }
   }
 }
 
@@ -94,6 +113,12 @@ void KeyFrames::Add(float time, Property::Value value, AlphaFunction alpha)
       kf->AddKeyFrame(time, value.Get<bool>(), alpha);
       break;
     }
+    case Property::INTEGER:
+    {
+      Internal::KeyFrameInteger* kf = static_cast<Internal::KeyFrameInteger*>(mKeyFrames.Get());
+      kf->AddKeyFrame(time, value.Get<int>(), alpha);
+      break;
+    }
     case Property::FLOAT:
     {
       Internal::KeyFrameNumber* kf = static_cast<Internal::KeyFrameNumber*>(mKeyFrames.Get());
@@ -135,7 +160,18 @@ KeyFrameSpec* KeyFrames::GetKeyFramesBase() const
   return mKeyFrames.Get();
 }
 
-} // Internal
-} // Dali
+Property::Value KeyFrames::GetLastKeyFrameValue() const
+{
+  Property::Value value;
 
+  std::size_t noOfKeyFrames = mKeyFrames->GetNumberOfKeyFrames();
+  if( noOfKeyFrames )
+  {
+    mKeyFrames->GetKeyFrameAsValue( noOfKeyFrames - 1, value );
+  }
 
+  return value;
+}
+
+} // Internal
+} // Dali