chore: add E2E tests

Author: Soare-Robert-Daniel

includes/admin/feedzy-rss-feeds-import.php

@@ -2027,10 +2027,6 @@ function ( $attr, $key ) {
 				);
 			}
 
-			Feedzy_Rss_Feeds_Log::error(
-				'Check error reported. Remove on done'
-			);
-
 			// Used as a new line character in import content.
 			$import_content = rawurldecode( $import_content );
 			$import_content = str_replace( PHP_EOL, "\r\n", $import_content );

tests/e2e/specs/logger.spec.js

@@ -0,0 +1,61 @@
+/**
+ * WordPress dependencies
+ */
+import { test, expect } from '@wordpress/e2e-test-utils-playwright';
+import { createAndRunSampleImport, deleteAllFeedImports } from '../utils';
+
+test.describe('Logger', () => {
+	test.beforeEach(async ({ requestUtils }) => {
+		await deleteAllFeedImports(requestUtils);
+		await requestUtils.deleteAllPosts();
+		await requestUtils.deleteAllMedia();
+	});
+
+	test('check settings rendering', async ({ page, admin }) => {
+		await admin.visitAdminPage('admin.php?page=feedzy-settings');
+
+		await expect(
+			page.locator('select[name="logs-logging-level"]')
+		).toBeVisible();
+
+		await expect(
+			page.getByText('Report errors via email (Once')
+		).toBeVisible();
+	});
+
+	test('check logs tabs', async ({ page, admin }) => {
+		await admin.visitAdminPage('admin.php?page=feedzy-settings');
+
+		await page
+			.locator('select[name="logs-logging-level"]')
+			.selectOption('debug');
+
+		await page
+			.getByRole('button', { name: 'Save Settings' })
+			.click({ force: true });
+
+		// Create some logs via a sample import.
+		await createAndRunSampleImport(page);
+
+		await admin.visitAdminPage('admin.php?page=feedzy-settings&tab=logs');
+
+		await expect(
+			page.getByRole('heading', { name: 'Recent Logs' })
+		).toBeVisible();
+
+		// Check that logs are displayed.
+		expect(
+			await page.locator('.fz-log-container--info').count()
+		).toBeGreaterThan(0);
+		expect(
+			await page.locator('.fz-log-container--debug').count()
+		).toBeGreaterThan(0);
+
+		// Filter messages by Debug.
+		await page.getByRole('link', { name: 'Debug' }).click();
+		expect(await page.locator('.fz-log-container--info').count()).toBe(0);
+		expect(
+			await page.locator('.fz-log-container--debug').count()
+		).toBeGreaterThan(0);
+	});
+});

tests/e2e/utils.js

@@ -5,14 +5,17 @@ import { expect } from '@wordpress/e2e-test-utils-playwright';
  */
 const { RequestUtils } = require('@wordpress/e2e-test-utils-playwright');
 
+export const CUSTOM_FEED_URL =
+	'https://s3.amazonaws.com/verti-utils/sample-feed.xml';
+
 /**
  * Close the tour modal if it is visible.
  *
  * @param {import('playwright').Page} page The page object.
  */
 export async function tryCloseTourModal(page) {
 	if (await page.getByRole('button', { name: 'Skip' }).isVisible()) {
-		await page.getByRole('button', { name: 'Skip' }).click();
+		await page.getByRole('button', { name: 'Skip' }).click({ force: true });
 		await page.waitForTimeout(500);
 	}
 }
@@ -25,6 +28,9 @@ export async function tryCloseTourModal(page) {
  * @return {Promise<void>} The promise that resolves when the feeds are added.
  */
 export async function addFeeds(page, feedURLs) {
+	await page.waitForSelector('input[name="feedzy_meta_data[source]"]', {
+		state: 'attached',
+	});
 	await page.evaluate((urls) => {
 		document.querySelector('input[name="feedzy_meta_data[source]"]').value =
 			urls?.join(', ');
@@ -67,11 +73,21 @@ export async function addContentMapping(page, mapping) {
  * @return {Promise<void>}
  */
 export async function setItemLimit(page, limit) {
-	await page.evaluate((limit) => {
-		document.querySelector(
-			'input[name="feedzy_meta_data[import_feed_limit]"]'
-		).value = limit;
-	}, limit);
+	try {
+		await page.waitForSelector(
+			'input[name="feedzy_meta_data[import_feed_limit]"]',
+			{
+				state: 'attached',
+			}
+		);
+		await page.evaluate((importLimit) => {
+			document.querySelector(
+				'input[name="feedzy_meta_data[import_feed_limit]"]'
+			).value = importLimit;
+		}, limit);
+	} catch (error) {
+		// Element not found or not attached - ignore silently
+	}
 }
 
 /**
@@ -192,3 +208,29 @@ export async function getPostsByFeedzy(requestUtils) {
 		},
 	});
 }
+
+/**
+ * Create and run a sample import with the given feed URL.
+ *
+ * @param {import('playwright').Page} page
+ * @param {string}                    feedUrl
+ *
+ * @return {Promise<void>} The promise that resolves when the import is created and run.
+ */
+export async function createAndRunSampleImport(
+	page,
+	feedUrl = CUSTOM_FEED_URL
+) {
+	const importName = `Create and run sample import: ${new Date().toISOString()}`;
+
+	await page.goto('/wp-admin/post-new.php?post_type=feedzy_imports');
+	await tryCloseTourModal(page);
+
+	await page.getByPlaceholder('Add a name for your import').fill(importName);
+	await addFeeds(page, [feedUrl]);
+	await page
+		.getByRole('button', { name: 'Save & Activate importing' })
+		.click({ force: true });
+
+	await runFeedImport(page);
+}