Skip to content

Commit 65883a2

Browse files
committed
Move invalid update values data provider to operation tests
1 parent 3a6483d commit 65883a2

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

tests/Operation/TestCase.php

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,4 +74,14 @@ public function provideEmptyUpdatePipelinesExcludingArray(): array
7474
'empty_pipeline:PackedArray' => [PackedArray::fromPHP([])],
7575
];
7676
}
77+
78+
public function provideInvalidUpdateValues(): array
79+
{
80+
return $this->wrapValuesForDataProvider($this->getInvalidUpdateValues());
81+
}
82+
83+
protected function getInvalidUpdateValues(): array
84+
{
85+
return [123, 3.14, 'foo', true];
86+
}
7787
}

tests/TestCase.php

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -140,11 +140,6 @@ public function provideInvalidIntegerValues()
140140
return $this->wrapValuesForDataProvider($this->getInvalidIntegerValues());
141141
}
142142

143-
public function provideInvalidUpdateValues()
144-
{
145-
return $this->wrapValuesForDataProvider($this->getInvalidUpdateValues());
146-
}
147-
148143
protected function assertDeprecated(callable $execution): void
149144
{
150145
$errors = [];
@@ -307,11 +302,6 @@ protected function getInvalidStringValues(bool $includeNull = false): array
307302
return array_merge([123, 3.14, true, [], new stdClass()], $includeNull ? [null] : []);
308303
}
309304

310-
protected function getInvalidUpdateValues(): array
311-
{
312-
return [123, 3.14, 'foo', true];
313-
}
314-
315305
/**
316306
* Return a list of invalid WriteConcern values.
317307
*/

0 commit comments

Comments
 (0)