From a484fb8f22c2f31090a8049400fb103582dd1f20 Mon Sep 17 00:00:00 2001
From: Matthieu Napoli <matthieu@mnapoli.fr>
Date: Wed, 21 Jan 2015 14:18:40 +1300
Subject: [PATCH] Cleared up the annotations
---
tests/PHPUnit/Unit/Period/RangeTest.php | 160 +-----------------------
1 file changed, 4 insertions(+), 156 deletions(-)
diff --git a/tests/PHPUnit/Unit/Period/RangeTest.php b/tests/PHPUnit/Unit/Period/RangeTest.php
index 7f2f715736..cdb3f125d9 100644
--- a/tests/PHPUnit/Unit/Period/RangeTest.php
+++ b/tests/PHPUnit/Unit/Period/RangeTest.php
@@ -15,12 +15,12 @@ use Piwik\Period\Range;
use Piwik\Period\Week;
use Piwik\Period\Year;
+/**
+ * @group Core
+ */
class RangeTest extends BasePeriodTest
{
// test range 1
- /**
- * @group Core
- */
public function testRangeToday()
{
$range = new Range('day', 'last1');
@@ -35,9 +35,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testRangeTodayUtcPlus12()
{
// rather ugly test, UTC+23 doesn't exist, but it's a way to test that last1 in UTC+23 will be "our" UTC tomorrow
@@ -54,9 +51,6 @@ class RangeTest extends BasePeriodTest
}
// test range 2
- /**
- * @group Core
- */
public function testRange2days()
{
$range = new Range('day', 'last2');
@@ -73,9 +67,6 @@ class RangeTest extends BasePeriodTest
}
// test range 3
- /**
- * @group Core
- */
public function testRange5days()
{
$range = new Range('day', 'last50');
@@ -92,9 +83,6 @@ class RangeTest extends BasePeriodTest
}
// test range 4
- /**
- * @group Core
- */
public function testRangePrevious3days()
{
$range = new Range('day', 'previous3');
@@ -111,9 +99,6 @@ class RangeTest extends BasePeriodTest
}
// test range date1,date2
- /**
- * @group Core
- */
public function testRangeComma1()
{
@@ -130,9 +115,6 @@ class RangeTest extends BasePeriodTest
}
// test range date1,date2
- /**
- * @group Core
- */
public function testRangeComma2()
{
@@ -161,9 +143,6 @@ class RangeTest extends BasePeriodTest
}
// test range date1,date2
- /**
- * @group Core
- */
public function testRangeWeekcomma1()
{
$range = new Range('week', '2007-12-22,2008-01-03');
@@ -208,9 +187,6 @@ class RangeTest extends BasePeriodTest
}
// test range date1,date2
- /**
- * @group Core
- */
public function testRangeYearcomma1()
{
$range = new Range('year', '2006-12-22,2007-01-03');
@@ -252,9 +228,6 @@ class RangeTest extends BasePeriodTest
}
// test range date1,date2
- /**
- * @group Core
- */
public function testRangeMonthcomma1()
{
$range = new Range('month', '2006-12-22,2007-01-03');
@@ -334,9 +307,6 @@ class RangeTest extends BasePeriodTest
}
// test range WEEK
- /**
- * @group Core
- */
public function testRangeWeek()
{
$range = new Range('week', 'last50');
@@ -356,9 +326,6 @@ class RangeTest extends BasePeriodTest
}
// test range WEEK last1
- /**
- * @group Core
- */
public function testRangeWeekLast1()
{
$range = new Range('week', 'last1');
@@ -368,9 +335,6 @@ class RangeTest extends BasePeriodTest
}
// test range MONTH
- /**
- * @group Core
- */
public function testRangeMonth()
{
$range = new Range('month', 'last20');
@@ -390,9 +354,6 @@ class RangeTest extends BasePeriodTest
}
// test range MONTH last1
- /**
- * @group Core
- */
public function testRangeMonthLast1()
{
$range = new Range('month', 'last1');
@@ -402,9 +363,6 @@ class RangeTest extends BasePeriodTest
}
// test range PREVIOUS MONTH
- /**
- * @group Core
- */
public function testRangePreviousmonth()
{
$range = new Range('month', 'previous10');
@@ -424,9 +382,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testRangePreviousmonth_onLastDayOfMonth()
{
$end = Date::factory('2013-10-31');
@@ -446,9 +401,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testRangePreviousweek_onLastDayOfWeek()
{
$end = Date::factory('2013-11-03');
@@ -465,9 +417,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testRangePreviousweek_onFirstDayOfWeek()
{
$end = Date::factory('2013-11-04');
@@ -484,9 +433,7 @@ class RangeTest extends BasePeriodTest
$correct = array_reverse($correct);
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
+
public function testRangeLastweek_onFirstDayOfWeek()
{
$end = Date::factory('2013-11-04');
@@ -503,9 +450,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testRangeLastmonth_onLastDayOfMonth()
{
$end = Date::factory('2013-10-31');
@@ -524,9 +468,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function _testRangePreviousmonth_onFirstOfMonth()
{
$end = Date::factory('2013-11-01');
@@ -546,9 +487,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function _testRangeLastmonth_onFirstOfMonth()
{
$end = Date::factory('2013-11-01');
@@ -568,9 +506,6 @@ class RangeTest extends BasePeriodTest
}
// test range YEAR
- /**
- * @group Core
- */
public function testRangeYear()
{
$range = new Range('year', 'last10');
@@ -590,9 +525,6 @@ class RangeTest extends BasePeriodTest
}
// test range YEAR last1
- /**
- * @group Core
- */
public function testRangeYearLast1()
{
$range = new Range('year', 'last1');
@@ -601,9 +533,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals(array($currentYear->toString()), $range->toString());
}
- /**
- * @group Core
- */
public function testCustomRangeYearUsesYearIfPossible()
{
$range = new Range('range', '2005-12-17,2008-01-03', 'UTC', Date::factory('2008-01-03'));
@@ -638,9 +567,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testCustomRangeIsYear_UsesFullYear()
{
$range = new Range('range', '2011-01-01,2011-12-31', 'UTC', Date::factory('2012-01-03'));
@@ -654,9 +580,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testCustomRangeYear_UsesCurrentYear()
{
$rangeString = '2013-01-01,2013-11-01';
@@ -673,9 +596,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($rangeString, $range->getRangeString());
}
- /**
- * @group Core
- */
public function testCustomRangeYearUsesCurrentYear_onLastDayOfYear()
{
$range = new Range('range', '2013-01-01,2013-12-31', 'UTC', Date::factory('2013-12-31'));
@@ -689,9 +609,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testCustomRangeWeekInsideEndingToday()
{
$range = new Range('range', '2007-12-22,2008-01-03', 'UTC', Date::factory('2008-01-03'));
@@ -722,9 +639,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testRangeEndDateIsTodayAndStartDateNotStartOfTheWeek()
{
$range = new Range('range', '2013-10-29,2013-10-30', 'UTC', Date::factory('2013-10-30'));
@@ -738,9 +652,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testRangeEndDateIsInFuture()
{
$range = new Range('range', '2013-10-29,2013-10-31', 'UTC', Date::factory('2013-10-30'));
@@ -755,9 +666,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testRangePreviousmonthEndDateIsInFutureAndEndOfTheWeek()
{
$range = new Range('range', '2013-10-29,2013-11-03', 'UTC', Date::factory('2013-10-30'));
@@ -775,9 +683,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testCustomRangeWeekInsideEndingYesterday()
{
$todays = array(
@@ -813,9 +718,6 @@ class RangeTest extends BasePeriodTest
}
}
- /**
- * @group Core
- */
public function testCustomRangeOnlyDaysLessThanOneWeek()
{
$range = new Range('range', '2007-12-30,2008-01-01');
@@ -829,9 +731,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testCustomRangeOneWeekOnly()
{
$range = new Range('range', '2007-12-31,2008-01-06');
@@ -851,9 +750,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testCustomRangeStartsWithWeek()
{
$range = new Range('range', '2007-12-31,2008-01-08');
@@ -875,9 +771,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testCustomRangeEndsWithWeek()
{
$range = new Range('range', '2007-12-21,2008-01-06');
@@ -909,9 +802,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testCustomRangeContainsMonthAndWeek()
{
$range = new Range('range', '2011-09-18,2011-11-02', 'UTC', Date::factory('2012-01-01'));
@@ -974,9 +864,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testCustomRangeContainsSeveralMonthsAndWeeksStartingWithMonth()
{
// Testing when "today" is in the same month, or later in the future
@@ -1088,9 +975,6 @@ class RangeTest extends BasePeriodTest
}
}
- /**
- * @group Core
- */
public function testCustomRangeOneMonthOnly()
{
$range = new Range('range', '2011-09-01,2011-09-30');
@@ -1132,9 +1016,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function test_CustomRange_startsWithWeek_EndsWithMonth()
{
$range = new Range('range', '2011-07-25,2011-08-31');
@@ -1187,9 +1068,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testCustomRangeBeforeIsAfterYearRight()
{
try {
@@ -1204,9 +1082,6 @@ class RangeTest extends BasePeriodTest
$this->fail('Expected exception not raised');
}
- /**
- * @group Core
- */
public function testCustomRangeLastN()
{
$range = new Range('range', 'last4');
@@ -1221,9 +1096,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testCustomRangePreviousN()
{
$range = new Range('range', 'previous3');
@@ -1237,9 +1109,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testCustomRangePreviousNEndToday()
{
$range = new Range('range', 'previous3');
@@ -1252,9 +1121,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($correct, $range->toString());
}
- /**
- * @group Core
- */
public function testInvalidRangeThrows()
{
try {
@@ -1266,9 +1132,6 @@ class RangeTest extends BasePeriodTest
$this->fail('Expected exception not raised');
}
- /**
- * @group Core
- */
public function testGetLocalizedShortString()
{
$month = new Range('range', '2000-12-09,2001-02-01');
@@ -1276,9 +1139,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($shouldBe, $month->getLocalizedShortString());
}
- /**
- * @group Core
- */
public function testGetLocalizedLongString()
{
$month = new Range('range', '2023-05-09,2023-05-21');
@@ -1286,9 +1146,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($shouldBe, $month->getLocalizedLongString());
}
- /**
- * @group Core
- */
public function testGetPrettyString()
{
$month = new Range('range', '2007-02-09,2007-03-15');
@@ -1308,9 +1165,6 @@ class RangeTest extends BasePeriodTest
}
/**
- * @group Core
- *
- *
* @dataProvider getDataForLastNLimitsTest
*/
public function testLastNLimits($period, $lastN, $expectedLastN)
@@ -1319,9 +1173,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals($expectedLastN, $range->getNumberOfSubperiods());
}
- /**
- * @group Core
- */
public function testLastWithoutNumber()
{
$range = new Range('day', 'last');
@@ -1329,9 +1180,6 @@ class RangeTest extends BasePeriodTest
$this->assertEquals(1, $range->getNumberOfSubperiods());
}
- /**
- * @group Core
- */
public function testPreviousWithoutNumber()
{
$range = new Range('day', 'previous');
--
GitLab