Skip to content

Commit

Permalink
Tests: Shorten some crazy long test names
Browse files Browse the repository at this point in the history
  • Loading branch information
@ferdnyc
ferdnyc committed Apr 19, 2021
1 parent 00caff0 commit 89b1643
Show file tree
Hide file tree
Showing 2 changed files with 32 additions and 32 deletions.
46 changes: 23 additions & 23 deletions tests/KeyFrame.cpp
View file
Expand Up @@ -38,15 +38,15 @@

using namespace openshot;

TEST_CASE( "GetPoint_With_No_Points", "[libopenshot][keyframe]" )
TEST_CASE( "GetPoint (no Points)", "[libopenshot][keyframe]" )
{
// Create an empty keyframe
Keyframe k1;

CHECK_THROWS_AS(k1.GetPoint(0), OutOfBoundsPoint);
}

TEST_CASE( "GetPoint_With_1_Points", "[libopenshot][keyframe]" )
TEST_CASE( "GetPoint (1 Point)", "[libopenshot][keyframe]" )
{
// Create an empty keyframe
Keyframe k1;
Expand All @@ -60,7 +60,7 @@ TEST_CASE( "GetPoint_With_1_Points", "[libopenshot][keyframe]" )
}


TEST_CASE( "AddPoint_With_1_Point", "[libopenshot][keyframe]" )
TEST_CASE( "AddPoint (1 Point)", "[libopenshot][keyframe]" )
{
// Create an empty keyframe
Keyframe k1;
Expand All @@ -71,7 +71,7 @@ TEST_CASE( "AddPoint_With_1_Point", "[libopenshot][keyframe]" )
CHECK_THROWS_AS(k1.GetPoint(1), OutOfBoundsPoint);
}

TEST_CASE( "AddPoint_With_2_Points", "[libopenshot][keyframe]" )
TEST_CASE( "AddPoint (2 Points)", "[libopenshot][keyframe]" )
{
// Create an empty keyframe
Keyframe k1;
Expand All @@ -84,7 +84,7 @@ TEST_CASE( "AddPoint_With_2_Points", "[libopenshot][keyframe]" )
CHECK_THROWS_AS(k1.GetPoint(2), OutOfBoundsPoint);
}

TEST_CASE( "GetValue_For_Bezier_Curve_2_Points", "[libopenshot][keyframe]" )
TEST_CASE( "GetValue (Bezier curve, 2 Points)", "[libopenshot][keyframe]" )
{
// Create a keyframe curve with 2 points
Keyframe kf;
Expand All @@ -103,7 +103,7 @@ TEST_CASE( "GetValue_For_Bezier_Curve_2_Points", "[libopenshot][keyframe]" )
CHECK(kf.GetLength() == 51);
}

