test: Updated Test Cases

Author: usernane

tests/bootstrap.php

@@ -12,15 +12,18 @@
 use webfiori\framework\App;
 use webfiori\framework\autoload\ClassLoader;
 use webfiori\framework\config\JsonDriver;
+use webfiori\framework\ThemeManager;
+use themes\fioriTheme\NewFTestTheme;
+use themes\fioriTheme2\NewTestTheme2;
 
 $DS = DIRECTORY_SEPARATOR;
 
 //the name of tests directory. Update as needed.
 define('TESTS_DIRECTORY', 'tests');
-define('SQL_SERVER_HOST', 'localhost\SQLEXPRESS');
-define('SQL_SERVER_USER', 'sa');
-define('SQL_SERVER_PASS', '1234567890@Eu');
-define('SQL_SERVER_DB', 'testing_db');
+define('SQL_SERVER_HOST', getenv('SQL_SERVER_HOST') ?: 'localhost');
+define('SQL_SERVER_USER', getenv('SQL_SERVER_USER') ?: 'sa');
+define('SQL_SERVER_PASS', getenv('SA_SQL_SERVER_PASSWORD') ?: '1234567890@Eu');
+define('SQL_SERVER_DB', getenv('SQL_SERVER_DB') ?: 'testing_db');
 define('ODBC_VERSION', 17);
 //an array that contains possible locations at which
 //WebFiori Framework might exist.
@@ -112,10 +115,10 @@
     $conn = new ConnectionInfo('mssql',SQL_SERVER_USER, SQL_SERVER_PASS, SQL_SERVER_DB, SQL_SERVER_HOST, 1433, [
         'TrustServerCertificate' => 'true'
     ]);
-    $runner = new SchemaRunner(APP_PATH, '', $conn);
+    
     try {
-        $runner->dropChangesTable();
-    } catch (\Exception $exc) {
+      //  $runner = new SchemaRunner($conn);
+      //  $runner->dropChangesTable();
 
     } catch (\Throwable $exc) {
         fprintf(STDOUT,'Error on register_shutdown_function:'."\n\n");
@@ -125,5 +128,10 @@
 });
 fprintf(STDOUT, "Registering shutdown function completed.\n");
 fprintf(STDOUT,"---------------------------------\n");
+fprintf(STDOUT,"Adding themes...\n");
+ThemeManager::register(new NewFTestTheme());
+ThemeManager::register(new NewTestTheme2());
+fprintf(STDOUT,"Done\n");
+fprintf(STDOUT,"---------------------------------\n");
 fprintf(STDOUT,"Starting to run tests...\n");
 fprintf(STDOUT,"---------------------------------\n");

tests/webfiori/framework/test/cli/AddCommandTest.php

@@ -83,7 +83,7 @@ public function testAddDBConnection01() {
             '127.0.0.1',
             "\n", // Hit Enter to pick default value (port 3306)
             'root',
-            '12345326',
+            getenv('MYSQL_ROOT_PASSWORD') ?: '12345326',
             'testing_db',
             "\n", // Hit Enter to pick default value (connection name)
             'y'
@@ -108,7 +108,7 @@ public function testAddDBConnection01() {
             "Trying to connect to the database...\n",
             "Trying with 'localhost'...\n",
             "Error: Unable to connect to the database.\n",
-            "Error: Unable to connect to database: 1045 - Access denied for user 'root'@'localhost' (using password: YES)\n",
+            "Error: Unable to connect to database: 1045 - Access denied for user 'root'@'127.0.0.1' (using password: YES)\n",
             "Would you like to store connection information anyway?(y/N)\n",
             "Success: Connection information was stored in application configuration.\n"
         ], $output);
@@ -129,7 +129,7 @@ public function testAddDBConnection02() {
             '127.0.0.1',
             "\n", // Hit Enter to pick default value (port 3306)
             'root',
-            '12345326',
+            getenv('MYSQL_ROOT_PASSWORD') ?: '12345326',
             'testing_db',
             "\n", // Hit Enter to pick default value (connection name)
             'n'
@@ -154,7 +154,7 @@ public function testAddDBConnection02() {
             "Trying to connect to the database...\n",
             "Trying with 'localhost'...\n",
             "Error: Unable to connect to the database.\n",
-            "Error: Unable to connect to database: 1045 - Access denied for user 'root'@'localhost' (using password: YES)\n",
+            "Error: Unable to connect to database: 1045 - Access denied for user 'root'@'127.0.0.1' (using password: YES)\n",
             "Would you like to store connection information anyway?(y/N)\n",
         ], $output);
     }
@@ -258,7 +258,7 @@ public function testAddSMTPConnection00() {
             '127.0.0.1',
             "\n", // Hit Enter to pick default value (port 25)
             'test@example.com',
-            '12345326',
+            getenv('MYSQL_ROOT_PASSWORD') ?: '12345326',
             'test@example.com',
             'test@example.com',
             "\n", // Hit Enter to pick default value (connection name)

tests/webfiori/framework/test/cli/CreateMigrationTest.php

@@ -10,15 +10,17 @@
 class CreateMigrationTest extends CLITestCase {
 
     protected function tearDown(): void {
-        // Clean up all migration files after each test
+        // Clean up only specific migration files created by this test
         $migrationsDir = APP_PATH . DS . 'database' . DS . 'migrations';
         if (is_dir($migrationsDir)) {
-            $files = glob($migrationsDir . DS . 'Migration*.php');
+            // Only remove Migration files directly in the migrations directory
+            $files = glob($migrationsDir . DS . 'Migration[0-9][0-9][0-9].php');
             foreach ($files as $file) {
                 if (is_file($file)) {
                     unlink($file);
                 }
             }
+            // Remove Cool* files created by this test
             $files = glob($migrationsDir . DS . 'Cool*.php');
             foreach ($files as $file) {
                 if (is_file($file)) {

tests/webfiori/framework/test/writers/DatabaseMigrationWriterTest.php

@@ -13,10 +13,11 @@
 class DatabaseMigrationWriterTest extends TestCase {
 
     protected function tearDown(): void {
-        // Clean up all migration files after each test
+        // Clean up only the Migration files created by this test (Migration000, Migration001, etc.)
         $migrationsDir = APP_PATH . DS . 'database' . DS . 'migrations';
         if (is_dir($migrationsDir)) {
-            $files = glob($migrationsDir . DS . 'Migration*.php');
+            // Only remove Migration files directly in the migrations directory, not in subdirectories
+            $files = glob($migrationsDir . DS . 'Migration[0-9][0-9][0-9].php');
             foreach ($files as $file) {
                 if (is_file($file)) {
                     unlink($file);