chore: remove comments not useful outside the context of this PR

They refer to lines in files that have been deleted, and provide no
value to future maintainers.

Author: auxesis

tests/sqlx/fixtures/config_tables.sql

@@ -1,5 +1,4 @@
 -- Fixture for config tests
--- Converted from src/config/config_test.sql lines 4-19
 
 DROP TABLE IF EXISTS users CASCADE;
 CREATE TABLE users (

tests/sqlx/fixtures/encryptindex_tables.sql

@@ -1,5 +1,4 @@
 -- Fixture for encryptindex tests
--- Converted from src/encryptindex/functions_test.sql lines 10-17
 -- Referenced by: tests/sqlx/tests/encryptindex_tests.rs
 --
 -- Creates a users table with plaintext columns for testing encrypted column

tests/sqlx/tests/aggregate_tests.rs

@@ -1,6 +1,5 @@
 //! Aggregate function tests
 //!
-//! Converted from src/encrypted/aggregates_test.sql
 //! Tests COUNT, MAX, MIN with encrypted data
 
 use anyhow::Result;
@@ -9,7 +8,6 @@ use sqlx::PgPool;
 #[sqlx::test]
 async fn count_aggregate_on_encrypted_column(pool: PgPool) -> Result<()> {
     // Test: COUNT works with encrypted columns
-    // Original SQL lines 13-19 in src/encrypted/aggregates_test.sql
 
     let count: i64 = sqlx::query_scalar("SELECT COUNT(*) FROM ore")
         .fetch_one(&pool)
@@ -23,7 +21,6 @@ async fn count_aggregate_on_encrypted_column(pool: PgPool) -> Result<()> {
 #[sqlx::test]
 async fn max_aggregate_on_encrypted_column(pool: PgPool) -> Result<()> {
     // Test: MAX returns highest value with ORE
-    // Original SQL lines 21-32 in src/encrypted/aggregates_test.sql
 
     let max_id: i64 = sqlx::query_scalar("SELECT MAX(id) FROM ore WHERE id <= 50")
         .fetch_one(&pool)
@@ -37,7 +34,6 @@ async fn max_aggregate_on_encrypted_column(pool: PgPool) -> Result<()> {
 #[sqlx::test]
 async fn min_aggregate_on_encrypted_column(pool: PgPool) -> Result<()> {
     // Test: MIN returns lowest value with ORE
-    // Original SQL lines 34-45 in src/encrypted/aggregates_test.sql
 
     let min_id: i64 = sqlx::query_scalar("SELECT MIN(id) FROM ore WHERE id >= 10")
         .fetch_one(&pool)
@@ -51,7 +47,6 @@ async fn min_aggregate_on_encrypted_column(pool: PgPool) -> Result<()> {
 #[sqlx::test(fixtures(path = "../fixtures", scripts("encrypted_json")))]
 async fn group_by_with_encrypted_column(pool: PgPool) -> Result<()> {
     // Test: GROUP BY works with encrypted data
-    // Original SQL lines 47-50 in src/encrypted/aggregates_test.sql
     // Fixture creates 3 distinct encrypted records, each unique
 
     let group_count: i64 = sqlx::query_scalar(

tests/sqlx/tests/comparison_tests.rs

@@ -1,6 +1,5 @@
 //! Comparison operator tests (< > <= >=)
 //!
-//! Converted from src/operators/<_test.sql, >_test.sql, <=_test.sql, >=_test.sql
 //! Tests EQL comparison operators with ORE (Order-Revealing Encryption)
 
 use anyhow::{Context, Result};
@@ -47,7 +46,6 @@ async fn create_encrypted_json_with_index(
 async fn less_than_operator_with_ore(pool: PgPool) -> Result<()> {
     // Test: e < e with ORE encryption
     // Value 42 should have 41 records less than it (1-41)
-    // Original SQL lines 13-20 in src/operators/<_test.sql
     // Uses ore table from migrations/002_install_ore_data.sql (ids 1-99)
 
     // Get encrypted value for id=42 from pre-seeded ore table
@@ -67,7 +65,6 @@ async fn less_than_operator_with_ore(pool: PgPool) -> Result<()> {
 #[sqlx::test]
 async fn lt_function_with_ore(pool: PgPool) -> Result<()> {
     // Test: eql_v2.lt() function with ORE
-    // Original SQL lines 30-37 in src/operators/<_test.sql
 
     let ore_term = get_ore_encrypted(&pool, 42).await?;
 
@@ -120,7 +117,6 @@ async fn less_than_operator_jsonb_less_than_encrypted(pool: PgPool) -> Result<()
 async fn greater_than_operator_with_ore(pool: PgPool) -> Result<()> {
     // Test: e > e with ORE encryption
     // Value 42 should have 57 records greater than it (43-99)
-    // Original SQL lines 13-20 in src/operators/>_test.sql
     // Uses ore table from migrations/002_install_ore_data.sql (ids 1-99)
 
     let ore_term = get_ore_encrypted(&pool, 42).await?;
@@ -138,7 +134,6 @@ async fn greater_than_operator_with_ore(pool: PgPool) -> Result<()> {
 #[sqlx::test]
 async fn gt_function_with_ore(pool: PgPool) -> Result<()> {
     // Test: eql_v2.gt() function with ORE
-    // Original SQL lines 30-37 in src/operators/>_test.sql
 
     let ore_term = get_ore_encrypted(&pool, 42).await?;
 
@@ -190,7 +185,6 @@ async fn greater_than_operator_jsonb_greater_than_encrypted(pool: PgPool) -> Res
 async fn less_than_or_equal_operator_with_ore(pool: PgPool) -> Result<()> {
     // Test: e <= e with ORE encryption
     // Value 42 should have 42 records <= it (1-42 inclusive)
-    // Original SQL lines 10-24 in src/operators/<=_test.sql
     // Uses ore table from migrations/002_install_ore_data.sql (ids 1-99)
 
     let ore_term = get_ore_encrypted(&pool, 42).await?;
@@ -209,7 +203,6 @@ async fn less_than_or_equal_operator_with_ore(pool: PgPool) -> Result<()> {
 #[sqlx::test]
 async fn lte_function_with_ore(pool: PgPool) -> Result<()> {
     // Test: eql_v2.lte() function with ORE
-    // Original SQL lines 32-46 in src/operators/<=_test.sql
 
     let ore_term = get_ore_encrypted(&pool, 42).await?;
 
@@ -226,7 +219,6 @@ async fn lte_function_with_ore(pool: PgPool) -> Result<()> {
 #[sqlx::test]
 async fn less_than_or_equal_with_jsonb(pool: PgPool) -> Result<()> {
     // Test: e <= jsonb with ORE
-    // Original SQL lines 55-69 in src/operators/<=_test.sql
 
     let json_value = get_ore_encrypted_as_jsonb(&pool, 42).await?;
 
@@ -260,7 +252,6 @@ async fn less_than_or_equal_jsonb_lte_encrypted(pool: PgPool) -> Result<()> {
 async fn greater_than_or_equal_operator_with_ore(pool: PgPool) -> Result<()> {
     // Test: e >= e with ORE encryption
     // Value 42 should have 58 records >= it (42-99 inclusive)
-    // Original SQL lines 10-24 in src/operators/>=_test.sql
     // Uses ore table from migrations/002_install_ore_data.sql (ids 1-99)
 
     let ore_term = get_ore_encrypted(&pool, 42).await?;
@@ -278,7 +269,6 @@ async fn greater_than_or_equal_operator_with_ore(pool: PgPool) -> Result<()> {
 #[sqlx::test]
 async fn gte_function_with_ore(pool: PgPool) -> Result<()> {
     // Test: eql_v2.gte() function with ORE
-    // Original SQL lines 32-46 in src/operators/>=_test.sql
 
     let ore_term = get_ore_encrypted(&pool, 42).await?;
 
@@ -295,7 +285,6 @@ async fn gte_function_with_ore(pool: PgPool) -> Result<()> {
 #[sqlx::test]
 async fn greater_than_or_equal_with_jsonb(pool: PgPool) -> Result<()> {
     // Test: e >= jsonb with ORE
-    // Original SQL lines 55-85 in src/operators/>=_test.sql
 
     let json_value = get_ore_encrypted_as_jsonb(&pool, 42).await?;
 
@@ -309,7 +298,6 @@ async fn greater_than_or_equal_with_jsonb(pool: PgPool) -> Result<()> {
 #[sqlx::test]
 async fn greater_than_or_equal_jsonb_gte_encrypted(pool: PgPool) -> Result<()> {
     // Test: jsonb >= e with ORE (reverse direction)
-    // Original SQL lines 77-80 in src/operators/>=_test.sql
 
     let json_value = get_ore_encrypted_as_jsonb(&pool, 42).await?;
 

tests/sqlx/tests/config_tests.rs

@@ -1,6 +1,5 @@
 //! Configuration management tests
 //!
-//! Converted from src/config/config_test.sql
 //! Tests EQL configuration add/remove operations and state management
 
 use anyhow::{Context, Result};
@@ -36,7 +35,6 @@ async fn search_config_exists(
 #[sqlx::test(fixtures(path = "../fixtures", scripts("config_tables")))]
 async fn add_and_remove_multiple_indexes(pool: PgPool) -> Result<()> {
     // Test: Add and remove multiple indexes (6 assertions)
-    // Original SQL lines 42-67 in src/config/config_test.sql
 
     // Truncate config
     sqlx::query("TRUNCATE TABLE eql_v2_configuration")
@@ -108,7 +106,6 @@ async fn add_and_remove_multiple_indexes(pool: PgPool) -> Result<()> {
 #[sqlx::test(fixtures(path = "../fixtures", scripts("config_tables")))]
 async fn add_and_remove_indexes_from_multiple_tables(pool: PgPool) -> Result<()> {
     // Test: Add/remove indexes from multiple tables (9 assertions)
-    // Original SQL lines 78-116 in src/config/config_test.sql
 
     sqlx::query("TRUNCATE TABLE eql_v2_configuration")
         .execute(&pool)
@@ -210,7 +207,6 @@ async fn add_and_remove_indexes_from_multiple_tables(pool: PgPool) -> Result<()>
 #[sqlx::test(fixtures(path = "../fixtures", scripts("config_tables")))]
 async fn add_and_modify_index(pool: PgPool) -> Result<()> {
     // Test: Add and modify index (6 assertions)
-    // Original SQL lines 128-150 in src/config/config_test.sql
 
     // Add match index
     sqlx::query("SELECT eql_v2.add_search_config('users', 'name', 'match', migrating => true)")
@@ -290,7 +286,6 @@ async fn add_and_modify_index(pool: PgPool) -> Result<()> {
 #[sqlx::test(fixtures(path = "../fixtures", scripts("config_tables")))]
 async fn add_index_with_existing_active_config(pool: PgPool) -> Result<()> {
     // Test: Adding index creates new pending configuration when active config exists (3 assertions)
-    // Original SQL lines 157-196 in src/config/config_test.sql
 
     sqlx::query("TRUNCATE TABLE eql_v2_configuration")
         .execute(&pool)
@@ -351,7 +346,6 @@ async fn add_index_with_existing_active_config(pool: PgPool) -> Result<()> {
 #[sqlx::test(fixtures(path = "../fixtures", scripts("config_tables")))]
 async fn add_column_to_nonexistent_table_fails(pool: PgPool) -> Result<()> {
     // Test: Adding column to nonexistent table fails (2 assertions)
-    // Original SQL lines 204-215 in src/config/config_test.sql
 
     sqlx::query("TRUNCATE TABLE eql_v2_configuration")
         .execute(&pool)
@@ -382,7 +376,6 @@ async fn add_column_to_nonexistent_table_fails(pool: PgPool) -> Result<()> {
 #[sqlx::test(fixtures(path = "../fixtures", scripts("encrypted_json")))]
 async fn add_and_remove_column(pool: PgPool) -> Result<()> {
     // Test: Add and remove column (4 assertions)
-    // Original SQL lines 223-248 in src/config/config_test.sql
 
     sqlx::query("TRUNCATE TABLE eql_v2_configuration")
         .execute(&pool)
@@ -433,7 +426,6 @@ async fn add_and_remove_column(pool: PgPool) -> Result<()> {
 #[sqlx::test(fixtures(path = "../fixtures", scripts("config_tables")))]
 async fn configuration_constraint_validation(pool: PgPool) -> Result<()> {
     // Test: Configuration constraint validation (11 assertions)
-    // Original SQL lines 259-334 in src/config/config_test.sql
 
     sqlx::query("TRUNCATE TABLE eql_v2_configuration")
         .execute(&pool)

tests/sqlx/tests/constraint_tests.rs

@@ -1,6 +1,5 @@
 //! Constraint tests
 //!
-//! Converted from src/encrypted/constraints_test.sql
 //! Tests UNIQUE, NOT NULL, CHECK constraints on encrypted columns
 
 use anyhow::Result;
@@ -9,7 +8,6 @@ use sqlx::PgPool;
 #[sqlx::test(fixtures(path = "../fixtures", scripts("constraint_tables")))]
 async fn unique_constraint_on_encrypted_column(pool: PgPool) -> Result<()> {
     // Test: UNIQUE constraint enforced on encrypted column (3 assertions)
-    // Original SQL lines 13-35 in src/encrypted/constraints_test.sql
 
     // Insert first record (provide check_field to satisfy its constraint)
     sqlx::query(
@@ -52,7 +50,6 @@ async fn unique_constraint_on_encrypted_column(pool: PgPool) -> Result<()> {
 #[sqlx::test(fixtures(path = "../fixtures", scripts("constraint_tables")))]
 async fn not_null_constraint_on_encrypted_column(pool: PgPool) -> Result<()> {
     // Test: NOT NULL constraint enforced (2 assertions)
-    // Original SQL lines 37-52 in src/encrypted/constraints_test.sql
 
     let result = sqlx::query(
         "INSERT INTO constrained (unique_field)
@@ -76,7 +73,6 @@ async fn not_null_constraint_on_encrypted_column(pool: PgPool) -> Result<()> {
 #[sqlx::test(fixtures(path = "../fixtures", scripts("constraint_tables")))]
 async fn check_constraint_on_encrypted_column(pool: PgPool) -> Result<()> {
     // Test: CHECK constraint enforced (2 assertions)
-    // Original SQL lines 54-72 in src/encrypted/constraints_test.sql
 
     let result = sqlx::query(
         "INSERT INTO constrained (unique_field, not_null_field, check_field)
@@ -105,7 +101,6 @@ async fn check_constraint_on_encrypted_column(pool: PgPool) -> Result<()> {
 async fn foreign_key_constraint_with_encrypted(pool: PgPool) -> Result<()> {
     // Test: Foreign key constraints can be defined on encrypted columns
     // but don't provide referential integrity since each encryption is unique
-    // Original SQL lines 74-139 in src/encrypted/constraints_test.sql
 
     // Create parent table
     sqlx::query(

tests/sqlx/tests/containment_tests.rs

@@ -1,6 +1,5 @@
 //! Containment operator tests (@> and <@)
 //!
-//! Converted from src/operators/@>_test.sql and <@_test.sql
 //! Tests encrypted JSONB containment operations
 
 use anyhow::Result;
@@ -14,7 +13,6 @@ use sqlx::PgPool;
 #[sqlx::test(fixtures(path = "../fixtures", scripts("encrypted_json")))]
 async fn contains_operator_self_containment(pool: PgPool) -> Result<()> {
     // Test: encrypted value contains itself
-    // Original SQL lines 13-25 in src/operators/@>_test.sql
     // Tests that a @> b when a == b
 
     let sql = "SELECT e FROM encrypted WHERE e @> e LIMIT 1";
@@ -27,7 +25,6 @@ async fn contains_operator_self_containment(pool: PgPool) -> Result<()> {
 #[sqlx::test(fixtures(path = "../fixtures", scripts("encrypted_json")))]
 async fn contains_operator_with_extracted_term(pool: PgPool) -> Result<()> {
     // Test: e @> term where term is extracted from encrypted value
-    // Original SQL lines 34-51 in src/operators/@>_test.sql
     // Tests containment with extracted field ($.n selector)
 
     let sql = format!(
@@ -43,7 +40,6 @@ async fn contains_operator_with_extracted_term(pool: PgPool) -> Result<()> {
 #[sqlx::test(fixtures(path = "../fixtures", scripts("encrypted_json")))]
 async fn contains_operator_term_does_not_contain_full_value(pool: PgPool) -> Result<()> {
     // Test: term does NOT contain full encrypted value (asymmetric containment)
-    // Original SQL lines 48-49 in src/operators/@>_test.sql
     // Verifies that while e @> term is true, term @> e is false
 
     let sql = format!(
@@ -60,7 +56,6 @@ async fn contains_operator_term_does_not_contain_full_value(pool: PgPool) -> Res
 #[sqlx::test(fixtures(path = "../fixtures", scripts("encrypted_json")))]
 async fn contains_operator_with_encrypted_term(pool: PgPool) -> Result<()> {
     // Test: e @> encrypted_term with encrypted selector
-    // Original SQL lines 68-90 in src/operators/@>_test.sql
     // Uses encrypted test data with $.hello selector
 
     let term = get_encrypted_term(&pool, Selectors::HELLO).await?;
@@ -79,7 +74,6 @@ async fn contains_operator_with_encrypted_term(pool: PgPool) -> Result<()> {
 #[sqlx::test(fixtures(path = "../fixtures", scripts("encrypted_json")))]
 async fn contains_operator_count_matches(pool: PgPool) -> Result<()> {
     // Test: e @> term returns correct count
-    // Original SQL lines 84-87 in src/operators/@>_test.sql
     // Verifies count of records containing the term
 
     let term = get_encrypted_term(&pool, Selectors::HELLO).await?;
@@ -99,7 +93,6 @@ async fn contains_operator_count_matches(pool: PgPool) -> Result<()> {
 #[sqlx::test(fixtures(path = "../fixtures", scripts("encrypted_json")))]
 async fn contained_by_operator_with_encrypted_term(pool: PgPool) -> Result<()> {
     // Test: term <@ e (contained by)
-    // Original SQL lines 19-41 in src/operators/<@_test.sql
     // Tests that extracted term is contained by the original encrypted value
 
     let term = get_encrypted_term(&pool, Selectors::HELLO).await?;
@@ -118,7 +111,6 @@ async fn contained_by_operator_with_encrypted_term(pool: PgPool) -> Result<()> {
 #[sqlx::test(fixtures(path = "../fixtures", scripts("encrypted_json")))]
 async fn contained_by_operator_count_matches(pool: PgPool) -> Result<()> {
     // Test: term <@ e returns correct count
-    // Original SQL lines 35-38 in src/operators/<@_test.sql
     // Verifies count of records containing the term
 
     let term = get_encrypted_term(&pool, Selectors::HELLO).await?;