@@ -5,7 +5,6 @@ |
||
5 | 5 | use App\Containers\Stripe\Tasks\CreateStripeAccountObjectTask; |
6 | 6 | use App\Containers\Stripe\Tasks\ChargeWithStripeTask; |
7 | 7 | use App\Containers\Stripe\Tests\TestCase; |
8 | -use App\Ship\Features\Payment\Proxies\PaymentsProxy; |
|
9 | 8 | use Illuminate\Support\Facades\App; |
10 | 9 | |
11 | 10 | /** |
@@ -37,7 +37,7 @@ discard block |
||
37 | 37 | } |
38 | 38 | |
39 | 39 | /** |
40 | - * @param $object |
|
40 | + * @param ChargeableInterface $object |
|
41 | 41 | * |
42 | 42 | * @return null |
43 | 43 | */ |
@@ -47,7 +47,7 @@ discard block |
||
47 | 47 | } |
48 | 48 | |
49 | 49 | /** |
50 | - * @param $object |
|
50 | + * @param ChargeableInterface $object |
|
51 | 51 | * |
52 | 52 | * @return null |
53 | 53 | */ |
@@ -112,9 +112,9 @@ |
||
112 | 112 | |
113 | 113 | |
114 | 114 | /** |
115 | - * @param $statusCode |
|
115 | + * @param integer $statusCode |
|
116 | 116 | * @param $message |
117 | - * @param $code |
|
117 | + * @param integer $code |
|
118 | 118 | */ |
119 | 119 | private function logTheError($statusCode, $message, $code) |
120 | 120 | { |
@@ -154,7 +154,7 @@ |
||
154 | 154 | } |
155 | 155 | |
156 | 156 | /** |
157 | - * @param $user |
|
157 | + * @param User $user |
|
158 | 158 | * @param $access |
159 | 159 | * |
160 | 160 | * @return mixed |
@@ -5,8 +5,6 @@ |
||
5 | 5 | use App; |
6 | 6 | use App\Containers\Authentication\Tasks\ApiLoginThisUserObjectTask; |
7 | 7 | use App\Containers\User\Models\User; |
8 | -use Artisan; |
|
9 | -use Dingo\Api\Http\Response as DingoAPIResponse; |
|
10 | 8 | use Illuminate\Support\Facades\Hash; |
11 | 9 | |
12 | 10 | /** |
@@ -78,7 +78,7 @@ discard block |
||
78 | 78 | */ |
79 | 79 | private function findOrCreateTestingUser($userDetails, $access) |
80 | 80 | { |
81 | - return $this->testingUser ? : $this->createTestingUser($userDetails, $access); |
|
81 | + return $this->testingUser ?: $this->createTestingUser($userDetails, $access); |
|
82 | 82 | } |
83 | 83 | |
84 | 84 | /** |
@@ -126,7 +126,7 @@ discard block |
||
126 | 126 | ]; |
127 | 127 | |
128 | 128 | // if no user detail provided, use the default details, to find the password or generate one before encoding it |
129 | - return $this->prepareUserPassword($userDetails ? : $defaultUserDetails);; |
|
129 | + return $this->prepareUserPassword($userDetails ?: $defaultUserDetails); ; |
|
130 | 130 | } |
131 | 131 | |
132 | 132 | /** |
@@ -161,7 +161,7 @@ discard block |
||
161 | 161 | */ |
162 | 162 | private function setupTestingUserAccess($user, $access) |
163 | 163 | { |
164 | - $access = $access ? : $this->getAccess(); |
|
164 | + $access = $access ?: $this->getAccess(); |
|
165 | 165 | |
166 | 166 | $user = $this->setupTestingUserPermissions($user, $access); |
167 | 167 | $user = $this->setupTestingUserRoles($user, $access); |
@@ -63,9 +63,7 @@ discard block |
||
63 | 63 | |
64 | 64 | /** |
65 | 65 | * @param $this ->endpointUrl |
66 | - * @param string $this ->endpointVerb |
|
67 | 66 | * @param array $data |
68 | - * @param bool $protected |
|
69 | 67 | * @param array $headers |
70 | 68 | * |
71 | 69 | * @return mixed |
@@ -91,7 +89,7 @@ discard block |
||
91 | 89 | * @param bool $skipEncoding |
92 | 90 | * @param string $replace |
93 | 91 | * |
94 | - * @return mixed |
|
92 | + * @return TestsRequestHelperTrait |
|
95 | 93 | */ |
96 | 94 | public function injectId($id, $skipEncoding = false, $replace = '{id}') |
97 | 95 | { |
@@ -108,7 +106,7 @@ discard block |
||
108 | 106 | * |
109 | 107 | * to be used as follow: $this->endpoint('verb@url')->makeCall($data); |
110 | 108 | * |
111 | - * @param $endpoint |
|
109 | + * @param string $endpoint |
|
112 | 110 | * |
113 | 111 | * @return $this |
114 | 112 | */ |
@@ -282,7 +280,7 @@ discard block |
||
282 | 280 | } |
283 | 281 | |
284 | 282 | /** |
285 | - * @param $separator |
|
283 | + * @param string $separator |
|
286 | 284 | */ |
287 | 285 | private function validateEndpointFormat($separator) |
288 | 286 | { |
@@ -61,7 +61,7 @@ |
||
61 | 61 | } |
62 | 62 | |
63 | 63 | /** |
64 | - * @param $keys |
|
64 | + * @param string[] $keys |
|
65 | 65 | * @param $httpResponse |
66 | 66 | */ |
67 | 67 | public function assertResponseContainKeys($keys, $httpResponse) |
@@ -2,7 +2,6 @@ |
||
2 | 2 | |
3 | 3 | namespace App\Ship\Parents\Tests\PhpUnit; |
4 | 4 | |
5 | -use App\Ship\Features\Tests\PhpUnit\TestingTrait; |
|
6 | 5 | use App\Ship\Features\Tests\PhpUnit\TestsAuthHelperTrait; |
7 | 6 | use App\Ship\Features\Tests\PhpUnit\TestsCustomHelperTrait; |
8 | 7 | use App\Ship\Features\Tests\PhpUnit\TestsMockHelperTrait; |
@@ -2,7 +2,7 @@ |
||
2 | 2 | |
3 | 3 | use Illuminate\Support\Facades\Hash; |
4 | 4 | |
5 | -$factory->define(App\Containers\User\Models\User::class, function (Faker\Generator $faker) { |
|
5 | +$factory->define(App\Containers\User\Models\User::class, function(Faker\Generator $faker) { |
|
6 | 6 | |
7 | 7 | return [ |
8 | 8 | 'name' => $faker->name, |
@@ -32,7 +32,7 @@ |
||
32 | 32 | return; |
33 | 33 | } |
34 | 34 | |
35 | - $url = ($url) ? : $this->baseUrl; |
|
35 | + $url = ($url) ?: $this->baseUrl; |
|
36 | 36 | |
37 | 37 | $info = parse_url($url); |
38 | 38 |