wip

Author: osbre

tests/Browser/PluginSettingsTest.php

@@ -4,8 +4,8 @@
 
 use function Tests\{asAdmin, asAuthor, asEditor};
 
-const SA_ADMIN_NOTICE = '<!-- Simple Analytics: Not logging requests from admins -->';
 const SA_DEFAULT_SCRIPT = 'src="https://scripts.simpleanalyticscdn.com/latest.js"></script>';
+const SA_INACTIVE_ADMIN_NOTICE = '<!-- Simple Analytics: Not logging requests from admins -->';
 const SA_INACTIVE_ADMIN_SCRIPT = 'src="http://localhost:8100/wp-content/plugins/simpleanalytics/resources/js/inactive.js"';
 
 it('can be activated', function () {
@@ -27,7 +27,7 @@
     $homePage = asAdmin()->navigate('http://localhost:8100');
 
     expect($homePage->content())
-        ->toContain(SA_ADMIN_NOTICE)
+        ->toContain(SA_INACTIVE_ADMIN_NOTICE)
         ->toContain(SA_INACTIVE_ADMIN_SCRIPT);
 });
 
@@ -54,19 +54,23 @@
 });
 
 it('adds inactive script for selected user roles', function () {
-    asAdmin()
-        ->navigate('http://localhost:8100/wp-admin/options-general.php?page=simpleanalytics&tab=ignore-rules')
+    $admin = asAdmin();
+
+    $admin->navigate('http://localhost:8100/wp-admin/options-general.php?page=simpleanalytics&tab=ignore-rules')
         ->check('simpleanalytics_exclude_user_roles-editor')
         ->check('simpleanalytics_exclude_user_roles-author')
         ->click('Save Changes')
         ->assertChecked('simpleanalytics_exclude_user_roles-editor')
         ->assertChecked('simpleanalytics_exclude_user_roles-author');
 
+    expect($admin->navigate('http://localhost:8100')->content())
+        ->toContain(SA_DEFAULT_SCRIPT);
+
     expect(asEditor()->navigate('http://localhost:8100')->content())
-        ->toContain(SA_ADMIN_NOTICE)
+        ->toContain(SA_INACTIVE_ADMIN_NOTICE)
         ->toContain(SA_INACTIVE_ADMIN_SCRIPT);
 
     expect(asAuthor()->navigate('http://localhost:8100')->content())
-        ->toContain(SA_ADMIN_NOTICE)
+        ->toContain(SA_INACTIVE_ADMIN_NOTICE)
         ->toContain(SA_INACTIVE_ADMIN_SCRIPT);
 });