test: Add conditions tests

Author: Greg0

src/Datafile/Conditions.php

@@ -8,6 +8,7 @@
 use Featurevisor\Datafile\Conditions\ConditionFactory;
 use Featurevisor\Datafile\Conditions\ConditionInterface;
 use Featurevisor\Datafile\Conditions\EveryoneCondition;
+use Featurevisor\Datafile\Conditions\NotCondition;
 
 final class Conditions implements ConditionInterface
 {
@@ -23,6 +24,10 @@ public static function createFromMixed($conditions): self
             return new self(new EveryoneCondition());
         }
 
+        if (is_string($conditions)) { // Unsupported string condition
+            return new self(new NotCondition(new EveryoneCondition()));
+        }
+
         if (is_array($conditions) === false) {
             throw new \InvalidArgumentException('Conditions must be array or string');
         }

src/Datafile/Conditions/AfterCondition.php

@@ -6,6 +6,8 @@
 
 
 use DateTimeImmutable;
+use DateTimeInterface;
+use Exception;
 
 final class AfterCondition implements ConditionInterface
 {
@@ -22,8 +24,23 @@ public function __construct(string $attribute, DateTimeImmutable $value)
 
     public function isSatisfiedBy(array $context): bool
     {
-        $contextDate = new DateTimeImmutable($this->getValueFromContext($context, $this->attribute));
-
-        return $contextDate > $this->value;
+        try {
+            $valueFromContext = $this->getValueFromContext($context, $this->attribute);
+            if ($valueFromContext === null) {
+                return false;
+            }
+            if ($valueFromContext instanceof DateTimeInterface) {
+                $contextDate = DateTimeImmutable::createFromFormat(
+                    DateTimeInterface::RFC3339,
+                    $valueFromContext->format(DateTimeInterface::RFC3339)
+                );
+            } else {
+                $contextDate = new DateTimeImmutable($valueFromContext);
+            }
+
+            return $contextDate > $this->value;
+        } catch (Exception $e) {
+            return false;
+        }
     }
 }

src/Datafile/Conditions/BeforeCondition.php

@@ -6,6 +6,7 @@
 
 
 use DateTimeImmutable;
+use DateTimeInterface;
 
 final class BeforeCondition implements ConditionInterface
 {
@@ -22,8 +23,23 @@ public function __construct(string $attribute, DateTimeImmutable $value)
 
     public function isSatisfiedBy(array $context): bool
     {
-        $contextDate = new DateTimeImmutable($this->getValueFromContext($context, $this->attribute));
-
-        return $contextDate < $this->value;
+        try {
+            $valueFromContext = $this->getValueFromContext($context, $this->attribute);
+            if ($valueFromContext === null) {
+                return false;
+            }
+            if ($valueFromContext instanceof DateTimeInterface) {
+                $contextDate = DateTimeImmutable::createFromFormat(
+                    DateTimeInterface::RFC3339,
+                    $valueFromContext->format(DateTimeInterface::RFC3339)
+                );
+            } else {
+                $contextDate = new DateTimeImmutable($valueFromContext);
+            }
+
+            return $contextDate < $this->value;
+        } catch (\Exception $e) {
+            return false;
+        }
     }
 }

src/Datafile/Conditions/ConditionFactory.php

