[1.x] Support DB configurations that _require_ a unique ID (#142)

timacdonald · jbrooksuk · web-flow · commit 03e1c2262213 · 2023-12-05T08:45:06.000-06:00
* Support DB configurations that _require_ a unique ID

* Add upgrading guide

* Unset the ID for expectations

* Fix code styling

* Update UPGRADE.MD

---------

Co-authored-by: timacdonald &lt;timacdonald@users.noreply.github.com&gt;
Co-authored-by: James Brooks &lt;james@alt-three.com&gt;
diff --git a/UPGRADE.MD b/UPGRADE.MD
@@ -0,0 +1,6 @@
+# Upgrade Guide
+
+# Beta to 1.x
+
+- [Auto-incrementing IDs were added to Pulse's tables](https://github.com/laravel/pulse/pull/142). This is recommended if you are using a configuration that requires tables to have a unique key on every table, e.g., PlanetScale.
+
diff --git a/database/migrations/2023_06_07_000001_create_pulse_tables.php b/database/migrations/2023_06_07_000001_create_pulse_tables.php
@@ -22,6 +22,7 @@ public function up(): void
     {
         Schema::create('pulse_values', function (Blueprint $table) {
             $table->engine = 'InnoDB';
+            $table->id();
             $table->unsignedInteger('timestamp');
             $table->string('type');
             $table->text('key');
@@ -35,6 +36,7 @@ public function up(): void
 
         Schema::create('pulse_entries', function (Blueprint $table) {
             $table->engine = 'InnoDB';
+            $table->id();
             $table->unsignedInteger('timestamp');
             $table->string('type');
             $table->text('key');
@@ -49,6 +51,7 @@ public function up(): void
 
         Schema::create('pulse_aggregates', function (Blueprint $table) {
             $table->engine = 'InnoDB';
+            $table->id();
             $table->unsignedInteger('bucket');
             $table->unsignedMediumInteger('period');
             $table->string('type');
diff --git a/tests/Pest.php b/tests/Pest.php
@@ -55,6 +55,10 @@
 expect()->extend('toContainAggregateForAllPeriods', function (string|array $type, string $aggregate, string $key, int $value, int $count = null, int $timestamp = null) {
     $this->toBeInstanceOf(Collection::class);
 
+    $values = $this->value->each(function (stdClass $value) {
+        unset($value->id);
+    });
+
     $types = (array) $type;
     $timestamp ??= now()->timestamp;
 
