[Rector] Apply Rector: RemoveDuplicatedArrayKeyRector

[samsonasik]

Remove duplicated key in defined arrays with RemoveDuplicatedArrayKeyRector rector rule.

Checklist:

• Securely signed commits

[MGatner]

The array keys on the provider in CreditCardRulesTest.php should stay. These are used to name data sets during test failure. (e.g. "testValidCCNumber failed with data set 'not_numierc'") and make it easier to track down the actual failure - otherwise it enumerates all the data from the provider which can make for a messy output.

[samsonasik]

@MGatner when the key are same, only last that used, do you think that should be named indexed, eg: hsbc1, etc

            'hsbc' => [
                 'hsbc',
                 $this->generateCardNum(56, 16),
                 true,
             ],
-             'hsbc' => [
+             'hsbc1' => [
                 'hsbc',
                 $this->generateCardNum(57, 16),

[MGatner]

Yes

[samsonasik]

@MGatner implemented

[samsonasik]

@MGatner I am merging it