@@ -12,6 +12,10 @@ final class ConditionFactory
 {
     public function create(array $conditions): ConditionInterface
     {
+        if (array_key_exists('attribute', $conditions)) {
+            return $this->createCondition($conditions);
+        }
+
         $mappedConditions = array_map(fn ($condition) => $this->map($condition), $conditions);
 
         if (count($mappedConditions) === 1) {
@@ -44,13 +48,12 @@ private function map(array $condition): ConditionInterface
 
     private function createCondition(array $condition): ConditionInterface
     {
-        if (!isset($condition['attribute']) || !isset($condition['operator']) || !isset($condition['value'])) {
-            var_dump($condition);
+        if (!isset($condition['attribute']) || !isset($condition['operator'])) {
             throw new InvalidArgumentException('Invalid condition format');
         }
 
         $attribute = $condition['attribute'];
-        $value = $condition['value'];
+        $value = $condition['value'] ?? null;
 
         switch ($condition['operator']) {
             case 'after':
@@ -88,9 +91,13 @@ private function createCondition(array $condition): ConditionInterface
             case 'lessThanOrEquals':
                 return new LessThanOrEqualsCondition($attribute, $value);
             case 'matches':
-                return new MatchesCondition($attribute, sprintf('/%s/%s', $value, $condition['regexFlags']));
+                return new MatchesCondition($attribute, sprintf('/%s/%s', $value, $condition['regexFlags'] ?? ''));
+            case 'notMatches':
+                return new NotCondition(new MatchesCondition($attribute, sprintf('/%s/%s', $value, $condition['regexFlags'] ?? '')));
             case 'semverEquals':
                 return new SemverEqualsCondition($attribute, $value);
+            case 'semverNotEquals':
+                return new NotCondition(new SemverEqualsCondition($attribute, $value));
             case 'semverGreaterThan':
                 return new SemverGreaterThanCondition($attribute, $value);
             case 'semverGreaterThanOrEquals':

src/Datafile/Conditions/ContainsCondition.php

@@ -20,6 +20,6 @@ public function __construct(string $attribute, string $value)
 
     public function isSatisfiedBy(array $context): bool
     {
-        return str_contains($this->getValueFromContext($context, $this->attribute), $this->value);
+        return str_contains((string) $this->getValueFromContext($context, $this->attribute), $this->value);
     }
 }

src/Datafile/Conditions/EndsWithCondition.php

@@ -20,6 +20,6 @@ public function __construct(string $attribute, string $value)
 
     public function isSatisfiedBy(array $context): bool
     {
-        return str_ends_with($this->getValueFromContext($context, $this->attribute), $this->value);
+        return str_ends_with((string) $this->getValueFromContext($context, $this->attribute), $this->value);
     }
 }

src/Datafile/Conditions/GreaterThanCondition.php

@@ -26,6 +26,11 @@ public function __construct(string $attribute, $value)
 
     public function isSatisfiedBy(array $context): bool
     {
-        return $this->getValueFromContext($context, $this->attribute) > $this->value;
+        $valueFromContext = $this->getValueFromContext($context, $this->attribute);
+        if ($valueFromContext === null) {
+            return false;
+        }
+
+        return $valueFromContext > $this->value;
     }
 }

src/Datafile/Conditions/GreaterThanOrEqualsCondition.php

@@ -16,7 +16,7 @@ final class GreaterThanOrEqualsCondition implements ConditionInterface
 
     public function __construct(string $attribute, $value)
     {
-        if (is_int($value) === false || is_float($value) === false) {
+        if (is_int($value) === false && is_float($value) === false) {
             throw new \InvalidArgumentException('GreaterThanOrEqualCondition value must be float or integer');
         }
 

src/Datafile/Conditions/LessThanCondition.php

@@ -26,6 +26,11 @@ public function __construct(string $attribute, $value)
 
     public function isSatisfiedBy(array $context): bool
     {
-        return $this->getValueFromContext($context, $this->attribute) < $this->value;
+        $valueFromContext = $this->getValueFromContext($context, $this->attribute);
+        if ($valueFromContext === null) {
+            return false;
+        }
+
+        return $valueFromContext < $this->value;
     }
 }

src/Datafile/Conditions/LessThanOrEqualsCondition.php

@@ -16,7 +16,7 @@ final class LessThanOrEqualsCondition implements ConditionInterface
 
     public function __construct(string $attribute, $value)
     {
-        if (is_int($value) === false || is_float($value) === false) {
+        if (is_int($value) === false && is_float($value) === false) {
             throw new \InvalidArgumentException('LessThanOrEqualsCondition value must be float or integer');
         }