bug #61 Fix PasswordStrength doesn't work with named args (sstok)

This PR was merged into the main branch.

Discussion
----------

| Q             | A
| ------------- | ---
| Branch?       | main
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Fixed tickets | Fix #60
| License       | MIT


Commits
-------

f3b8a51 Fix PasswordStrength doesn't work with named args

src/Validator/Constraints/PasswordStrength.php

@@ -37,9 +37,21 @@ public function __construct(
         string $message = null,
         string $tooShortMessage = null
     ) {
-        parent::__construct($options ?? [], $groups, $payload);
+        $finalOptions = [];
+
+        if (is_array($options)) {
+            $finalOptions = $options;
+        } else {
+            $finalOptions['minStrength'] = $options;
+        }
+
+        // The minStrength option is required.
+        if ($minStrength !== null) {
+            $finalOptions['minStrength'] = $minStrength;
+        }
+
+        parent::__construct($finalOptions ?? [], $groups, $payload);
 
-        $this->minStrength = $minStrength ?? $this->minStrength;
         $this->minLength = $minLength ?? $this->minLength;
         $this->unicodeEquality = $unicodeEquality ?? $this->unicodeEquality;
         $this->message = $message ?? $this->message;

tests/Validator/PasswordStrengthTest.php

@@ -38,6 +38,22 @@ protected function createValidator()
         return new PasswordStrengthValidator(new Translator('en'));
     }
 
+    /** @test */
+    public function constraints_options_are_properly_resolved(): void
+    {
+        // Default option
+        $constraint = new PasswordStrength(3);
+        self::assertEquals(3, $constraint->minStrength);
+
+        // By option
+        $constraint = new PasswordStrength(['minStrength' => 3]);
+        self::assertEquals(3, $constraint->minStrength);
+
+        // Specific argument
+        $constraint = new PasswordStrength(null, null, null, 3);
+        self::assertEquals(3, $constraint->minStrength);
+    }
+
     /**
      * @test
      */