test(functional/activation): add functional and unit test coverage

Add functional tests for database activation lifecycle and table storage behavior. Extend unit tests for `ActivationExecute` and `ActivationTasks` to ensure proper task registration and lifecycle execution. Update `yarn.lock` to use npm registry consistently.

Author: widoz

phpunit.xml

@@ -24,6 +24,7 @@
 		<exclude>
 			<directory>./sources/**/**/build-module</directory>
 			<directory>./sources/**/**/dist</directory>
+			<file>./sources/Blocks/blocks-manifest.php</file>
 		</exclude>
 	</source>
 

tests/functional/php/Blocks/BlockHooksTest.php

@@ -4,6 +4,8 @@
 
 namespace SpaghettiDojo\Konomi\Tests\Functional\Blocks;
 
+use SpaghettiDojo\Konomi\Blocks\Konomi\HookedContent;
+
 // Block Hooks auto-insertion of `konomi/konomi` after `core/post-title`. The hooked
 // instance is inserted bare by WordPress; the `hooked_block_konomi/konomi` filter
 // (SpaghettiDojo\Konomi\Blocks\Konomi\HookedContent) must populate it with the default
@@ -47,7 +49,7 @@
     });
 
     it('does not override a block that already has inner blocks', function (): void {
-        $hookedContent = \SpaghettiDojo\Konomi\Blocks\Konomi\HookedContent::new();
+        $hookedContent = HookedContent::new();
 
         $alreadyPopulated = [
             'blockName' => 'konomi/konomi',
@@ -69,4 +71,19 @@
 
         expect($result)->toBe($alreadyPopulated);
     });
+
+    it('leaves a suppressed (non-array) hooked block untouched', function (): void {
+        $hookedContent = HookedContent::new();
+
+        // A prior filter may have returned null to suppress the insertion.
+        $result = $hookedContent->injectDefaultInnerBlocks(
+            null,
+            'konomi/konomi',
+            'after',
+            null,
+            null
+        );
+
+        expect($result)->toBeNull();
+    });
 });

tests/functional/php/Database/ModuleTest.php

@@ -0,0 +1,53 @@
+<?php
+
+declare(strict_types=1);
+
+namespace SpaghettiDojo\Konomi\Tests\Functional\Database;
+
+use SpaghettiDojo\Konomi\Activation\ActivationTasks;
+use SpaghettiDojo\Konomi\Database\InteractionsTable;
+use SpaghettiDojo\Konomi\Database\Module;
+
+use function SpaghettiDojo\Konomi\package;
+
+function tableExists(string $name): bool
+{
+    global $wpdb;
+
+    $wpdb->suppress_errors(true);
+    // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
+    $result = $wpdb->query("SELECT 1 FROM {$name} LIMIT 1");
+    $wpdb->suppress_errors(false);
+
+    return $result !== false;
+}
+
+describe('Database\Module::services()', function (): void {
+    it('exposes the InteractionsTable named from the wpdb prefix', function (): void {
+        global $wpdb;
+        $table = package()->container()->get(InteractionsTable::class);
+        expect($table->name())->toBe($wpdb->prefix . 'konomi_interactions');
+    });
+});
+
+describe('Database\Module::activate()', function (): void {
+    it('registers activation/uninstall tasks that create and drop the real table', function (): void {
+        $container = package()->container();
+        $table = $container->get(InteractionsTable::class);
+        $tasks = $container->get(ActivationTasks::class);
+
+        Module::new()->activate($tasks, $container);
+
+        expect($tasks->activationTasks())->toHaveCount(1)
+            ->and($tasks->uninstallTasks())->toHaveCount(1)
+            ->and($tasks->deactivationTasks())->toBe([]);
+
+        // The table already exists (created in WpTestCase::setUp). The uninstall
+        // task drops it, the activation task recreates it (idempotent via dbDelta).
+        ($tasks->uninstallTasks()[0])();
+        expect(tableExists($table->name()))->toBeFalse();
+
+        ($tasks->activationTasks()[0])();
+        expect(tableExists($table->name()))->toBeTrue();
+    });
+});

tests/functional/php/Storage/TableStorageTest.php

@@ -0,0 +1,85 @@
+<?php
+
+declare(strict_types=1);
+
+namespace SpaghettiDojo\Konomi\Tests\Functional\Storage;
+
+use SpaghettiDojo\Konomi\Database\InteractionsTable;
+use SpaghettiDojo\Konomi\Storage\Axis;
+use SpaghettiDojo\Konomi\Storage\Record;
+use SpaghettiDojo\Konomi\Storage\TableStorage;
+
+beforeEach(function (): void {
+    global $wpdb;
+    $this->table = InteractionsTable::new($wpdb->prefix);
+    $this->storage = TableStorage::new($this->table);
+});
+
+describe('TableStorage round trip', function (): void {
+    it('writes and reads records on the Entity axis', function (): void {
+        $ok = $this->storage->write(Axis::Entity, 10, 'reaction', [
+            new Record(0, 5, 'post'),
+            new Record(0, 7, 'post'),
+        ]);
+        expect($ok)->toBeTrue();
+
+        $records = $this->storage->read(Axis::Entity, 10, 'reaction');
+        $userIds = array_map(static fn (Record $record) => $record->userId, $records);
+        sort($userIds);
+
+        expect($records)->toHaveCount(2)
+            ->and($userIds)->toBe([5, 7])
+            ->and($records[0]->entityId)->toBe(10)
+            ->and($records[0]->entityType)->toBe('post');
+    });
+
+    it('writes and reads records on the User axis', function (): void {
+        $ok = $this->storage->write(Axis::User, 99, 'bookmark', [
+            new Record(3, 0, 'post'),
+            new Record(4, 0, 'page'),
+        ]);
+        expect($ok)->toBeTrue();
+
+        $records = $this->storage->read(Axis::User, 99, 'bookmark');
+        $entityIds = array_map(static fn (Record $record) => $record->entityId, $records);
+        sort($entityIds);
+
+        expect($records)->toHaveCount(2)
+            ->and($entityIds)->toBe([3, 4])
+            ->and($records[0]->userId)->toBe(99);
+    });
+
+    it('replaces the previous set for the same axis id and group on write', function (): void {
+        $this->storage->write(Axis::Entity, 10, 'reaction', [new Record(0, 5, 'post')]);
+        $this->storage->write(Axis::Entity, 10, 'reaction', [new Record(0, 8, 'post')]);
+
+        $records = $this->storage->read(Axis::Entity, 10, 'reaction');
+
+        expect($records)->toHaveCount(1)
+            ->and($records[0]->userId)->toBe(8);
+    });
+
+    it('keeps different group keys isolated', function (): void {
+        $this->storage->write(Axis::Entity, 10, 'reaction', [new Record(0, 5, 'post')]);
+
+        expect($this->storage->read(Axis::Entity, 10, 'bookmark'))->toBe([]);
+    });
+
+    it('filters out malformed rows on read', function (): void {
+        global $wpdb;
+
+        $this->storage->write(Axis::Entity, 10, 'reaction', [new Record(0, 5, 'post')]);
+
+        // A row with an empty entity_type must be dropped by TableStorage::mapRow().
+        $wpdb->insert(
+            $this->table->name(),
+            ['entity_id' => 10, 'user_id' => 6, 'entity_type' => '', 'group_key' => 'reaction'],
+            ['%d', '%d', '%s', '%s']
+        );
+
+        $records = $this->storage->read(Axis::Entity, 10, 'reaction');
+
+        expect($records)->toHaveCount(1)
+            ->and($records[0]->userId)->toBe(5);
+    });
+});