fix(test): Use valid hex commit hash in test_add_test_entry_db_commit_failure

The test was using 'testcommithash' which fails is_valid_commit_hash()
validation, causing early return before safe_db_commit is called.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>

Author: cfsmp3

tests/test_ci/test_controllers.py

@@ -1463,7 +1463,8 @@ def test_add_test_entry_db_commit_failure(
 
         mock_safe_commit.return_value = False
 
-        add_test_entry(g.db, 'testcommithash', TestType.commit)
+        # Use valid hex commit hash (is_valid_commit_hash validates before db commit)
+        add_test_entry(g.db, 'abcdef1234567890abcdef1234567890abcdef12', TestType.commit)
 
         mock_safe_commit.assert_called_once()
         mock_log.error.assert_called()