TEST_CASE( "GetValue_For_Bezier_Curve_5_Points_40_Percent_Handle", "[libopenshot][keyframe]" )
TEST_CASE( "GetValue (Bezier, 5 Points, 40% handle)", "[libopenshot][keyframe]" )
{
// Create a keyframe curve with 2 points
Keyframe kf;
Expand All @@ -126,7 +126,7 @@ TEST_CASE( "GetValue_For_Bezier_Curve_5_Points_40_Percent_Handle", "[libopenshot
CHECK(kf.GetLength() == 201);
}

TEST_CASE( "GetValue_For_Bezier_Curve_5_Points_25_Percent_Handle", "[libopenshot][keyframe]" )
TEST_CASE( "GetValue (Bezier, 5 Points, 25% Handle)", "[libopenshot][keyframe]" )
{
// Create a keyframe curve with 2 points
Keyframe kf;
Expand All @@ -149,7 +149,7 @@ TEST_CASE( "GetValue_For_Bezier_Curve_5_Points_25_Percent_Handle", "[libopenshot
CHECK(kf.GetLength() == 201);
}

TEST_CASE( "GetValue_For_Linear_Curve_3_Points", "[libopenshot][keyframe]" )
TEST_CASE( "GetValue (Linear, 3 Points)", "[libopenshot][keyframe]" )
{
// Create a keyframe curve with 2 points
Keyframe kf;
Expand All @@ -169,7 +169,7 @@ TEST_CASE( "GetValue_For_Linear_Curve_3_Points", "[libopenshot][keyframe]" )
CHECK(kf.GetLength() == 51);
}

TEST_CASE( "GetValue_For_Constant_Curve_3_Points", "[libopenshot][keyframe]" )
TEST_CASE( "GetValue (Constant, 3 Points)", "[libopenshot][keyframe]" )
{
// Create a keyframe curve with 2 points
Keyframe kf;
Expand All @@ -190,7 +190,7 @@ TEST_CASE( "GetValue_For_Constant_Curve_3_Points", "[libopenshot][keyframe]" )
CHECK(kf.GetLength() == 51);
}

TEST_CASE( "Check_Direction_and_Repeat_Fractions", "[libopenshot][keyframe]" )
TEST_CASE( "GetDelta and GetRepeatFraction", "[libopenshot][keyframe]" )
{
// Create a keyframe curve with 2 points
Keyframe kf;
Expand Down Expand Up @@ -225,7 +225,7 @@ TEST_CASE( "Check_Direction_and_Repeat_Fractions", "[libopenshot][keyframe]" )
}


TEST_CASE( "Get_Closest_Point", "[libopenshot][keyframe]" )
TEST_CASE( "GetClosestPoint", "[libopenshot][keyframe]" )
{
// Create a keyframe curve with 2 points
Keyframe kf;
Expand Down Expand Up @@ -253,7 +253,7 @@ TEST_CASE( "Get_Closest_Point", "[libopenshot][keyframe]" )
}


TEST_CASE( "Get_Previous_Point", "[libopenshot][keyframe]" )
TEST_CASE( "GetPreviousPoint", "[libopenshot][keyframe]" )
{
// Create a keyframe curve with 2 points
Keyframe kf;
Expand All @@ -272,7 +272,7 @@ TEST_CASE( "Get_Previous_Point", "[libopenshot][keyframe]" )

}

TEST_CASE( "Get_Max_Point", "[libopenshot][keyframe]" )
TEST_CASE( "GetMaxPoint", "[libopenshot][keyframe]" )
{
// Create a keyframe curve
Keyframe kf;
Expand All @@ -297,7 +297,7 @@ TEST_CASE( "Get_Max_Point", "[libopenshot][keyframe]" )
CHECK(kf.GetMaxPoint().co.Y == 2.0);
}

TEST_CASE( "Scale_Keyframe", "[libopenshot][keyframe]" )
TEST_CASE( "Keyframe scaling", "[libopenshot][keyframe]" )
{
// Create a keyframe curve with 2 points
Keyframe kf;
Expand Down Expand Up @@ -339,7 +339,7 @@ TEST_CASE( "Scale_Keyframe", "[libopenshot][keyframe]" )

}

TEST_CASE( "Flip_Keyframe", "[libopenshot][keyframe]" )
TEST_CASE( "flip Keyframe", "[libopenshot][keyframe]" )
{
// Create a keyframe curve with 2 points
Keyframe kf;
Expand Down Expand Up @@ -373,7 +373,7 @@ TEST_CASE( "Flip_Keyframe", "[libopenshot][keyframe]" )
CHECK(kf.GetValue(100) == Approx(10.0f).margin(0.01));
}

TEST_CASE( "Remove_Duplicate_Point", "[libopenshot][keyframe]" )
TEST_CASE( "remove duplicate Point", "[libopenshot][keyframe]" )
{
// Create a keyframe curve with 2 points
Keyframe kf;
Expand All @@ -386,7 +386,7 @@ TEST_CASE( "Remove_Duplicate_Point", "[libopenshot][keyframe]" )
CHECK(kf.GetPoint(0).co.Y == Approx(2.0).margin(0.01));
}

TEST_CASE( "Large_Number_Values", "[libopenshot][keyframe]" )
TEST_CASE( "large number values", "[libopenshot][keyframe]" )
{
// Large value
int64_t const large_value = 30 * 60 * 90;
Expand All @@ -402,7 +402,7 @@ TEST_CASE( "Large_Number_Values", "[libopenshot][keyframe]" )
CHECK(kf.GetPoint(1).co.Y == Approx(100.0).margin(0.01));
}

TEST_CASE( "Remove_Point", "[libopenshot][keyframe]" )
TEST_CASE( "remove Point", "[libopenshot][keyframe]" )
{
Keyframe kf;
kf.AddPoint(openshot::Point(Coordinate(1, 1), CONSTANT));
Expand All @@ -415,7 +415,7 @@ TEST_CASE( "Remove_Point", "[libopenshot][keyframe]" )
CHECK_THROWS_AS(kf.RemovePoint(100), OutOfBoundsPoint);
}

TEST_CASE( "Constant_Interpolation_First_Segment", "[libopenshot][keyframe]" )
TEST_CASE( "Constant interp, first segment", "[libopenshot][keyframe]" )
{
Keyframe kf;
kf.AddPoint(Point(Coordinate(1, 1), CONSTANT));
Expand All @@ -428,7 +428,7 @@ TEST_CASE( "Constant_Interpolation_First_Segment", "[libopenshot][keyframe]" )
CHECK(kf.GetInt(4) == 100);
}

TEST_CASE( "isIncreasing", "[libopenshot][keyframe]" )
TEST_CASE( "IsIncreasing", "[libopenshot][keyframe]" )
{
// Which cases need to be tested to keep same behaviour as
// previously?
Expand Down Expand Up @@ -477,15 +477,15 @@ TEST_CASE( "GetLength", "[libopenshot][keyframe]" )
CHECK(g.GetLength() == 201);
}

TEST_CASE( "Use_Interpolation_of_Segment_End_Point", "[libopenshot][keyframe]" )
TEST_CASE( "use segment end point interpolation", "[libopenshot][keyframe]" )
{
Keyframe f;
f.AddPoint(1,0, CONSTANT);
f.AddPoint(100,155, BEZIER);
CHECK(f.GetValue(50) == Approx(75.9).margin(0.1));
}

TEST_CASE( "Handle_Large_Segment", "[libopenshot][keyframe]" )
TEST_CASE( "handle large segment", "[libopenshot][keyframe]" )
{
Keyframe kf;
kf.AddPoint(1, 0, CONSTANT);
Expand All @@ -498,7 +498,7 @@ TEST_CASE( "Handle_Large_Segment", "[libopenshot][keyframe]" )
CHECK((double)fr.num / fr.den == Approx(0.5).margin(0.01));
}

TEST_CASE( "Point_Vector_Constructor", "[libopenshot][keyframe]" )
TEST_CASE( "std::vector<Point> constructor", "[libopenshot][keyframe]" )
{
std::vector<Point> points{Point(1, 10), Point(5, 20), Point(10, 30)};
Keyframe k1(points);
Expand Down
18 changes: 9 additions & 9 deletions tests/Point.cpp
View file
Expand Up @@ -36,7 +36,7 @@
#include "Coordinate.h"
#include "Json.h"

TEST_CASE( "Default_Constructor", "[libopenshot][point]" )
TEST_CASE( "default constructor", "[libopenshot][point]" )
{
openshot::Point p;

Expand All @@ -50,7 +50,7 @@ TEST_CASE( "Default_Constructor", "[libopenshot][point]" )
CHECK(p.interpolation == openshot::InterpolationType::BEZIER);
CHECK(p.handle_type == openshot::HandleType::AUTO);
}
TEST_CASE( "XY_Constructor", "[libopenshot][point]" )
TEST_CASE( "XY constructor", "[libopenshot][point]" )
{
// Create a point with X and Y values
openshot::Point p1(2,9);
Expand All @@ -60,7 +60,7 @@ TEST_CASE( "XY_Constructor", "[libopenshot][point]" )
CHECK(p1.interpolation == openshot::InterpolationType::BEZIER);
}

TEST_CASE( "Pair_Constructor", "[libopenshot][point]" )
TEST_CASE( "std::pair constructor", "[libopenshot][point]" )
{
// Create a point from a std::pair
std::pair<double, double> coordinates(22, 5);
Expand All @@ -70,7 +70,7 @@ TEST_CASE( "Pair_Constructor", "[libopenshot][point]" )
CHECK(p1.co.Y == Approx(5.0f).margin(0.00001));
}

TEST_CASE( "Constructor_With_Coordinate", "[libopenshot][point]" )
TEST_CASE( "constructor from Coordinate", "[libopenshot][point]" )
{
// Create a point with a coordinate
openshot::Coordinate c1(3,7);
Expand All @@ -81,7 +81,7 @@ TEST_CASE( "Constructor_With_Coordinate", "[libopenshot][point]" )
CHECK(p1.interpolation == openshot::InterpolationType::BEZIER);
}

TEST_CASE( "Constructor_With_Coordinate_And_LINEAR_Interpolation", "[libopenshot][point]" )
TEST_CASE( "constructor from Coordinate, LINEAR", "[libopenshot][point]" )
{
// Create a point with a coordinate and interpolation
openshot::Coordinate c1(3,9);
Expand All @@ -93,7 +93,7 @@ TEST_CASE( "Constructor_With_Coordinate_And_LINEAR_Interpolation", "[libopenshot
CHECK(p1.interpolation == openshot::InterpolationType::LINEAR);
}

TEST_CASE( "Constructor_With_Coordinate_And_BEZIER_Interpolation", "[libopenshot][point]" )
TEST_CASE( "constructor from Coordinate, BEZIER", "[libopenshot][point]" )
{
// Create a point with a coordinate and interpolation
openshot::Coordinate c1(3,9);
Expand All @@ -105,7 +105,7 @@ TEST_CASE( "Constructor_With_Coordinate_And_BEZIER_Interpolation", "[libopenshot
CHECK(p1.interpolation == openshot::InterpolationType::BEZIER);
}

TEST_CASE( "Constructor_With_Coordinate_And_CONSTANT_Interpolation", "[libopenshot][point]" )
TEST_CASE( "constructor from Coordinate, CONSTANT", "[libopenshot][point]" )
{
// Create a point with a coordinate and interpolation
openshot::Coordinate c1(2,8);
Expand All @@ -117,7 +117,7 @@ TEST_CASE( "Constructor_With_Coordinate_And_CONSTANT_Interpolation", "[libopensh
CHECK(p1.interpolation == openshot::InterpolationType::CONSTANT);
}

TEST_CASE( "Constructor_With_Coordinate_And_BEZIER_And_AUTO_Handle", "[libopenshot][point]" )
TEST_CASE( "constructor from Coordinate, BEZIER+AUTO", "[libopenshot][point]" )
{
// Create a point with a coordinate and interpolation
openshot::Coordinate c1(3,9);
Expand All @@ -131,7 +131,7 @@ TEST_CASE( "Constructor_With_Coordinate_And_BEZIER_And_AUTO_Handle", "[libopensh
CHECK(p1.handle_type == openshot::HandleType::AUTO);
}

TEST_CASE( "Constructor_With_Coordinate_And_BEZIER_And_MANUAL_Handle", "[libopenshot][point]" )
TEST_CASE( "constructor from Coordinate, BEZIER+MANUAL", "[libopenshot][point]" )
{
// Create a point with a coordinate and interpolation
openshot::Coordinate c1(3,9);
Expand Down

0 comments on commit 89b1643

Please sign in to comment.