Skip to content

Commit 509d298

Browse files
authored
fix broken test and update to redislabs/redistimeseries:1.2.1 (palicao#11)
* fix broken test and update to redislabs/redistimeseries:1.2.1 * Improve readme
1 parent e460e24 commit 509d298

File tree

4 files changed

+8
-8
lines changed

4 files changed

+8
-8
lines changed

.travis.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,8 @@ services:
1212
- docker
1313

1414
before_install:
15-
- docker pull redislabs/redistimeseries:1.0.3
16-
- docker run -d -p 127.0.0.1:6379:6379 --name redis redislabs/redistimeseries:1.0.3
15+
- docker pull redislabs/redistimeseries:1.2.1
16+
- docker run -d -p 127.0.0.1:6379:6379 --name redis redislabs/redistimeseries:1.2.1
1717
- yes | pecl install igbinary redis || true
1818
- echo 'extension = redis.so' >> ~/.phpenv/versions/$(phpenv version-name)/etc/php.ini
1919

README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@
22

33
Use [Redis Time Series](https://oss.redislabs.com/redistimeseries/) in PHP!
44

5-
![Code Climate maintainability](https://img.shields.io/codeclimate/coverage-letter/palicao/phpRedisTimeSeries?label=maintainability&logo=code-climate)
6-
![Code Climate coverage](https://img.shields.io/codeclimate/coverage/palicao/phpRedisTimeSeries?logo=code-climate)
5+
[![Maintainability](https://api.codeclimate.com/v1/badges/fea927b90378dd63a9d8/maintainability)](https://codeclimate.com/github/palicao/phpRedisTimeSeries/maintainability)
6+
[![Test Coverage](https://api.codeclimate.com/v1/badges/fea927b90378dd63a9d8/test_coverage)](https://codeclimate.com/github/palicao/phpRedisTimeSeries/test_coverage)
77
[![Build Status](https://travis-ci.com/palicao/phpRedisTimeSeries.svg?branch=master)](https://travis-ci.com/palicao/phpRedisTimeSeries)
88
[![Latest Stable Version](https://img.shields.io/packagist/v/palicao/php-redis-time-series.svg)](https://packagist.org/packages/palicao/php-redis-time-series)
99
![PHP version](https://img.shields.io/packagist/php-v/palicao/php-redis-time-series/2.0.0)

docker-compose.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,4 +11,4 @@ services:
1111
- .:/app
1212
redis:
1313
container_name: redis
14-
image: redislabs/redistimeseries:1.0.3
14+
image: redislabs/redistimeseries:1.2.1

tests/Integration/IntegrationTest.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,11 @@
99
use DateTime;
1010
use DateTimeImmutable;
1111
use Palicao\PhpRedisTimeSeries\AggregationRule;
12+
use Palicao\PhpRedisTimeSeries\Client\RedisClient;
13+
use Palicao\PhpRedisTimeSeries\Client\RedisConnectionParams;
1214
use Palicao\PhpRedisTimeSeries\DateTimeUtils;
1315
use Palicao\PhpRedisTimeSeries\Filter;
1416
use Palicao\PhpRedisTimeSeries\Label;
15-
use Palicao\PhpRedisTimeSeries\Client\RedisClient;
16-
use Palicao\PhpRedisTimeSeries\Client\RedisConnectionParams;
1717
use Palicao\PhpRedisTimeSeries\Sample;
1818
use Palicao\PhpRedisTimeSeries\TimeSeries;
1919
use PHPUnit\Framework\TestCase;
@@ -156,7 +156,7 @@ public function testAddAndRetrieveKeysWithMultipleFilters(): void
156156
public function testAddAndRetrieveWithDateTimeObjectAsMultiRangeWithMultipleFilters(): void
157157
{
158158
$currentDate = new DateTime();
159-
$from = $currentDate->sub(new DateInterval('P1D'));
159+
$from = (clone $currentDate)->sub(new DateInterval('P1D'));
160160
$to = $currentDate;
161161

162162
$this->sut->create(

0 commit comments

Comments
 (0)