Merge pull request #13 from JonPurvis/update/testing-changes

stronger tests

README.md

@@ -191,27 +191,19 @@ beforeEach(function () {
     $this->fake->addProvider(new Stripe($this->fake));
 });
 
-it('shows an example', function () {
-    $this->fake->stripeConnectAccountId() // acct_xBXg7yyrSyQVbsjM
+it('generates a well structured stripe account id', function () {
+   expect($this->fake->stripeConnectAccountId())->toStartWith('acct_')->toHaveLength(21)->toBeString();
 });
 ```
 
 ### PHPUnit
 ```php
-
-private Generator $faker;
-
-protected function setUp(): void
+public function testGeneratesWellStructuredStripeAccountId()
 {
     $faker = Factory::create();
     $faker->addProvider(new Stripe($faker));
 
-    $this->faker = $faker;
-}
-
-public function testShowExample()
-{
-    $this->faker->stripeConnectAccountId() // acct_xBXg7yyrSyQVbsjM
+    $this->assertStringStartsWith('acct_', $faker->stripeConnectAccountId());
 }
 ```
 
@@ -223,6 +215,10 @@ public function definition(): array
 {
     $this->faker->addProvider(new Stripe($this->faker));
 
-    $this->faker->stripeConnectAccountId() // acct_xBXg7yyrSyQVbsjM
+    return [
+        'id' => $this->faker->stripeConnectAccountId(),
+        'name' => 'John Doe',
+        'email' => 'john.doe@testing.co.uk'
+    ];   
 }
 ```

composer.json

@@ -13,10 +13,10 @@
     ],
     "require": {
         "php": "^8.1",
-        "fakerphp/faker": "^1.10"
+        "fakerphp/faker": "^1.23"
     },
     "require-dev": {
-        "pestphp/pest": "^2.6",
+        "pestphp/pest": "^2.12",
         "pestphp/pest-plugin-faker": "^2.0"
     },
     "autoload": {

tests/ArchitectureTest.php

@@ -0,0 +1,9 @@
+<?php
+
+use Faker\Provider\Base;
+use Faker\Provider\Stripe;
+
+test('provider')
+    ->expect(Stripe::class)
+    ->toExtend(Base::class)
+    ->toOnlyUse(Base::class);