Skip to content

Commit

Permalink
fix uninstall e2e test failure
Browse files Browse the repository at this point in the history
  • Loading branch information
diosmosis committed Dec 25, 2024
1 parent c33402c commit c5e50c5
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
5 changes: 5 additions & 0 deletions tests/e2e-uninstall/get-matomo-tables.php
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,11 @@
exit;
}

$multi = isset( $_GET['multi'] ) ? ( (int) $_GET['multi'] ) : 0;
if ( $multi ) {
$dbname = preg_replace( '/-multi$/', '', $dbname );
}

$pdo = new \PDO( "mysql:host=$host;dbname={$dbname}", 'root', 'pass' );
$query = $pdo->prepare( 'SHOW TABLES' );
$query->execute();
Expand Down
5 changes: 1 addition & 4 deletions tests/e2e-uninstall/uninstall.e2e.ts
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,7 @@ describe('MWP Uninstall', () => {
});

async function getTablesInstalled(): Promise<any> {
console.log('root url: ' + `${await Website.rootUrl()}/matomo-for-wordpress/tests/e2e-uninstall/get-matomo-tables.php`);
let result: any = await fetch(`${await Website.rootUrl()}/matomo-for-wordpress/tests/e2e-uninstall/get-matomo-tables.php`);
let result: any = await fetch(`${await Website.rootUrl()}/matomo-for-wordpress/tests/e2e-uninstall/get-matomo-tables.php?multi=0`);
result = await result.text();
try {
result = JSON.parse(result);
Expand Down Expand Up @@ -66,8 +65,6 @@ describe('MWP Uninstall', () => {

expect(result.tables).toBeInstanceOf(Array); // sanity check

console.log(`result.dbname is: ${result.dbname}`);

let tables = result.tables;
tables = tables.filter((t: string) => t.includes('matomo'));
expect(tables).toEqual([]); // check there are no tables with "matomo" in the name
Expand Down

0 comments on commit c5e50c5

Please sign in to comment.