Merge pull request #2926 from 10up/burhan/improve-commands-test-coverage

Improve wpcli commands test coverage

.wp-env.json

@@ -19,6 +19,7 @@
         "wp-content/plugins/unsupported-server-software.php": "./tests/cypress/wordpress-files/test-plugins/unsupported-server-software.php",
         "wp-content/plugins/unsupported-elasticsearch-version.php": "./tests/cypress/wordpress-files/test-plugins/unsupported-elasticsearch-version.php",
         "wp-content/plugins/shorten-autosave.php": "./tests/cypress/wordpress-files/test-plugins/shorten-autosave.php",
+        "wp-content/plugins/fake-log-messages.php": "./tests/cypress/wordpress-files/test-plugins/fake-log-messages.php",
         "wp-content/uploads/content-example.xml": "./tests/cypress/wordpress-files/test-docs/content-example.xml"
       }
     }

tests/cypress/integration/wp-cli.spec.js

@@ -115,6 +115,17 @@ describe('WP-CLI Commands', () => {
 				expect(indexPerPostType).to.not.equal(indexTotal);
 			});
 		});
+
+		it('Can index without using dynamic bulk requests if user specifies --static-bulk parameter', () => {
+			cy.activatePlugin('fake-log-messages');
+
+			cy.wpCli('wp elasticpress index --static-bulk')
+				.its('stdout')
+				.should('contain', 'Index command with --static-bulk flag completed')
+				.should('contain', 'Done');
+
+			cy.deactivatePlugin('fake-log-messages');
+		});
 	});
 
 	it('Can delete the index of current blog if user runs wp elasticpress delete-index', () => {
@@ -291,4 +302,68 @@ describe('WP-CLI Commands', () => {
 			checkIfNotMissingIndexes('network');
 		});
 	});
+
+	it('Can set the algorithm version', () => {
+		cy.wpCli('wp elasticpress set-algorithm-version --default')
+			.its('stdout')
+			.should('contain', 'Done');
+
+		cy.wpCli('wp elasticpress get-algorithm-version')
+			.its('stdout')
+			.should('contain', 'default');
+
+		cy.wpCli('wp elasticpress set-algorithm-version --version=1.0.0')
+			.its('stdout')
+			.should('contain', 'Done');
+
+		cy.wpCli('wp elasticpress get-algorithm-version').its('stdout').should('contain', '1.0.0');
+
+		cy.wpCli('wp elasticpress set-algorithm-version', true)
+			.its('stderr')
+			.should('contain', 'This command expects a version number or the --default flag');
+	});
+
+	it('Can get the mapping information', () => {
+		cy.wpCli('wp elasticpress get-mapping').its('stdout').should('contain', 'mapping_version');
+	});
+
+	it('Can get the cluster indexes information', () => {
+		cy.wpCli('wp elasticpress get-cluster-indexes').its('stdout').should('contain', 'health');
+	});
+
+	it('Can get the indexes names', () => {
+		cy.wpCli('wp elasticpress get-indexes').its('code').should('equal', 0);
+
+		cy.wpCli('wp elasticpress get-indexes --pretty').its('stdout').should('contain', '\n');
+	});
+
+	it('Can stop the sync operation and clear it', () => {
+		// if no index is running, this will fail.
+		cy.wpCli('wp elasticpress stop-indexing')
+			.its('stderr')
+			.should('contain', 'There is no indexing operation running');
+
+		// mock the indexing process
+		cy.wpCliEval(
+			`update_option('ep_index_meta', true); set_transient('ep_sync_interrupted', true);`,
+		);
+
+		cy.wpCli('wp elasticpress stop-indexing').its('stdout').should('contain', 'Done');
+
+		cy.wpCli('wp elasticpress clear-index').its('stdout').should('contain', 'Index cleared');
+	});
+
+	it('can send an HTTP request to Elasticsearch', () => {
+		cy.wpCli('wp elasticpress  request _cat/indices').its('code').should('equal', 0);
+
+		// check if it throw an error if non supported method is used?
+		cy.wpCli('wp elasticpress request _cat/indices --method=POST')
+			.its('stdout')
+			.should('contain', 'Incorrect HTTP method for uri');
+
+		// check if it print the debugging info?
+		cy.wpCli('wp elasticpress request _cat/indices --debug-http-request')
+			.its('stdout')
+			.should('contain', '[http_response] => WP_HTTP_Requests_Response Object');
+	});
 });

tests/cypress/wordpress-files/test-plugins/fake-log-messages.php

@@ -0,0 +1,17 @@
+<?php
+
+/**
+ * Plugin Name: Fake Log Messages
+ * Description: Fake log messages for E2E testings
+ * Version:     1.0.0
+ * Author:      10up Inc.
+ * License:     GPLv2 or later
+ */
+
+
+/**
+ * Add log message for index command with --static-bulk flag.
+ */
+add_action( 'ep_after_bulk_index', function() {
+	WP_CLI::log('Index command with --static-bulk flag completed.');
+} );