wip

Author: osbre

.github/workflows/tests.yml

@@ -36,7 +36,7 @@ jobs:
 
         env:
             WP_VERSION: ${{ matrix.wordpress }}
-            WP_SITE_URL: http://127.0.0.1:8100
+            WP_SITE_URL: http://localhost:8100
             WP_DB_NAME: wordpress
             WP_DB_USER: root
             WP_DB_PASS: root
@@ -104,7 +104,7 @@ jobs:
 
             -   name: Start PHP server
                 run: |
-                    php -S 127.0.0.1:8100 -t wordpress > /dev/null 2>&1 & until curl -sSf http://127.0.0.1:8100 > /dev/null; do sleep 0.2; done
+                    php -S localhost:8100 -t wordpress > /dev/null 2>&1 & until curl -sSf http://localhost:8100 > /dev/null; do sleep 0.2; done
 
             -   name: Install pnpm
                 uses: pnpm/action-setup@v4

tests/Browser/PluginSettingsTest.php

@@ -4,14 +4,25 @@
 
 use function Tests\asAdmin;
 
-const DEFAULT_SA_NOTICE = '<!-- Simple Analytics: Not logging requests from admins -->';
+const SA_ADMIN_NOTICE = '<!-- Simple Analytics: Not logging requests from admins -->';
+const SA_DEFAULT_SCRIPT = '<script type="text/javascript" src="https://scripts.simpleanalyticscdn.com/latest.js"></script>';
+const WP_PLUGIN_ROW_SELECTOR = 'tr[data-slug="simpleanalytics"]';
+const WP_ACTIVATE_PLUGIN_SELECTOR = '#activate-simpleanalytics';
 
-test('Sign in as admin and activate the plugin', function () {
-    $page = asAdmin();
+it('can be activated', function () {
+    asAdmin()
+        ->navigate('http://localhost:8100/wp-admin/plugins.php')
+        ->assertPresent(WP_PLUGIN_ROW_SELECTOR)
+        ->click(WP_ACTIVATE_PLUGIN_SELECTOR);
+});
 
-    $page->navigate('http://127.0.0.1:8100/wp-admin/plugins.php')->screenshot();
-    $page->assertSee('Simple Analytics Official');
+it('adds a script by default', function () {
+    $homePage = visit('http://localhost:8100');
+    expect($homePage->content())->toContain(SA_DEFAULT_SCRIPT);
+});
 
-    $page2 = visit('http://127.0.0.1:8100');//->screenshot();
-    expect($page2->content())->not->toContain(DEFAULT_SA_NOTICE);
+it('adds a comment when an authenticated user visits', function () {
+    $homePage = asAdmin()->navigate('http://localhost:8100');
+    expect($homePage->content())->toContain(SA_ADMIN_NOTICE);
+    $homePage->assertNoConsoleLogs();
 });

tests/Pest.php

@@ -43,12 +43,9 @@
 
 function asAdmin()
 {
-    $page = visit('http://127.0.0.1:8100/wp-admin');
-
-    $page->fill('user_login', 'admin');
-    $page->fill('user_pass', 'admin');
-    $page->press('wp-submit');
-    $page->assertUrlIs('http://127.0.0.1:8100/wp-admin');
-
-    return $page;
+    return visit('http://localhost:8100/wp-admin')
+        ->fill('user_login', 'admin')
+        ->fill('user_pass', 'admin')
+        ->press('wp-submit')
+        ->assertUrlIs('http://localhost:8100/wp-admin');
 }