diff --git a/src/Common/Library.fs b/src/Common/Library.fs index 471ac37..c50f815 100644 --- a/src/Common/Library.fs +++ b/src/Common/Library.fs @@ -5,7 +5,7 @@ open System.Security.Cryptography /// The types of comparisons available for JSON fields type Comparison = /// Equals (=) - | Equals of obj + | Equal of obj /// Greater Than (>) | Greater of obj /// Greater Than or Equal To (>=) @@ -20,7 +20,7 @@ type Comparison = | Between of obj * obj /// In (IN) | In of obj seq - // Array Contains/Exists (PostgreSQL: |? SQLite: EXISTS / json_each / IN) + // Array Contains (PostgreSQL: |? SQLite: EXISTS / json_each / IN) | Contains of obj /// Exists (IS NOT NULL) | Exists @@ -30,7 +30,7 @@ type Comparison = /// Get the operator SQL for this comparison member this.OpSql = match this with - | Equals _ -> "=" + | Equal _ -> "=" | Greater _ -> ">" | GreaterOrEqual _ -> ">=" | Less _ -> "<" @@ -50,68 +50,94 @@ type Dialect = | SQLite /// Criteria for a field WHERE clause -type Field = { - /// The name of the field - Name: string +type Field = + { /// The name of the field + Name: string - Comparison: Comparison - // /// The operation by which the field will be compared - // Op: Op - // - // /// The value of the field - // Value: obj + /// The comparison for the field + Comparison: Comparison - /// The name of the parameter for this field - ParameterName: string option + /// The name of the parameter for this field + ParameterName: string option - /// The table qualifier for this field - Qualifier: string option -} with + /// The table qualifier for this field + Qualifier: string option } +with /// Create a comparison against a field static member Where name comparison = { Name = name; Comparison = comparison; ParameterName = None; Qualifier = None } /// Create an equals (=) field criterion - static member EQ name (value: obj) = - Field.Where name (Equals value) + static member Equal name (value: obj) = + Field.Where name (Equal value) + + /// Create an equals (=) field criterion (alias) + static member EQ name (value: obj) = Field.Equal name value /// Create a greater than (>) field criterion - static member GT name (value: obj) = + static member Greater name (value: obj) = Field.Where name (Greater value) + /// Create a greater than (>) field criterion (alias) + static member GT name (value: obj) = Field.Greater name value + /// Create a greater than or equal to (>=) field criterion - static member GE name (value: obj) = + static member GreaterOrEqual name (value: obj) = Field.Where name (GreaterOrEqual value) + /// Create a greater than or equal to (>=) field criterion (alias) + static member GE name (value: obj) = Field.GreaterOrEqual name value + /// Create a less than (<) field criterion - static member LT name (value: obj) = + static member Less name (value: obj) = Field.Where name (Less value) + /// Create a less than (<) field criterion (alias) + static member LT name (value: obj) = Field.Less name value + /// Create a less than or equal to (<=) field criterion - static member LE name (value: obj) = + static member LessOrEqual name (value: obj) = Field.Where name (LessOrEqual value) + /// Create a less than or equal to (<=) field criterion (alias) + static member LE name (value: obj) = Field.LessOrEqual name value + /// Create a not equals (<>) field criterion - static member NE name (value: obj) = + static member NotEqual name (value: obj) = Field.Where name (NotEqual value) + /// Create a not equals (<>) field criterion (alias) + static member NE name (value: obj) = Field.NotEqual name value + /// Create a BETWEEN field criterion - static member BT name (min: obj) (max: obj) = + static member Between name (min: obj) (max: obj) = Field.Where name (Between(min, max)) + /// Create a BETWEEN field criterion (alias) + static member BT name (min: obj) (max: obj) = Field.Between name min max + /// Create an IN field criterion - static member IN name (values: obj seq) = + static member In name (values: obj seq) = Field.Where name (In values) + /// Create an IN field criterion (alias) + static member IN name (values: obj seq) = Field.In name values + /// Create an exists (IS NOT NULL) field criterion - static member EX name = + static member Exists name = Field.Where name Exists + /// Create an exists (IS NOT NULL) field criterion (alias) + static member EX name = Field.Exists name + /// Create a not exists (IS NULL) field criterion - static member NEX name = + static member NotExists name = Field.Where name NotExists + /// Create a not exists (IS NULL) field criterion (alias) + static member NEX name = Field.NotExists name + /// Transform a field name (a.b.c) to a path for the given SQL dialect static member NameToPath (name: string) dialect = let path = @@ -124,7 +150,7 @@ type Field = { /// Create a field with a given name, but no other properties filled (op will be EQ, value will be "") static member Named name = - Field.Where name (Equals "") + Field.Where name (Equal "") /// Specify the name of the parameter for this field member this.WithParameterName name = diff --git a/src/Postgres/Library.fs b/src/Postgres/Library.fs index 3dd90ac..0cfb7b8 100644 --- a/src/Postgres/Library.fs +++ b/src/Postgres/Library.fs @@ -101,7 +101,7 @@ module Parameters = let paramName = $"{p}_{idx}" paramName, Sql.parameter (NpgsqlParameter(paramName, v))) | Contains _ -> () // TODO - | Equals v | Greater v | GreaterOrEqual v | Less v | LessOrEqual v | NotEqual v -> + | Equal v | Greater v | GreaterOrEqual v | Less v | LessOrEqual v | NotEqual v -> let p = name.Derive it.ParameterName yield (p, parameterFor v (fun l -> Sql.parameter (NpgsqlParameter(p, l)))) }) |> Seq.collect id @@ -147,7 +147,7 @@ module Query = let paramNames = values |> Seq.mapi (fun idx _ -> $"{p}_{idx}") |> String.concat ", " $"({paramNames})", defaultArg (Seq.tryHead values) (obj ()) | Contains _ -> p, "" // TODO: may need to use -> vs ->> in field SQL - | Equals v | Greater v | GreaterOrEqual v | Less v | LessOrEqual v | NotEqual v -> p, v + | Equal v | Greater v | GreaterOrEqual v | Less v | LessOrEqual v | NotEqual v -> p, v | _ -> p, "" if isNumeric value then $"({it.Path PostgreSQL})::numeric {it.Comparison.OpSql} {param}" @@ -157,7 +157,7 @@ module Query = /// Create a WHERE clause fragment to implement an ID-based query [] let whereById<'TKey> (docId: 'TKey) = - whereByFields Any [ { Field.EQ (Configuration.idField ()) docId with ParameterName = Some "@id" } ] + whereByFields Any [ { Field.Equal (Configuration.idField ()) docId with ParameterName = Some "@id" } ] /// Table and index definition queries module Definition = diff --git a/src/Sqlite/Library.fs b/src/Sqlite/Library.fs index a965110..69f4e8a 100644 --- a/src/Sqlite/Library.fs +++ b/src/Sqlite/Library.fs @@ -53,7 +53,7 @@ module Query = /// Create a WHERE clause fragment to implement an ID-based query [] let whereById paramName = - whereByFields Any [ { Field.EQ (Configuration.idField ()) 0 with ParameterName = Some paramName } ] + whereByFields Any [ { Field.Equal (Configuration.idField ()) 0 with ParameterName = Some paramName } ] /// Create an UPDATE statement to patch documents [] @@ -71,7 +71,7 @@ module Query = let byId<'TKey> statement (docId: 'TKey) = Query.statementWhere statement - (whereByFields Any [ { Field.EQ (Configuration.idField ()) docId with ParameterName = Some "@id" } ]) + (whereByFields Any [ { Field.Equal (Configuration.idField ()) docId with ParameterName = Some "@id" } ]) /// Create a query on JSON fields [] @@ -117,7 +117,7 @@ module Parameters = let p = name.Derive it.ParameterName yield! values |> Seq.mapi (fun idx v -> SqliteParameter($"{p}_{idx}", v)) | Contains _ -> () // TODO - | Equals v | Greater v | GreaterOrEqual v | Less v | LessOrEqual v | NotEqual v -> + | Equal v | Greater v | GreaterOrEqual v | Less v | LessOrEqual v | NotEqual v -> yield SqliteParameter(name.Derive it.ParameterName, v) }) |> Seq.collect id |> Seq.append parameters diff --git a/src/Tests.CSharp/CommonCSharpTests.cs b/src/Tests.CSharp/CommonCSharpTests.cs index 3a68f85..75b6400 100644 --- a/src/Tests.CSharp/CommonCSharpTests.cs +++ b/src/Tests.CSharp/CommonCSharpTests.cs @@ -25,9 +25,9 @@ public static class CommonCSharpTests /// private static readonly Test OpTests = TestList("Comparison.OpSql", [ - TestCase("Equals succeeds", () => + TestCase("Equal succeeds", () => { - Expect.equal(Comparison.NewEquals("").OpSql, "=", "The Equals SQL was not correct"); + Expect.equal(Comparison.NewEqual("").OpSql, "=", "The Equals SQL was not correct"); }), TestCase("Greater succeeds", () => { @@ -76,64 +76,64 @@ public static class CommonCSharpTests /// private static readonly Test FieldTests = TestList("Field", [ - TestCase("EQ succeeds", () => + TestCase("Equal succeeds", () => { - var field = Field.EQ("Test", 14); + var field = Field.Equal("Test", 14); Expect.equal(field.Name, "Test", "Field name incorrect"); - Expect.equal(field.Comparison, Comparison.NewEquals(14), "Comparison incorrect"); + Expect.equal(field.Comparison, Comparison.NewEqual(14), "Comparison incorrect"); }), - TestCase("GT succeeds", () => + TestCase("Greater succeeds", () => { - var field = Field.GT("Great", "night"); + var field = Field.Greater("Great", "night"); Expect.equal(field.Name, "Great", "Field name incorrect"); Expect.equal(field.Comparison, Comparison.NewGreater("night"), "Comparison incorrect"); }), - TestCase("GE succeeds", () => + TestCase("GreaterOrEqual succeeds", () => { - var field = Field.GE("Nice", 88L); + var field = Field.GreaterOrEqual("Nice", 88L); Expect.equal(field.Name, "Nice", "Field name incorrect"); Expect.equal(field.Comparison, Comparison.NewGreaterOrEqual(88L), "Comparison incorrect"); }), - TestCase("LT succeeds", () => + TestCase("Less succeeds", () => { - var field = Field.LT("Lesser", "seven"); + var field = Field.Less("Lesser", "seven"); Expect.equal(field.Name, "Lesser", "Field name incorrect"); Expect.equal(field.Comparison, Comparison.NewLess("seven"), "Comparison incorrect"); }), - TestCase("LE succeeds", () => + TestCase("LessOrEqual succeeds", () => { - var field = Field.LE("Nobody", "KNOWS"); + var field = Field.LessOrEqual("Nobody", "KNOWS"); Expect.equal(field.Name, "Nobody", "Field name incorrect"); Expect.equal(field.Comparison, Comparison.NewLessOrEqual("KNOWS"), "Comparison incorrect"); }), - TestCase("NE succeeds", () => + TestCase("NotEqual succeeds", () => { - var field = Field.NE("Park", "here"); + var field = Field.NotEqual("Park", "here"); Expect.equal(field.Name, "Park", "Field name incorrect"); Expect.equal(field.Comparison, Comparison.NewNotEqual("here"), "Comparison incorrect"); }), - TestCase("BT succeeds", () => + TestCase("Between succeeds", () => { - var field = Field.BT("Age", 18, 49); + var field = Field.Between("Age", 18, 49); Expect.equal(field.Name, "Age", "Field name incorrect"); Expect.equal(field.Comparison, Comparison.NewBetween(18, 49), "Comparison incorrect"); }), - TestCase("IN succeeds", () => + TestCase("In succeeds", () => { - var field = Field.IN("Here", [8, 16, 32]); + var field = Field.In("Here", [8, 16, 32]); Expect.equal(field.Name, "Here", "Field name incorrect"); Expect.isTrue(field.Comparison.IsIn, "Comparison incorrect"); Expect.sequenceEqual(((Comparison.In)field.Comparison).Item, [8, 16, 32], "Value incorrect"); }), - TestCase("EX succeeds", () => + TestCase("Exists succeeds", () => { - var field = Field.EX("Groovy"); + var field = Field.Exists("Groovy"); Expect.equal(field.Name, "Groovy", "Field name incorrect"); Expect.isTrue(field.Comparison.IsExists, "Comparison incorrect"); }), - TestCase("NEX succeeds", () => + TestCase("NotExists succeeds", () => { - var field = Field.NEX("Rad"); + var field = Field.NotExists("Rad"); Expect.equal(field.Name, "Rad", "Field name incorrect"); Expect.isTrue(field.Comparison.IsNotExists, "Comparison incorrect"); }), @@ -164,13 +164,13 @@ public static class CommonCSharpTests ]), TestCase("WithParameterName succeeds", () => { - var field = Field.EQ("Bob", "Tom").WithParameterName("@name"); + var field = Field.Equal("Bob", "Tom").WithParameterName("@name"); Expect.isSome(field.ParameterName, "The parameter name should have been filled"); Expect.equal("@name", field.ParameterName.Value, "The parameter name is incorrect"); }), TestCase("WithQualifier succeeds", () => { - var field = Field.EQ("Bill", "Matt").WithQualifier("joe"); + var field = Field.Equal("Bill", "Matt").WithQualifier("joe"); Expect.isSome(field.Qualifier, "The table qualifier should have been filled"); Expect.equal("joe", field.Qualifier.Value, "The table qualifier is incorrect"); }), @@ -178,47 +178,47 @@ public static class CommonCSharpTests [ TestCase("succeeds for a PostgreSQL single field with no qualifier", () => { - var field = Field.GE("SomethingCool", 18); + var field = Field.GreaterOrEqual("SomethingCool", 18); Expect.equal("data->>'SomethingCool'", field.Path(Dialect.PostgreSQL), "The PostgreSQL path is incorrect"); }), TestCase("succeeds for a PostgreSQL single field with a qualifier", () => { - var field = Field.LT("SomethingElse", 9).WithQualifier("this"); + var field = Field.Less("SomethingElse", 9).WithQualifier("this"); Expect.equal("this.data->>'SomethingElse'", field.Path(Dialect.PostgreSQL), "The PostgreSQL path is incorrect"); }), TestCase("succeeds for a PostgreSQL nested field with no qualifier", () => { - var field = Field.EQ("My.Nested.Field", "howdy"); + var field = Field.Equal("My.Nested.Field", "howdy"); Expect.equal("data#>>'{My,Nested,Field}'", field.Path(Dialect.PostgreSQL), "The PostgreSQL path is incorrect"); }), TestCase("succeeds for a PostgreSQL nested field with a qualifier", () => { - var field = Field.EQ("Nest.Away", "doc").WithQualifier("bird"); + var field = Field.Equal("Nest.Away", "doc").WithQualifier("bird"); Expect.equal("bird.data#>>'{Nest,Away}'", field.Path(Dialect.PostgreSQL), "The PostgreSQL path is incorrect"); }), TestCase("succeeds for a SQLite single field with no qualifier", () => { - var field = Field.GE("SomethingCool", 18); + var field = Field.GreaterOrEqual("SomethingCool", 18); Expect.equal("data->>'SomethingCool'", field.Path(Dialect.SQLite), "The SQLite path is incorrect"); }), TestCase("succeeds for a SQLite single field with a qualifier", () => { - var field = Field.LT("SomethingElse", 9).WithQualifier("this"); + var field = Field.Less("SomethingElse", 9).WithQualifier("this"); Expect.equal("this.data->>'SomethingElse'", field.Path(Dialect.SQLite), "The SQLite path is incorrect"); }), TestCase("succeeds for a SQLite nested field with no qualifier", () => { - var field = Field.EQ("My.Nested.Field", "howdy"); + var field = Field.Equal("My.Nested.Field", "howdy"); Expect.equal("data->>'My'->>'Nested'->>'Field'", field.Path(Dialect.SQLite), "The SQLite path is incorrect"); }), TestCase("succeeds for a SQLite nested field with a qualifier", () => { - var field = Field.EQ("Nest.Away", "doc").WithQualifier("bird"); + var field = Field.Equal("Nest.Away", "doc").WithQualifier("bird"); Expect.equal("bird.data->>'Nest'->>'Away'", field.Path(Dialect.SQLite), "The SQLite path is incorrect"); }) ]) diff --git a/src/Tests.CSharp/PostgresCSharpExtensionTests.cs b/src/Tests.CSharp/PostgresCSharpExtensionTests.cs index 054143a..65b0743 100644 --- a/src/Tests.CSharp/PostgresCSharpExtensionTests.cs +++ b/src/Tests.CSharp/PostgresCSharpExtensionTests.cs @@ -205,7 +205,7 @@ public class PostgresCSharpExtensionTests } }) ]), - TestList("save", + TestList("Save", [ TestCase("succeeds when a document is inserted", async () => { @@ -253,7 +253,7 @@ public class PostgresCSharpExtensionTests await LoadDocs(); var theCount = await conn.CountByFields(PostgresDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "purple")]); + [Field.Equal("Value", "purple")]); Expect.equal(theCount, 2, "There should have been 2 matching documents"); }), TestCase("CountByContains succeeds", async () => @@ -303,7 +303,7 @@ public class PostgresCSharpExtensionTests await using var conn = MkConn(db); await LoadDocs(); - var exists = await conn.ExistsByFields(PostgresDb.TableName, FieldMatch.Any, [Field.EX("Sub")]); + var exists = await conn.ExistsByFields(PostgresDb.TableName, FieldMatch.Any, [Field.Exists("Sub")]); Expect.isTrue(exists, "There should have been existing documents"); }), TestCase("succeeds when documents do not exist", async () => @@ -313,7 +313,7 @@ public class PostgresCSharpExtensionTests await LoadDocs(); var exists = - await conn.ExistsByFields(PostgresDb.TableName, FieldMatch.Any, [Field.EQ("NumValue", "six")]); + await conn.ExistsByFields(PostgresDb.TableName, FieldMatch.Any, [Field.Equal("NumValue", "six")]); Expect.isFalse(exists, "There should not have been existing documents"); }) ]), @@ -450,7 +450,7 @@ public class PostgresCSharpExtensionTests await LoadDocs(); var docs = await conn.FindByFields(PostgresDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "another")]); + [Field.Equal("Value", "another")]); Expect.equal(docs.Count, 1, "There should have been one document returned"); }), TestCase("succeeds when documents are not found", async () => @@ -460,7 +460,7 @@ public class PostgresCSharpExtensionTests await LoadDocs(); var docs = await conn.FindByFields(PostgresDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "mauve")]); + [Field.Equal("Value", "mauve")]); Expect.isEmpty(docs, "There should have been no documents returned"); }) ]), @@ -473,7 +473,7 @@ public class PostgresCSharpExtensionTests await LoadDocs(); var docs = await conn.FindByFieldsOrdered(PostgresDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "purple")], [Field.Named("Id")]); + [Field.Equal("Value", "purple")], [Field.Named("Id")]); Expect.hasLength(docs, 2, "There should have been two document returned"); Expect.equal(string.Join('|', docs.Select(x => x.Id)), "five|four", "The documents were not ordered correctly"); @@ -485,7 +485,7 @@ public class PostgresCSharpExtensionTests await LoadDocs(); var docs = await conn.FindByFieldsOrdered(PostgresDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "purple")], [Field.Named("Id DESC")]); + [Field.Equal("Value", "purple")], [Field.Named("Id DESC")]); Expect.hasLength(docs, 2, "There should have been two document returned"); Expect.equal(string.Join('|', docs.Select(x => x.Id)), "four|five", "The documents were not ordered correctly"); @@ -599,7 +599,7 @@ public class PostgresCSharpExtensionTests await LoadDocs(); var doc = await conn.FindFirstByFields(PostgresDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "another")]); + [Field.Equal("Value", "another")]); Expect.isNotNull(doc, "There should have been a document returned"); Expect.equal(doc.Id, "two", "The incorrect document was returned"); }), @@ -610,7 +610,7 @@ public class PostgresCSharpExtensionTests await LoadDocs(); var doc = await conn.FindFirstByFields(PostgresDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "purple")]); + [Field.Equal("Value", "purple")]); Expect.isNotNull(doc, "There should have been a document returned"); Expect.contains(["five", "four"], doc.Id, "An incorrect document was returned"); }), @@ -621,7 +621,7 @@ public class PostgresCSharpExtensionTests await LoadDocs(); var doc = await conn.FindFirstByFields(PostgresDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "absent")]); + [Field.Equal("Value", "absent")]); Expect.isNull(doc, "There should not have been a document returned"); }) ]), @@ -634,7 +634,7 @@ public class PostgresCSharpExtensionTests await LoadDocs(); var doc = await conn.FindFirstByFieldsOrdered(PostgresDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "purple")], [Field.Named("Id")]); + [Field.Equal("Value", "purple")], [Field.Named("Id")]); Expect.isNotNull(doc, "There should have been a document returned"); Expect.equal("five", doc.Id, "An incorrect document was returned"); }), @@ -645,7 +645,7 @@ public class PostgresCSharpExtensionTests await LoadDocs(); var doc = await conn.FindFirstByFieldsOrdered(PostgresDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "purple")], [Field.Named("Id DESC")]); + [Field.Equal("Value", "purple")], [Field.Named("Id DESC")]); Expect.isNotNull(doc, "There should have been a document returned"); Expect.equal("four", doc.Id, "An incorrect document was returned"); }) @@ -859,10 +859,10 @@ public class PostgresCSharpExtensionTests await using var conn = MkConn(db); await LoadDocs(); - await conn.PatchByFields(PostgresDb.TableName, FieldMatch.Any, [Field.EQ("Value", "purple")], + await conn.PatchByFields(PostgresDb.TableName, FieldMatch.Any, [Field.Equal("Value", "purple")], new { NumValue = 77 }); var after = await conn.CountByFields(PostgresDb.TableName, FieldMatch.Any, - [Field.EQ("NumValue", "77")]); + [Field.Equal("NumValue", 77)]); Expect.equal(after, 2, "There should have been 2 documents returned"); }), TestCase("succeeds when no document is updated", async () => @@ -873,7 +873,7 @@ public class PostgresCSharpExtensionTests Expect.equal(before, 0, "There should have been no documents returned"); // This not raising an exception is the test - await conn.PatchByFields(PostgresDb.TableName, FieldMatch.Any, [Field.EQ("Value", "burgundy")], + await conn.PatchByFields(PostgresDb.TableName, FieldMatch.Any, [Field.Equal("Value", "burgundy")], new { Foo = "green" }); }) ]), @@ -975,7 +975,7 @@ public class PostgresCSharpExtensionTests await using var conn = MkConn(db); await LoadDocs(); - await conn.RemoveFieldsByFields(PostgresDb.TableName, FieldMatch.Any, [Field.EQ("NumValue", "17")], + await conn.RemoveFieldsByFields(PostgresDb.TableName, FieldMatch.Any, [Field.Equal("NumValue", "17")], ["Sub", "Value"]); var updated = await Find.ById(PostgresDb.TableName, "four"); Expect.isNotNull(updated, "The updated document should have been retrieved"); @@ -988,7 +988,7 @@ public class PostgresCSharpExtensionTests await using var conn = MkConn(db); await LoadDocs(); - await conn.RemoveFieldsByFields(PostgresDb.TableName, FieldMatch.Any, [Field.EQ("NumValue", "17")], + await conn.RemoveFieldsByFields(PostgresDb.TableName, FieldMatch.Any, [Field.Equal("NumValue", "17")], ["Sub"]); var updated = await Find.ById(PostgresDb.TableName, "four"); Expect.isNotNull(updated, "The updated document should have been retrieved"); @@ -1002,7 +1002,7 @@ public class PostgresCSharpExtensionTests await LoadDocs(); // This not raising an exception is the test - await conn.RemoveFieldsByFields(PostgresDb.TableName, FieldMatch.Any, [Field.EQ("NumValue", "17")], + await conn.RemoveFieldsByFields(PostgresDb.TableName, FieldMatch.Any, [Field.Equal("NumValue", "17")], ["Nothing"]); }), TestCase("succeeds when no document is matched", async () => @@ -1012,7 +1012,7 @@ public class PostgresCSharpExtensionTests // This not raising an exception is the test await conn.RemoveFieldsByFields(PostgresDb.TableName, FieldMatch.Any, - [Field.NE("Abracadabra", "apple")], ["Value"]); + [Field.NotEqual("Abracadabra", "apple")], ["Value"]); }) ]), TestList("RemoveFieldsByContains", @@ -1134,7 +1134,7 @@ public class PostgresCSharpExtensionTests await using var conn = MkConn(db); await LoadDocs(); - await conn.DeleteByFields(PostgresDb.TableName, FieldMatch.Any, [Field.NE("Value", "purple")]); + await conn.DeleteByFields(PostgresDb.TableName, FieldMatch.Any, [Field.NotEqual("Value", "purple")]); var remaining = await conn.CountAll(PostgresDb.TableName); Expect.equal(remaining, 2, "There should have been 2 documents remaining"); }), @@ -1144,7 +1144,7 @@ public class PostgresCSharpExtensionTests await using var conn = MkConn(db); await LoadDocs(); - await conn.DeleteByFields(PostgresDb.TableName, FieldMatch.Any, [Field.EQ("Value", "crimson")]); + await conn.DeleteByFields(PostgresDb.TableName, FieldMatch.Any, [Field.Equal("Value", "crimson")]); var remaining = await conn.CountAll(PostgresDb.TableName); Expect.equal(remaining, 5, "There should have been 5 documents remaining"); }) diff --git a/src/Tests.CSharp/PostgresCSharpTests.cs b/src/Tests.CSharp/PostgresCSharpTests.cs index 63460ed..570fdfd 100644 --- a/src/Tests.CSharp/PostgresCSharpTests.cs +++ b/src/Tests.CSharp/PostgresCSharpTests.cs @@ -111,7 +111,7 @@ public static class PostgresCSharpTests [ TestCase("succeeds when a parameter is added", () => { - var paramList = Parameters.AddFields([Field.EQ("it", "242")], []).ToList(); + var paramList = Parameters.AddFields([Field.Equal("it", "242")], []).ToList(); Expect.hasLength(paramList, 1, "There should have been a parameter added"); var (name, value) = paramList[0]; Expect.equal(name, "@field0", "Field parameter name not correct"); @@ -119,7 +119,7 @@ public static class PostgresCSharpTests }), TestCase("succeeds when multiple independent parameters are added", () => { - var paramList = Parameters.AddFields([Field.EQ("me", "you"), Field.GT("us", "them")], + var paramList = Parameters.AddFields([Field.Equal("me", "you"), Field.Greater("us", "them")], [Parameters.Id(14)]).ToList(); Expect.hasLength(paramList, 3, "There should have been 2 parameters added"); var (name, value) = paramList[0]; @@ -134,13 +134,13 @@ public static class PostgresCSharpTests }), TestCase("succeeds when a parameter is not added", () => { - var paramList = Parameters.AddFields([Field.EX("tacos")], []).ToList(); + var paramList = Parameters.AddFields([Field.Exists("tacos")], []).ToList(); Expect.isEmpty(paramList, "There should not have been any parameters added"); }), TestCase("succeeds when two parameters are added for one field", () => { var paramList = - Parameters.AddFields([Field.BT("that", "eh", "zed").WithParameterName("@test")], []).ToList(); + Parameters.AddFields([Field.Between("that", "eh", "zed").WithParameterName("@test")], []).ToList(); Expect.hasLength(paramList, 2, "There should have been 2 parameters added"); var (name, value) = paramList[0]; Expect.equal(name, "@testmin", "Minimum field name not correct"); @@ -188,40 +188,40 @@ public static class PostgresCSharpTests { Expect.equal( Postgres.Query.WhereByFields(FieldMatch.Any, - [Field.GT("theField", "0").WithParameterName("@test")]), + [Field.Greater("theField", "0").WithParameterName("@test")]), "data->>'theField' > @test", "WHERE clause not correct"); }), TestCase("succeeds for a single field when an existence operator is passed", () => { - Expect.equal(Postgres.Query.WhereByFields(FieldMatch.Any, [Field.NEX("thatField")]), + Expect.equal(Postgres.Query.WhereByFields(FieldMatch.Any, [Field.NotExists("thatField")]), "data->>'thatField' IS NULL", "WHERE clause not correct"); }), TestCase("succeeds for a single field when a between operator is passed with numeric values", () => { Expect.equal( Postgres.Query.WhereByFields(FieldMatch.All, - [Field.BT("aField", 50, 99).WithParameterName("@range")]), + [Field.Between("aField", 50, 99).WithParameterName("@range")]), "(data->>'aField')::numeric BETWEEN @rangemin AND @rangemax", "WHERE clause not correct"); }), TestCase("succeeds for a single field when a between operator is passed with non-numeric values", () => { Expect.equal( Postgres.Query.WhereByFields(FieldMatch.Any, - [Field.BT("field0", "a", "b").WithParameterName("@alpha")]), + [Field.Between("field0", "a", "b").WithParameterName("@alpha")]), "data->>'field0' BETWEEN @alphamin AND @alphamax", "WHERE clause not correct"); }), TestCase("succeeds for all multiple fields with logical operators", () => { Expect.equal( Postgres.Query.WhereByFields(FieldMatch.All, - [Field.EQ("theFirst", "1"), Field.EQ("numberTwo", "2")]), + [Field.Equal("theFirst", "1"), Field.Equal("numberTwo", "2")]), "data->>'theFirst' = @field0 AND data->>'numberTwo' = @field1", "WHERE clause not correct"); }), TestCase("succeeds for any multiple fields with an existence operator", () => { Expect.equal( Postgres.Query.WhereByFields(FieldMatch.Any, - [Field.NEX("thatField"), Field.GE("thisField", 18)]), + [Field.NotExists("thatField"), Field.GreaterOrEqual("thisField", 18)]), "data->>'thatField' IS NULL OR (data->>'thisField')::numeric >= @field0", "WHERE clause not correct"); }), @@ -229,7 +229,7 @@ public static class PostgresCSharpTests { Expect.equal( Postgres.Query.WhereByFields(FieldMatch.All, - [Field.BT("aField", 50, 99), Field.BT("anotherField", "a", "b")]), + [Field.Between("aField", 50, 99), Field.Between("anotherField", "a", "b")]), "(data->>'aField')::numeric BETWEEN @field0min AND @field0max AND data->>'anotherField' BETWEEN @field1min AND @field1max", "WHERE clause not correct"); }) @@ -299,7 +299,7 @@ public static class PostgresCSharpTests }), TestCase("ByFields succeeds", () => { - Expect.equal(Postgres.Query.ByFields("unit", FieldMatch.Any, [Field.GT("That", 14)]), + Expect.equal(Postgres.Query.ByFields("unit", FieldMatch.Any, [Field.Greater("That", 14)]), "unit WHERE (data->>'That')::numeric > @field0", "By-Field query not correct"); }), TestCase("ByContains succeeds", () => @@ -667,7 +667,7 @@ public static class PostgresCSharpTests await LoadDocs(); var theCount = await Count.ByFields(PostgresDb.TableName, FieldMatch.Any, - [Field.BT("NumValue", 10, 20)]); + [Field.Between("NumValue", 10, 20)]); Expect.equal(theCount, 3, "There should have been 3 matching documents"); }), TestCase("succeeds for non-numeric range", async () => @@ -676,7 +676,7 @@ public static class PostgresCSharpTests await LoadDocs(); var theCount = await Count.ByFields(PostgresDb.TableName, FieldMatch.All, - [Field.BT("Value", "aardvark", "apple")]); + [Field.Between("Value", "aardvark", "apple")]); Expect.equal(theCount, 1, "There should have been 1 matching document"); }) ]), @@ -729,7 +729,7 @@ public static class PostgresCSharpTests await using var db = PostgresDb.BuildDb(); await LoadDocs(); - var exists = await Exists.ByFields(PostgresDb.TableName, FieldMatch.Any, [Field.NEX("Sub")]); + var exists = await Exists.ByFields(PostgresDb.TableName, FieldMatch.Any, [Field.NotExists("Sub")]); Expect.isTrue(exists, "There should have been existing documents"); }), TestCase("succeeds when documents do not exist", async () => @@ -737,7 +737,8 @@ public static class PostgresCSharpTests await using var db = PostgresDb.BuildDb(); await LoadDocs(); - var exists = await Exists.ByFields(PostgresDb.TableName, FieldMatch.Any, [Field.EQ("NumValue", "six")]); + var exists = await Exists.ByFields(PostgresDb.TableName, FieldMatch.Any, + [Field.Equal("NumValue", "six")]); Expect.isFalse(exists, "There should not have been existing documents"); }) ]), @@ -869,7 +870,7 @@ public static class PostgresCSharpTests await LoadDocs(); var docs = await Find.ByFields(PostgresDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "another")]); + [Field.Equal("Value", "another")]); Expect.hasLength(docs, 1, "There should have been one document returned"); }), TestCase("succeeds when documents are found using IN with numeric field", async () => @@ -878,7 +879,7 @@ public static class PostgresCSharpTests await LoadDocs(); var docs = await Find.ByFields(PostgresDb.TableName, FieldMatch.All, - [Field.IN("NumValue", [2, 4, 6, 8])]); + [Field.In("NumValue", [2, 4, 6, 8])]); Expect.hasLength(docs, 1, "There should have been one document returned"); }), TestCase("succeeds when documents are not found", async () => @@ -887,7 +888,7 @@ public static class PostgresCSharpTests await LoadDocs(); var docs = await Find.ByFields(PostgresDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "mauve")]); + [Field.Equal("Value", "mauve")]); Expect.isEmpty(docs, "There should have been no documents returned"); }) ]), @@ -899,7 +900,7 @@ public static class PostgresCSharpTests await LoadDocs(); var docs = await Find.ByFieldsOrdered(PostgresDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "purple")], [Field.Named("Id")]); + [Field.Equal("Value", "purple")], [Field.Named("Id")]); Expect.hasLength(docs, 2, "There should have been two document returned"); Expect.equal(string.Join('|', docs.Select(x => x.Id)), "five|four", "The documents were not ordered correctly"); @@ -910,7 +911,7 @@ public static class PostgresCSharpTests await LoadDocs(); var docs = await Find.ByFieldsOrdered(PostgresDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "purple")], [Field.Named("Id DESC")]); + [Field.Equal("Value", "purple")], [Field.Named("Id DESC")]); Expect.hasLength(docs, 2, "There should have been two document returned"); Expect.equal(string.Join('|', docs.Select(x => x.Id)), "four|five", "The documents were not ordered correctly"); @@ -1015,7 +1016,7 @@ public static class PostgresCSharpTests await LoadDocs(); var doc = await Find.FirstByFields(PostgresDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "another")]); + [Field.Equal("Value", "another")]); Expect.isNotNull(doc, "There should have been a document returned"); Expect.equal(doc.Id, "two", "The incorrect document was returned"); }), @@ -1025,7 +1026,7 @@ public static class PostgresCSharpTests await LoadDocs(); var doc = await Find.FirstByFields(PostgresDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "purple")]); + [Field.Equal("Value", "purple")]); Expect.isNotNull(doc, "There should have been a document returned"); Expect.contains(["five", "four"], doc.Id, "An incorrect document was returned"); }), @@ -1035,7 +1036,7 @@ public static class PostgresCSharpTests await LoadDocs(); var doc = await Find.FirstByFields(PostgresDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "absent")]); + [Field.Equal("Value", "absent")]); Expect.isNull(doc, "There should not have been a document returned"); }) ]), @@ -1047,7 +1048,7 @@ public static class PostgresCSharpTests await LoadDocs(); var doc = await Find.FirstByFieldsOrdered(PostgresDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "purple")], [Field.Named("Id")]); + [Field.Equal("Value", "purple")], [Field.Named("Id")]); Expect.isNotNull(doc, "There should have been a document returned"); Expect.equal("five", doc.Id, "An incorrect document was returned"); }), @@ -1057,7 +1058,7 @@ public static class PostgresCSharpTests await LoadDocs(); var doc = await Find.FirstByFieldsOrdered(PostgresDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "purple")], [Field.Named("Id DESC")]); + [Field.Equal("Value", "purple")], [Field.Named("Id DESC")]); Expect.isNotNull(doc, "There should have been a document returned"); Expect.equal("four", doc.Id, "An incorrect document was returned"); }) @@ -1271,9 +1272,9 @@ public static class PostgresCSharpTests await using var db = PostgresDb.BuildDb(); await LoadDocs(); - await Patch.ByFields(PostgresDb.TableName, FieldMatch.Any, [Field.EQ("Value", "purple")], + await Patch.ByFields(PostgresDb.TableName, FieldMatch.Any, [Field.Equal("Value", "purple")], new { NumValue = 77 }); - var after = await Count.ByFields(PostgresDb.TableName, FieldMatch.Any, [Field.EQ("NumValue", "77")]); + var after = await Count.ByFields(PostgresDb.TableName, FieldMatch.Any, [Field.Equal("NumValue", "77")]); Expect.equal(after, 2, "There should have been 2 documents returned"); }), TestCase("succeeds when no document is updated", async () => @@ -1284,7 +1285,7 @@ public static class PostgresCSharpTests Expect.equal(before, 0, "There should have been no documents returned"); // This not raising an exception is the test - await Patch.ByFields(PostgresDb.TableName, FieldMatch.Any, [Field.EQ("Value", "burgundy")], + await Patch.ByFields(PostgresDb.TableName, FieldMatch.Any, [Field.Equal("Value", "burgundy")], new { Foo = "green" }); }) ]), @@ -1386,7 +1387,7 @@ public static class PostgresCSharpTests await using var db = PostgresDb.BuildDb(); await LoadDocs(); - await RemoveFields.ByFields(PostgresDb.TableName, FieldMatch.Any, [Field.EQ("NumValue", "17")], + await RemoveFields.ByFields(PostgresDb.TableName, FieldMatch.Any, [Field.Equal("NumValue", "17")], ["Sub", "Value"]); var updated = await Find.ById(PostgresDb.TableName, "four"); Expect.isNotNull(updated, "The updated document should have been retrieved"); @@ -1398,7 +1399,7 @@ public static class PostgresCSharpTests await using var db = PostgresDb.BuildDb(); await LoadDocs(); - await RemoveFields.ByFields(PostgresDb.TableName, FieldMatch.Any, [Field.EQ("NumValue", "17")], + await RemoveFields.ByFields(PostgresDb.TableName, FieldMatch.Any, [Field.Equal("NumValue", "17")], ["Sub"]); var updated = await Find.ById(PostgresDb.TableName, "four"); Expect.isNotNull(updated, "The updated document should have been retrieved"); @@ -1411,7 +1412,7 @@ public static class PostgresCSharpTests await LoadDocs(); // This not raising an exception is the test - await RemoveFields.ByFields(PostgresDb.TableName, FieldMatch.Any, [Field.EQ("NumValue", "17")], + await RemoveFields.ByFields(PostgresDb.TableName, FieldMatch.Any, [Field.Equal("NumValue", "17")], ["Nothing"]); }), TestCase("succeeds when no document is matched", async () => @@ -1419,8 +1420,8 @@ public static class PostgresCSharpTests await using var db = PostgresDb.BuildDb(); // This not raising an exception is the test - await RemoveFields.ByFields(PostgresDb.TableName, FieldMatch.Any, [Field.NE("Abracadabra", "apple")], - ["Value"]); + await RemoveFields.ByFields(PostgresDb.TableName, FieldMatch.Any, + [Field.NotEqual("Abracadabra", "apple")], ["Value"]); }) ]), TestList("ByContains", @@ -1538,7 +1539,7 @@ public static class PostgresCSharpTests await using var db = PostgresDb.BuildDb(); await LoadDocs(); - await Delete.ByFields(PostgresDb.TableName, FieldMatch.Any, [Field.EQ("Value", "purple")]); + await Delete.ByFields(PostgresDb.TableName, FieldMatch.Any, [Field.Equal("Value", "purple")]); var remaining = await Count.All(PostgresDb.TableName); Expect.equal(remaining, 3, "There should have been 3 documents remaining"); }), @@ -1547,7 +1548,7 @@ public static class PostgresCSharpTests await using var db = PostgresDb.BuildDb(); await LoadDocs(); - await Delete.ByFields(PostgresDb.TableName, FieldMatch.Any, [Field.EQ("Value", "crimson")]); + await Delete.ByFields(PostgresDb.TableName, FieldMatch.Any, [Field.Equal("Value", "crimson")]); var remaining = await Count.All(PostgresDb.TableName); Expect.equal(remaining, 5, "There should have been 5 documents remaining"); }) diff --git a/src/Tests.CSharp/PostgresDb.cs b/src/Tests.CSharp/PostgresDb.cs index 4d84e2c..063c299 100644 --- a/src/Tests.CSharp/PostgresDb.cs +++ b/src/Tests.CSharp/PostgresDb.cs @@ -1,4 +1,3 @@ -using BitBadger.Documents.Postgres; using Npgsql; using Npgsql.FSharp; using ThrowawayDb.Postgres; diff --git a/src/Tests.CSharp/SqliteCSharpExtensionTests.cs b/src/Tests.CSharp/SqliteCSharpExtensionTests.cs index 7290183..5d23375 100644 --- a/src/Tests.CSharp/SqliteCSharpExtensionTests.cs +++ b/src/Tests.CSharp/SqliteCSharpExtensionTests.cs @@ -221,7 +221,8 @@ public static class SqliteCSharpExtensionTests await using var conn = Sqlite.Configuration.DbConn(); await LoadDocs(); - var theCount = await conn.CountByFields(SqliteDb.TableName, FieldMatch.Any, [Field.EQ("Value", "purple")]); + var theCount = await conn.CountByFields(SqliteDb.TableName, FieldMatch.Any, + [Field.Equal("Value", "purple")]); Expect.equal(theCount, 2L, "There should have been 2 matching documents"); }), TestList("ExistsById", @@ -253,7 +254,8 @@ public static class SqliteCSharpExtensionTests await using var conn = Sqlite.Configuration.DbConn(); await LoadDocs(); - var exists = await conn.ExistsByFields(SqliteDb.TableName, FieldMatch.Any, [Field.GE("NumValue", 10)]); + var exists = await conn.ExistsByFields(SqliteDb.TableName, FieldMatch.Any, + [Field.GreaterOrEqual("NumValue", 10)]); Expect.isTrue(exists, "There should have been existing documents"); }), TestCase("succeeds when no matching documents exist", async () => @@ -263,7 +265,7 @@ public static class SqliteCSharpExtensionTests await LoadDocs(); var exists = - await conn.ExistsByFields(SqliteDb.TableName, FieldMatch.Any, [Field.EQ("Nothing", "none")]); + await conn.ExistsByFields(SqliteDb.TableName, FieldMatch.Any, [Field.Equal("Nothing", "none")]); Expect.isFalse(exists, "There should not have been any existing documents"); }) ]), @@ -357,7 +359,7 @@ public static class SqliteCSharpExtensionTests await LoadDocs(); var docs = await conn.FindByFields(SqliteDb.TableName, FieldMatch.Any, - [Field.GT("NumValue", 15)]); + [Field.Greater("NumValue", 15)]); Expect.equal(docs.Count, 2, "There should have been two documents returned"); }), TestCase("succeeds when documents are not found", async () => @@ -367,7 +369,7 @@ public static class SqliteCSharpExtensionTests await LoadDocs(); var docs = await conn.FindByFields(SqliteDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "mauve")]); + [Field.Equal("Value", "mauve")]); Expect.isEmpty(docs, "There should have been no documents returned"); }) ]), @@ -380,7 +382,7 @@ public static class SqliteCSharpExtensionTests await LoadDocs(); var docs = await conn.FindByFieldsOrdered(SqliteDb.TableName, FieldMatch.Any, - [Field.GT("NumValue", 15)], [Field.Named("Id")]); + [Field.Greater("NumValue", 15)], [Field.Named("Id")]); Expect.equal(string.Join('|', docs.Select(x => x.Id)), "five|four", "There should have been two documents returned"); }), @@ -391,7 +393,7 @@ public static class SqliteCSharpExtensionTests await LoadDocs(); var docs = await conn.FindByFieldsOrdered(SqliteDb.TableName, FieldMatch.Any, - [Field.GT("NumValue", 15)], [Field.Named("Id DESC")]); + [Field.Greater("NumValue", 15)], [Field.Named("Id DESC")]); Expect.equal(string.Join('|', docs.Select(x => x.Id)), "four|five", "There should have been two documents returned"); }) @@ -405,7 +407,7 @@ public static class SqliteCSharpExtensionTests await LoadDocs(); var doc = await conn.FindFirstByFields(SqliteDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "another")]); + [Field.Equal("Value", "another")]); Expect.isNotNull(doc, "There should have been a document returned"); Expect.equal(doc!.Id, "two", "The incorrect document was returned"); }), @@ -416,7 +418,7 @@ public static class SqliteCSharpExtensionTests await LoadDocs(); var doc = await conn.FindFirstByFields(SqliteDb.TableName, FieldMatch.Any, - [Field.EQ("Sub.Foo", "green")]); + [Field.Equal("Sub.Foo", "green")]); Expect.isNotNull(doc, "There should have been a document returned"); Expect.contains(["two", "four"], doc!.Id, "An incorrect document was returned"); }), @@ -427,7 +429,7 @@ public static class SqliteCSharpExtensionTests await LoadDocs(); var doc = await conn.FindFirstByFields(SqliteDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "absent")]); + [Field.Equal("Value", "absent")]); Expect.isNull(doc, "There should not have been a document returned"); }) ]), @@ -440,7 +442,7 @@ public static class SqliteCSharpExtensionTests await LoadDocs(); var doc = await conn.FindFirstByFieldsOrdered(SqliteDb.TableName, FieldMatch.Any, - [Field.EQ("Sub.Foo", "green")], [Field.Named("Sub.Bar")]); + [Field.Equal("Sub.Foo", "green")], [Field.Named("Sub.Bar")]); Expect.isNotNull(doc, "There should have been a document returned"); Expect.equal("two", doc!.Id, "An incorrect document was returned"); }), @@ -451,7 +453,7 @@ public static class SqliteCSharpExtensionTests await LoadDocs(); var doc = await conn.FindFirstByFieldsOrdered(SqliteDb.TableName, FieldMatch.Any, - [Field.EQ("Sub.Foo", "green")], [Field.Named("Sub.Bar DESC")]); + [Field.Equal("Sub.Foo", "green")], [Field.Named("Sub.Bar DESC")]); Expect.isNotNull(doc, "There should have been a document returned"); Expect.equal("four", doc!.Id, "An incorrect document was returned"); }) @@ -547,9 +549,9 @@ public static class SqliteCSharpExtensionTests await using var conn = Sqlite.Configuration.DbConn(); await LoadDocs(); - await conn.PatchByFields(SqliteDb.TableName, FieldMatch.Any, [Field.EQ("Value", "purple")], + await conn.PatchByFields(SqliteDb.TableName, FieldMatch.Any, [Field.Equal("Value", "purple")], new { NumValue = 77 }); - var after = await conn.CountByFields(SqliteDb.TableName, FieldMatch.Any, [Field.EQ("NumValue", 77)]); + var after = await conn.CountByFields(SqliteDb.TableName, FieldMatch.Any, [Field.Equal("NumValue", 77)]); Expect.equal(after, 2L, "There should have been 2 documents returned"); }), TestCase("succeeds when no document is updated", async () => @@ -560,7 +562,7 @@ public static class SqliteCSharpExtensionTests Expect.isEmpty(before, "There should have been no documents returned"); // This not raising an exception is the test - await conn.PatchByFields(SqliteDb.TableName, FieldMatch.Any, [Field.EQ("Value", "burgundy")], + await conn.PatchByFields(SqliteDb.TableName, FieldMatch.Any, [Field.Equal("Value", "burgundy")], new { Foo = "green" }); }) ]), @@ -604,7 +606,7 @@ public static class SqliteCSharpExtensionTests await using var conn = Sqlite.Configuration.DbConn(); await LoadDocs(); - await conn.RemoveFieldsByFields(SqliteDb.TableName, FieldMatch.Any, [Field.EQ("NumValue", 17)], + await conn.RemoveFieldsByFields(SqliteDb.TableName, FieldMatch.Any, [Field.Equal("NumValue", 17)], ["Sub"]); var updated = await Find.ById(SqliteDb.TableName, "four"); Expect.isNotNull(updated, "The updated document should have been retrieved"); @@ -617,7 +619,7 @@ public static class SqliteCSharpExtensionTests await LoadDocs(); // This not raising an exception is the test - await conn.RemoveFieldsByFields(SqliteDb.TableName, FieldMatch.Any, [Field.EQ("NumValue", 17)], + await conn.RemoveFieldsByFields(SqliteDb.TableName, FieldMatch.Any, [Field.Equal("NumValue", 17)], ["Nothing"]); }), TestCase("succeeds when no document is matched", async () => @@ -626,8 +628,8 @@ public static class SqliteCSharpExtensionTests await using var conn = Sqlite.Configuration.DbConn(); // This not raising an exception is the test - await conn.RemoveFieldsByFields(SqliteDb.TableName, FieldMatch.Any, [Field.NE("Abracadabra", "apple")], - ["Value"]); + await conn.RemoveFieldsByFields(SqliteDb.TableName, FieldMatch.Any, + [Field.NotEqual("Abracadabra", "apple")], ["Value"]); }) ]), TestList("DeleteById", @@ -661,7 +663,7 @@ public static class SqliteCSharpExtensionTests await using var conn = Sqlite.Configuration.DbConn(); await LoadDocs(); - await conn.DeleteByFields(SqliteDb.TableName, FieldMatch.Any, [Field.NE("Value", "purple")]); + await conn.DeleteByFields(SqliteDb.TableName, FieldMatch.Any, [Field.NotEqual("Value", "purple")]); var remaining = await conn.CountAll(SqliteDb.TableName); Expect.equal(remaining, 2L, "There should have been 2 documents remaining"); }), @@ -671,7 +673,7 @@ public static class SqliteCSharpExtensionTests await using var conn = Sqlite.Configuration.DbConn(); await LoadDocs(); - await conn.DeleteByFields(SqliteDb.TableName, FieldMatch.Any, [Field.EQ("Value", "crimson")]); + await conn.DeleteByFields(SqliteDb.TableName, FieldMatch.Any, [Field.Equal("Value", "crimson")]); var remaining = await conn.CountAll(SqliteDb.TableName); Expect.equal(remaining, 5L, "There should have been 5 documents remaining"); }) diff --git a/src/Tests.CSharp/SqliteCSharpTests.cs b/src/Tests.CSharp/SqliteCSharpTests.cs index 44a50c3..731a512 100644 --- a/src/Tests.CSharp/SqliteCSharpTests.cs +++ b/src/Tests.CSharp/SqliteCSharpTests.cs @@ -22,38 +22,41 @@ public static class SqliteCSharpTests TestCase("succeeds for a single field when a logical operator is passed", () => { Expect.equal( - Sqlite.Query.WhereByFields(FieldMatch.Any, [Field.GT("theField", 0).WithParameterName("@test")]), + Sqlite.Query.WhereByFields(FieldMatch.Any, + [Field.Greater("theField", 0).WithParameterName("@test")]), "data->>'theField' > @test", "WHERE clause not correct"); }), TestCase("succeeds for a single field when an existence operator is passed", () => { - Expect.equal(Sqlite.Query.WhereByFields(FieldMatch.Any, [Field.NEX("thatField")]), + Expect.equal(Sqlite.Query.WhereByFields(FieldMatch.Any, [Field.NotExists("thatField")]), "data->>'thatField' IS NULL", "WHERE clause not correct"); }), TestCase("succeeds for a single field when a between operator is passed", () => { Expect.equal( Sqlite.Query.WhereByFields(FieldMatch.All, - [Field.BT("aField", 50, 99).WithParameterName("@range")]), + [Field.Between("aField", 50, 99).WithParameterName("@range")]), "data->>'aField' BETWEEN @rangemin AND @rangemax", "WHERE clause not correct"); }), TestCase("succeeds for all multiple fields with logical operators", () => { Expect.equal( - Sqlite.Query.WhereByFields(FieldMatch.All, [Field.EQ("theFirst", "1"), Field.EQ("numberTwo", "2")]), + Sqlite.Query.WhereByFields(FieldMatch.All, + [Field.Equal("theFirst", "1"), Field.Equal("numberTwo", "2")]), "data->>'theFirst' = @field0 AND data->>'numberTwo' = @field1", "WHERE clause not correct"); }), TestCase("succeeds for any multiple fields with an existence operator", () => { Expect.equal( - Sqlite.Query.WhereByFields(FieldMatch.Any, [Field.NEX("thatField"), Field.GE("thisField", 18)]), + Sqlite.Query.WhereByFields(FieldMatch.Any, + [Field.NotExists("thatField"), Field.GreaterOrEqual("thisField", 18)]), "data->>'thatField' IS NULL OR data->>'thisField' >= @field0", "WHERE clause not correct"); }), TestCase("succeeds for all multiple fields with between operators", () => { Expect.equal( Sqlite.Query.WhereByFields(FieldMatch.All, - [Field.BT("aField", 50, 99), Field.BT("anotherField", "a", "b")]), + [Field.Between("aField", 50, 99), Field.Between("anotherField", "a", "b")]), "data->>'aField' BETWEEN @field0min AND @field0max AND data->>'anotherField' BETWEEN @field1min AND @field1max", "WHERE clause not correct"); }) @@ -79,7 +82,7 @@ public static class SqliteCSharpTests }), TestCase("ByFields succeeds", () => { - Expect.equal(Sqlite.Query.ByFields("unit", FieldMatch.Any, [Field.GT("That", 14)]), + Expect.equal(Sqlite.Query.ByFields("unit", FieldMatch.Any, [Field.Greater("That", 14)]), "unit WHERE data->>'That' > @field0", "By-Field query not correct"); }), TestCase("Definition.EnsureTable succeeds", () => @@ -109,7 +112,7 @@ public static class SqliteCSharpTests #pragma warning disable CS0618 TestCase("AddField succeeds when adding a parameter", () => { - var paramList = Parameters.AddField("@field", Field.EQ("it", 99), []).ToList(); + var paramList = Parameters.AddField("@field", Field.Equal("it", 99), []).ToList(); Expect.hasLength(paramList, 1, "There should have been a parameter added"); var theParam = paramList[0]; Expect.equal(theParam.ParameterName, "@field", "The parameter name is incorrect"); @@ -117,7 +120,7 @@ public static class SqliteCSharpTests }), TestCase("AddField succeeds when not adding a parameter", () => { - var paramSeq = Parameters.AddField("@it", Field.EX("Coffee"), []); + var paramSeq = Parameters.AddField("@it", Field.Exists("Coffee"), []); Expect.isEmpty(paramSeq, "There should not have been any parameters added"); }), #pragma warning restore CS0618 @@ -447,7 +450,8 @@ public static class SqliteCSharpTests await using var db = await SqliteDb.BuildDb(); await LoadDocs(); - var theCount = await Count.ByFields(SqliteDb.TableName, FieldMatch.Any, [Field.BT("NumValue", 10, 20)]); + var theCount = await Count.ByFields(SqliteDb.TableName, FieldMatch.Any, + [Field.Between("NumValue", 10, 20)]); Expect.equal(theCount, 3L, "There should have been 3 matching documents"); }), TestCase("succeeds for non-numeric range", async () => @@ -456,7 +460,7 @@ public static class SqliteCSharpTests await LoadDocs(); var theCount = await Count.ByFields(SqliteDb.TableName, FieldMatch.Any, - [Field.BT("Value", "aardvark", "apple")]); + [Field.Between("Value", "aardvark", "apple")]); Expect.equal(theCount, 1L, "There should have been 1 matching document"); }) ]) @@ -493,7 +497,8 @@ public static class SqliteCSharpTests await using var db = await SqliteDb.BuildDb(); await LoadDocs(); - var exists = await Exists.ByFields(SqliteDb.TableName, FieldMatch.Any, [Field.GE("NumValue", 10)]); + var exists = await Exists.ByFields(SqliteDb.TableName, FieldMatch.Any, + [Field.GreaterOrEqual("NumValue", 10)]); Expect.isTrue(exists, "There should have been existing documents"); }), TestCase("succeeds when no matching documents exist", async () => @@ -501,7 +506,8 @@ public static class SqliteCSharpTests await using var db = await SqliteDb.BuildDb(); await LoadDocs(); - var exists = await Exists.ByFields(SqliteDb.TableName, FieldMatch.Any, [Field.EQ("Nothing", "none")]); + var exists = await Exists.ByFields(SqliteDb.TableName, FieldMatch.Any, + [Field.Equal("Nothing", "none")]); Expect.isFalse(exists, "There should not have been any existing documents"); }) ]) @@ -593,7 +599,7 @@ public static class SqliteCSharpTests await LoadDocs(); var docs = await Find.ByFields(SqliteDb.TableName, FieldMatch.Any, - [Field.GT("NumValue", 15)]); + [Field.Greater("NumValue", 15)]); Expect.equal(docs.Count, 2, "There should have been two documents returned"); }), TestCase("succeeds when documents are found using IN with numeric field", async () => @@ -602,7 +608,7 @@ public static class SqliteCSharpTests await LoadDocs(); var docs = await Find.ByFields(SqliteDb.TableName, FieldMatch.All, - [Field.IN("NumValue", [2, 4, 6, 8])]); + [Field.In("NumValue", [2, 4, 6, 8])]); Expect.hasLength(docs, 1, "There should have been one document returned"); }), TestCase("succeeds when documents are not found", async () => @@ -611,7 +617,7 @@ public static class SqliteCSharpTests await LoadDocs(); var docs = await Find.ByFields(SqliteDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "mauve")]); + [Field.Equal("Value", "mauve")]); Expect.isEmpty(docs, "There should have been no documents returned"); }) ]), @@ -623,7 +629,7 @@ public static class SqliteCSharpTests await LoadDocs(); var docs = await Find.ByFieldsOrdered(SqliteDb.TableName, FieldMatch.Any, - [Field.GT("NumValue", 15)], [Field.Named("Id")]); + [Field.Greater("NumValue", 15)], [Field.Named("Id")]); Expect.hasLength(docs, 2, "There should have been two documents returned"); Expect.equal(string.Join('|', docs.Select(x => x.Id)), "five|four", "The documents were not sorted correctly"); @@ -634,7 +640,7 @@ public static class SqliteCSharpTests await LoadDocs(); var docs = await Find.ByFieldsOrdered(SqliteDb.TableName, FieldMatch.Any, - [Field.GT("NumValue", 15)], [Field.Named("Id DESC")]); + [Field.Greater("NumValue", 15)], [Field.Named("Id DESC")]); Expect.hasLength(docs, 2, "There should have been two documents returned"); Expect.equal(string.Join('|', docs.Select(x => x.Id)), "four|five", "The documents were not sorted correctly"); @@ -645,7 +651,7 @@ public static class SqliteCSharpTests await LoadDocs(); var docs = await Find.ByFieldsOrdered(SqliteDb.TableName, FieldMatch.Any, - [Field.LE("NumValue", 10)], [Field.Named("Value")]); + [Field.LessOrEqual("NumValue", 10)], [Field.Named("Value")]); Expect.hasLength(docs, 3, "There should have been three documents returned"); Expect.equal(string.Join('|', docs.Select(x => x.Id)), "three|one|two", "The documents were not sorted correctly"); @@ -656,7 +662,7 @@ public static class SqliteCSharpTests await LoadDocs(); var docs = await Find.ByFieldsOrdered(SqliteDb.TableName, FieldMatch.Any, - [Field.LE("NumValue", 10)], [Field.Named("i:Value")]); + [Field.LessOrEqual("NumValue", 10)], [Field.Named("i:Value")]); Expect.hasLength(docs, 3, "There should have been three documents returned"); Expect.equal(string.Join('|', docs.Select(x => x.Id)), "three|two|one", "The documents were not sorted correctly"); @@ -670,7 +676,7 @@ public static class SqliteCSharpTests await LoadDocs(); var doc = await Find.FirstByFields(SqliteDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "another")]); + [Field.Equal("Value", "another")]); Expect.isNotNull(doc, "There should have been a document returned"); Expect.equal(doc!.Id, "two", "The incorrect document was returned"); }), @@ -680,7 +686,7 @@ public static class SqliteCSharpTests await LoadDocs(); var doc = await Find.FirstByFields(SqliteDb.TableName, FieldMatch.Any, - [Field.EQ("Sub.Foo", "green")]); + [Field.Equal("Sub.Foo", "green")]); Expect.isNotNull(doc, "There should have been a document returned"); Expect.contains(["two", "four"], doc!.Id, "An incorrect document was returned"); }), @@ -690,7 +696,7 @@ public static class SqliteCSharpTests await LoadDocs(); var doc = await Find.FirstByFields(SqliteDb.TableName, FieldMatch.Any, - [Field.EQ("Value", "absent")]); + [Field.Equal("Value", "absent")]); Expect.isNull(doc, "There should not have been a document returned"); }) ]), @@ -702,7 +708,7 @@ public static class SqliteCSharpTests await LoadDocs(); var doc = await Find.FirstByFieldsOrdered(SqliteDb.TableName, FieldMatch.Any, - [Field.EQ("Sub.Foo", "green")], [Field.Named("Sub.Bar")]); + [Field.Equal("Sub.Foo", "green")], [Field.Named("Sub.Bar")]); Expect.isNotNull(doc, "There should have been a document returned"); Expect.equal("two", doc!.Id, "An incorrect document was returned"); }), @@ -712,7 +718,7 @@ public static class SqliteCSharpTests await LoadDocs(); var doc = await Find.FirstByFieldsOrdered(SqliteDb.TableName, FieldMatch.Any, - [Field.EQ("Sub.Foo", "green")], [Field.Named("Sub.Bar DESC")]); + [Field.Equal("Sub.Foo", "green")], [Field.Named("Sub.Bar DESC")]); Expect.isNotNull(doc, "There should have been a document returned"); Expect.equal("four", doc!.Id, "An incorrect document was returned"); }) @@ -818,9 +824,9 @@ public static class SqliteCSharpTests await using var db = await SqliteDb.BuildDb(); await LoadDocs(); - await Patch.ByFields(SqliteDb.TableName, FieldMatch.Any, [Field.EQ("Value", "purple")], + await Patch.ByFields(SqliteDb.TableName, FieldMatch.Any, [Field.Equal("Value", "purple")], new { NumValue = 77 }); - var after = await Count.ByFields(SqliteDb.TableName, FieldMatch.Any, [Field.EQ("NumValue", 77)]); + var after = await Count.ByFields(SqliteDb.TableName, FieldMatch.Any, [Field.Equal("NumValue", 77)]); Expect.equal(after, 2L, "There should have been 2 documents returned"); }), TestCase("succeeds when no document is updated", async () => @@ -831,7 +837,7 @@ public static class SqliteCSharpTests Expect.isEmpty(before, "There should have been no documents returned"); // This not raising an exception is the test - await Patch.ByFields(SqliteDb.TableName, FieldMatch.Any, [Field.EQ("Value", "burgundy")], + await Patch.ByFields(SqliteDb.TableName, FieldMatch.Any, [Field.Equal("Value", "burgundy")], new { Foo = "green" }); }) ]) @@ -878,7 +884,7 @@ public static class SqliteCSharpTests await using var db = await SqliteDb.BuildDb(); await LoadDocs(); - await RemoveFields.ByFields(SqliteDb.TableName, FieldMatch.Any, [Field.EQ("NumValue", 17)], ["Sub"]); + await RemoveFields.ByFields(SqliteDb.TableName, FieldMatch.Any, [Field.Equal("NumValue", 17)], ["Sub"]); var updated = await Find.ById(SqliteDb.TableName, "four"); Expect.isNotNull(updated, "The updated document should have been retrieved"); Expect.isNull(updated.Sub, "The sub-document should have been removed"); @@ -889,7 +895,7 @@ public static class SqliteCSharpTests await LoadDocs(); // This not raising an exception is the test - await RemoveFields.ByFields(SqliteDb.TableName, FieldMatch.Any, [Field.EQ("NumValue", 17)], + await RemoveFields.ByFields(SqliteDb.TableName, FieldMatch.Any, [Field.Equal("NumValue", 17)], ["Nothing"]); }), TestCase("succeeds when no document is matched", async () => @@ -897,8 +903,8 @@ public static class SqliteCSharpTests await using var db = await SqliteDb.BuildDb(); // This not raising an exception is the test - await RemoveFields.ByFields(SqliteDb.TableName, FieldMatch.Any, [Field.NE("Abracadabra", "apple")], - ["Value"]); + await RemoveFields.ByFields(SqliteDb.TableName, FieldMatch.Any, + [Field.NotEqual("Abracadabra", "apple")], ["Value"]); }) ]) ]); @@ -936,7 +942,7 @@ public static class SqliteCSharpTests await using var db = await SqliteDb.BuildDb(); await LoadDocs(); - await Delete.ByFields(SqliteDb.TableName, FieldMatch.Any, [Field.NE("Value", "purple")]); + await Delete.ByFields(SqliteDb.TableName, FieldMatch.Any, [Field.NotEqual("Value", "purple")]); var remaining = await Count.All(SqliteDb.TableName); Expect.equal(remaining, 2L, "There should have been 2 documents remaining"); }), @@ -945,7 +951,7 @@ public static class SqliteCSharpTests await using var db = await SqliteDb.BuildDb(); await LoadDocs(); - await Delete.ByFields(SqliteDb.TableName, FieldMatch.All, [Field.EQ("Value", "crimson")]); + await Delete.ByFields(SqliteDb.TableName, FieldMatch.All, [Field.Equal("Value", "crimson")]); var remaining = await Count.All(SqliteDb.TableName); Expect.equal(remaining, 5L, "There should have been 5 documents remaining"); }) diff --git a/src/Tests/CommonTests.fs b/src/Tests/CommonTests.fs index f3f73ad..7230af8 100644 --- a/src/Tests/CommonTests.fs +++ b/src/Tests/CommonTests.fs @@ -8,8 +8,8 @@ let tbl = "test_table" /// Unit tests for the Op DU let comparisonTests = testList "Comparison.OpSql" [ - test "Equals succeeds" { - Expect.equal (Equals "").OpSql "=" "The Equals SQL was not correct" + test "Equal succeeds" { + Expect.equal (Equal "").OpSql "=" "The Equals SQL was not correct" } test "Greater succeeds" { Expect.equal (Greater "").OpSql ">" "The Greater SQL was not correct" @@ -45,71 +45,71 @@ let comparisonTests = testList "Comparison.OpSql" [ /// Unit tests for the Field class let fieldTests = testList "Field" [ - test "EQ succeeds" { - let field = Field.EQ "Test" 14 + test "Equal succeeds" { + let field = Field.Equal "Test" 14 Expect.equal field.Name "Test" "Field name incorrect" - Expect.equal field.Comparison (Equals 14) "Comparison incorrect" + Expect.equal field.Comparison (Equal 14) "Comparison incorrect" Expect.isNone field.ParameterName "The default parameter name should be None" Expect.isNone field.Qualifier "The default table qualifier should be None" } - test "GT succeeds" { - let field = Field.GT "Great" "night" + test "Greater succeeds" { + let field = Field.Greater "Great" "night" Expect.equal field.Name "Great" "Field name incorrect" Expect.equal field.Comparison (Greater "night") "Comparison incorrect" Expect.isNone field.ParameterName "The default parameter name should be None" Expect.isNone field.Qualifier "The default table qualifier should be None" } - test "GE succeeds" { - let field = Field.GE "Nice" 88L + test "GreaterOrEqual succeeds" { + let field = Field.GreaterOrEqual "Nice" 88L Expect.equal field.Name "Nice" "Field name incorrect" Expect.equal field.Comparison (GreaterOrEqual 88L) "Comparison incorrect" Expect.isNone field.ParameterName "The default parameter name should be None" Expect.isNone field.Qualifier "The default table qualifier should be None" } - test "LT succeeds" { - let field = Field.LT "Lesser" "seven" + test "Less succeeds" { + let field = Field.Less "Lesser" "seven" Expect.equal field.Name "Lesser" "Field name incorrect" Expect.equal field.Comparison (Less "seven") "Comparison incorrect" Expect.isNone field.ParameterName "The default parameter name should be None" Expect.isNone field.Qualifier "The default table qualifier should be None" } - test "LE succeeds" { - let field = Field.LE "Nobody" "KNOWS"; + test "LessOrEqual succeeds" { + let field = Field.LessOrEqual "Nobody" "KNOWS"; Expect.equal field.Name "Nobody" "Field name incorrect" Expect.equal field.Comparison (LessOrEqual "KNOWS") "Comparison incorrect" Expect.isNone field.ParameterName "The default parameter name should be None" Expect.isNone field.Qualifier "The default table qualifier should be None" } - test "NE succeeds" { - let field = Field.NE "Park" "here" + test "NotEqual succeeds" { + let field = Field.NotEqual "Park" "here" Expect.equal field.Name "Park" "Field name incorrect" Expect.equal field.Comparison (NotEqual "here") "Comparison incorrect" Expect.isNone field.ParameterName "The default parameter name should be None" Expect.isNone field.Qualifier "The default table qualifier should be None" } - test "BT succeeds" { - let field = Field.BT "Age" 18 49 + test "Between succeeds" { + let field = Field.Between "Age" 18 49 Expect.equal field.Name "Age" "Field name incorrect" Expect.equal field.Comparison (Between(18, 49)) "Comparison incorrect" Expect.isNone field.ParameterName "The default parameter name should be None" Expect.isNone field.Qualifier "The default table qualifier should be None" } - test "IN succeeds" { - let field = Field.IN "Here" [| 8; 16; 32 |] + test "In succeeds" { + let field = Field.In "Here" [| 8; 16; 32 |] Expect.equal field.Name "Here" "Field name incorrect" Expect.equal field.Comparison (In [| 8; 16; 32 |]) "Comparison incorrect" Expect.isNone field.ParameterName "The default parameter name should be None" Expect.isNone field.Qualifier "The default table qualifier should be None" } - test "EX succeeds" { - let field = Field.EX "Groovy" + test "Exists succeeds" { + let field = Field.Exists "Groovy" Expect.equal field.Name "Groovy" "Field name incorrect" Expect.equal field.Comparison Exists "Comparison incorrect" Expect.isNone field.ParameterName "The default parameter name should be None" Expect.isNone field.Qualifier "The default table qualifier should be None" } - test "NEX succeeds" { - let field = Field.NEX "Rad" + test "NotExists succeeds" { + let field = Field.NotExists "Rad" Expect.equal field.Name "Rad" "Field name incorrect" Expect.equal field.Comparison NotExists "Comparison incorrect" Expect.isNone field.ParameterName "The default parameter name should be None" @@ -136,46 +136,46 @@ let fieldTests = testList "Field" [ } ] test "WithParameterName succeeds" { - let field = (Field.EQ "Bob" "Tom").WithParameterName "@name" + let field = (Field.Equal "Bob" "Tom").WithParameterName "@name" Expect.isSome field.ParameterName "The parameter name should have been filled" Expect.equal "@name" field.ParameterName.Value "The parameter name is incorrect" } test "WithQualifier succeeds" { - let field = (Field.EQ "Bill" "Matt").WithQualifier "joe" + let field = (Field.Equal "Bill" "Matt").WithQualifier "joe" Expect.isSome field.Qualifier "The table qualifier should have been filled" Expect.equal "joe" field.Qualifier.Value "The table qualifier is incorrect" } testList "Path" [ test "succeeds for a PostgreSQL single field with no qualifier" { - let field = Field.GE "SomethingCool" 18 + let field = Field.GreaterOrEqual "SomethingCool" 18 Expect.equal "data->>'SomethingCool'" (field.Path PostgreSQL) "The PostgreSQL path is incorrect" } test "succeeds for a PostgreSQL single field with a qualifier" { - let field = { Field.LT "SomethingElse" 9 with Qualifier = Some "this" } + let field = { Field.Less "SomethingElse" 9 with Qualifier = Some "this" } Expect.equal "this.data->>'SomethingElse'" (field.Path PostgreSQL) "The PostgreSQL path is incorrect" } test "succeeds for a PostgreSQL nested field with no qualifier" { - let field = Field.EQ "My.Nested.Field" "howdy" + let field = Field.Equal "My.Nested.Field" "howdy" Expect.equal "data#>>'{My,Nested,Field}'" (field.Path PostgreSQL) "The PostgreSQL path is incorrect" } test "succeeds for a PostgreSQL nested field with a qualifier" { - let field = { Field.EQ "Nest.Away" "doc" with Qualifier = Some "bird" } + let field = { Field.Equal "Nest.Away" "doc" with Qualifier = Some "bird" } Expect.equal "bird.data#>>'{Nest,Away}'" (field.Path PostgreSQL) "The PostgreSQL path is incorrect" } test "succeeds for a SQLite single field with no qualifier" { - let field = Field.GE "SomethingCool" 18 + let field = Field.GreaterOrEqual "SomethingCool" 18 Expect.equal "data->>'SomethingCool'" (field.Path SQLite) "The SQLite path is incorrect" } test "succeeds for a SQLite single field with a qualifier" { - let field = { Field.LT "SomethingElse" 9 with Qualifier = Some "this" } + let field = { Field.Less "SomethingElse" 9 with Qualifier = Some "this" } Expect.equal "this.data->>'SomethingElse'" (field.Path SQLite) "The SQLite path is incorrect" } test "succeeds for a SQLite nested field with no qualifier" { - let field = Field.EQ "My.Nested.Field" "howdy" + let field = Field.Equal "My.Nested.Field" "howdy" Expect.equal "data->>'My'->>'Nested'->>'Field'" (field.Path SQLite) "The SQLite path is incorrect" } test "succeeds for a SQLite nested field with a qualifier" { - let field = { Field.EQ "Nest.Away" "doc" with Qualifier = Some "bird" } + let field = { Field.Equal "Nest.Away" "doc" with Qualifier = Some "bird" } Expect.equal "bird.data->>'Nest'->>'Away'" (field.Path SQLite) "The SQLite path is incorrect" } ] diff --git a/src/Tests/PostgresExtensionTests.fs b/src/Tests/PostgresExtensionTests.fs index 533cf12..60fcf31 100644 --- a/src/Tests/PostgresExtensionTests.fs +++ b/src/Tests/PostgresExtensionTests.fs @@ -214,7 +214,7 @@ let integrationTests = use conn = mkConn db do! loadDocs conn - let! theCount = conn.countByFields PostgresDb.TableName Any [ Field.EQ "Value" "purple" ] + let! theCount = conn.countByFields PostgresDb.TableName Any [ Field.Equal "Value" "purple" ] Expect.equal theCount 2 "There should have been 2 matching documents" } testTask "countByContains succeeds" { @@ -257,7 +257,7 @@ let integrationTests = use conn = mkConn db do! loadDocs conn - let! exists = conn.existsByFields PostgresDb.TableName Any [ Field.EX "Sub" ] + let! exists = conn.existsByFields PostgresDb.TableName Any [ Field.Exists "Sub" ] Expect.isTrue exists "There should have been existing documents" } testTask "succeeds when documents do not exist" { @@ -265,7 +265,7 @@ let integrationTests = use conn = mkConn db do! loadDocs conn - let! exists = conn.existsByFields PostgresDb.TableName Any [ Field.EQ "NumValue" "six" ] + let! exists = conn.existsByFields PostgresDb.TableName Any [ Field.Equal "NumValue" "six" ] Expect.isFalse exists "There should not have been existing documents" } ] @@ -390,7 +390,7 @@ let integrationTests = use conn = mkConn db do! loadDocs conn - let! docs = conn.findByFields PostgresDb.TableName Any [ Field.EQ "Value" "another" ] + let! docs = conn.findByFields PostgresDb.TableName Any [ Field.Equal "Value" "another" ] Expect.equal (List.length docs) 1 "There should have been one document returned" } testTask "succeeds when documents are not found" { @@ -398,7 +398,7 @@ let integrationTests = use conn = mkConn db do! loadDocs conn - let! docs = conn.findByFields PostgresDb.TableName Any [ Field.EQ "Value" "mauve" ] + let! docs = conn.findByFields PostgresDb.TableName Any [ Field.Equal "Value" "mauve" ] Expect.isEmpty docs "There should have been no documents returned" } ] @@ -410,7 +410,7 @@ let integrationTests = let! docs = conn.findByFieldsOrdered - PostgresDb.TableName All [ Field.EQ "Value" "purple" ] [ Field.Named "Id" ] + PostgresDb.TableName All [ Field.Equal "Value" "purple" ] [ Field.Named "Id" ] Expect.hasLength docs 2 "There should have been two documents returned" Expect.equal (docs |> List.map _.Id |> String.concat "|") "five|four" "Documents not ordered correctly" @@ -422,7 +422,7 @@ let integrationTests = let! docs = conn.findByFieldsOrdered - PostgresDb.TableName All [ Field.EQ "Value" "purple" ] [ Field.Named "Id DESC" ] + PostgresDb.TableName All [ Field.Equal "Value" "purple" ] [ Field.Named "Id DESC" ] Expect.hasLength docs 2 "There should have been two documents returned" Expect.equal (docs |> List.map _.Id |> String.concat "|") "four|five" "Documents not ordered correctly" @@ -524,7 +524,8 @@ let integrationTests = use conn = mkConn db do! loadDocs conn - let! doc = conn.findFirstByFields PostgresDb.TableName Any [ Field.EQ "Value" "another" ] + let! doc = + conn.findFirstByFields PostgresDb.TableName Any [ Field.Equal "Value" "another" ] Expect.isSome doc "There should have been a document returned" Expect.equal doc.Value.Id "two" "The incorrect document was returned" } @@ -533,7 +534,8 @@ let integrationTests = use conn = mkConn db do! loadDocs conn - let! doc = conn.findFirstByFields PostgresDb.TableName Any [ Field.EQ "Value" "purple" ] + let! doc = + conn.findFirstByFields PostgresDb.TableName Any [ Field.Equal "Value" "purple" ] Expect.isSome doc "There should have been a document returned" Expect.contains [ "five"; "four" ] doc.Value.Id "An incorrect document was returned" } @@ -542,7 +544,8 @@ let integrationTests = use conn = mkConn db do! loadDocs conn - let! doc = conn.findFirstByFields PostgresDb.TableName Any [ Field.EQ "Value" "absent" ] + let! doc = + conn.findFirstByFields PostgresDb.TableName Any [ Field.Equal "Value" "absent" ] Expect.isNone doc "There should not have been a document returned" } ] @@ -554,7 +557,7 @@ let integrationTests = let! doc = conn.findFirstByFieldsOrdered - PostgresDb.TableName Any [ Field.EQ "Value" "purple" ] [ Field.Named "Id" ] + PostgresDb.TableName Any [ Field.Equal "Value" "purple" ] [ Field.Named "Id" ] Expect.isSome doc "There should have been a document returned" Expect.equal "five" doc.Value.Id "An incorrect document was returned" } @@ -565,7 +568,7 @@ let integrationTests = let! doc = conn.findFirstByFieldsOrdered - PostgresDb.TableName Any [ Field.EQ "Value" "purple" ] [ Field.Named "Id DESC" ] + PostgresDb.TableName Any [ Field.Equal "Value" "purple" ] [ Field.Named "Id DESC" ] Expect.isSome doc "There should have been a document returned" Expect.equal "four" doc.Value.Id "An incorrect document was returned" } @@ -750,8 +753,8 @@ let integrationTests = use conn = mkConn db do! loadDocs conn - do! conn.patchByFields PostgresDb.TableName Any [ Field.EQ "Value" "purple" ] {| NumValue = 77 |} - let! after = conn.countByFields PostgresDb.TableName Any [ Field.EQ "NumValue" "77" ] + do! conn.patchByFields PostgresDb.TableName Any [ Field.Equal "Value" "purple" ] {| NumValue = 77 |} + let! after = conn.countByFields PostgresDb.TableName Any [ Field.Equal "NumValue" "77" ] Expect.equal after 2 "There should have been 2 documents returned" } testTask "succeeds when no document is updated" { @@ -761,7 +764,7 @@ let integrationTests = Expect.equal before 0 "There should have been no documents returned" // This not raising an exception is the test - do! conn.patchByFields PostgresDb.TableName Any [ Field.EQ "Value" "burgundy" ] {| Foo = "green" |} + do! conn.patchByFields PostgresDb.TableName Any [ Field.Equal "Value" "burgundy" ] {| Foo = "green" |} } ] testList "patchByContains" [ @@ -811,9 +814,9 @@ let integrationTests = do! loadDocs conn do! conn.removeFieldsById PostgresDb.TableName "two" [ "Sub"; "Value" ] - let! noSubs = conn.countByFields PostgresDb.TableName Any [ Field.NEX "Sub" ] + let! noSubs = conn.countByFields PostgresDb.TableName Any [ Field.NotExists "Sub" ] Expect.equal noSubs 4 "There should now be 4 documents without Sub fields" - let! noValue = conn.countByFields PostgresDb.TableName Any [ Field.NEX "Value" ] + let! noValue = conn.countByFields PostgresDb.TableName Any [ Field.NotExists "Value" ] Expect.equal noValue 1 "There should be 1 document without Value fields" } testTask "succeeds when a single field is removed" { @@ -822,9 +825,9 @@ let integrationTests = do! loadDocs conn do! conn.removeFieldsById PostgresDb.TableName "two" [ "Sub" ] - let! noSubs = conn.countByFields PostgresDb.TableName Any [ Field.NEX "Sub" ] + let! noSubs = conn.countByFields PostgresDb.TableName Any [ Field.NotExists "Sub" ] Expect.equal noSubs 4 "There should now be 4 documents without Sub fields" - let! noValue = conn.countByFields PostgresDb.TableName Any [ Field.NEX "Value" ] + let! noValue = conn.countByFields PostgresDb.TableName Any [ Field.NotExists "Value" ] Expect.equal noValue 0 "There should be no documents without Value fields" } testTask "succeeds when a field is not removed" { @@ -849,10 +852,11 @@ let integrationTests = use conn = mkConn db do! loadDocs conn - do! conn.removeFieldsByFields PostgresDb.TableName Any [ Field.EQ "NumValue" "17" ] [ "Sub"; "Value" ] - let! noSubs = conn.countByFields PostgresDb.TableName Any [ Field.NEX "Sub" ] + do! conn.removeFieldsByFields + PostgresDb.TableName Any [ Field.Equal "NumValue" "17" ] [ "Sub"; "Value" ] + let! noSubs = conn.countByFields PostgresDb.TableName Any [ Field.NotExists "Sub" ] Expect.equal noSubs 4 "There should now be 4 documents without Sub fields" - let! noValue = conn.countByFields PostgresDb.TableName Any [ Field.NEX "Value" ] + let! noValue = conn.countByFields PostgresDb.TableName Any [ Field.NotExists "Value" ] Expect.equal noValue 1 "There should be 1 document without Value fields" } testTask "succeeds when a single field is removed" { @@ -860,10 +864,10 @@ let integrationTests = use conn = mkConn db do! loadDocs conn - do! conn.removeFieldsByFields PostgresDb.TableName Any [ Field.EQ "NumValue" "17" ] [ "Sub" ] - let! noSubs = conn.countByFields PostgresDb.TableName Any [ Field.NEX "Sub" ] + do! conn.removeFieldsByFields PostgresDb.TableName Any [ Field.Equal "NumValue" "17" ] [ "Sub" ] + let! noSubs = conn.countByFields PostgresDb.TableName Any [ Field.NotExists "Sub" ] Expect.equal noSubs 4 "There should now be 4 documents without Sub fields" - let! noValue = conn.countByFields PostgresDb.TableName Any [ Field.NEX "Value" ] + let! noValue = conn.countByFields PostgresDb.TableName Any [ Field.NotExists "Value" ] Expect.equal noValue 0 "There should be no documents without Value fields" } testTask "succeeds when a field is not removed" { @@ -872,14 +876,15 @@ let integrationTests = do! loadDocs conn // This not raising an exception is the test - do! conn.removeFieldsByFields PostgresDb.TableName Any [ Field.EQ "NumValue" "17" ] [ "Nothing" ] + do! conn.removeFieldsByFields PostgresDb.TableName Any [ Field.Equal "NumValue" "17" ] [ "Nothing" ] } testTask "succeeds when no document is matched" { use db = PostgresDb.BuildDb() use conn = mkConn db // This not raising an exception is the test - do! conn.removeFieldsByFields PostgresDb.TableName Any [ Field.NE "Abracadabra" "apple" ] [ "Value" ] + do! conn.removeFieldsByFields + PostgresDb.TableName Any [ Field.NotEqual "Abracadabra" "apple" ] [ "Value" ] } ] testList "removeFieldsByContains" [ @@ -889,9 +894,9 @@ let integrationTests = do! loadDocs conn do! conn.removeFieldsByContains PostgresDb.TableName {| NumValue = 17 |} [ "Sub"; "Value" ] - let! noSubs = conn.countByFields PostgresDb.TableName Any [ Field.NEX "Sub" ] + let! noSubs = conn.countByFields PostgresDb.TableName Any [ Field.NotExists "Sub" ] Expect.equal noSubs 4 "There should now be 4 documents without Sub fields" - let! noValue = conn.countByFields PostgresDb.TableName Any [ Field.NEX "Value" ] + let! noValue = conn.countByFields PostgresDb.TableName Any [ Field.NotExists "Value" ] Expect.equal noValue 1 "There should be 1 document without Value fields" } testTask "succeeds when a single field is removed" { @@ -900,9 +905,9 @@ let integrationTests = do! loadDocs conn do! conn.removeFieldsByContains PostgresDb.TableName {| NumValue = 17 |} [ "Sub" ] - let! noSubs = conn.countByFields PostgresDb.TableName Any [ Field.NEX "Sub" ] + let! noSubs = conn.countByFields PostgresDb.TableName Any [ Field.NotExists "Sub" ] Expect.equal noSubs 4 "There should now be 4 documents without Sub fields" - let! noValue = conn.countByFields PostgresDb.TableName Any [ Field.NEX "Value" ] + let! noValue = conn.countByFields PostgresDb.TableName Any [ Field.NotExists "Value" ] Expect.equal noValue 0 "There should be no documents without Value fields" } testTask "succeeds when a field is not removed" { @@ -928,9 +933,9 @@ let integrationTests = do! loadDocs conn do! conn.removeFieldsByJsonPath PostgresDb.TableName "$.NumValue ? (@ == 17)" [ "Sub"; "Value" ] - let! noSubs = conn.countByFields PostgresDb.TableName Any [ Field.NEX "Sub" ] + let! noSubs = conn.countByFields PostgresDb.TableName Any [ Field.NotExists "Sub" ] Expect.equal noSubs 4 "There should now be 4 documents without Sub fields" - let! noValue = conn.countByFields PostgresDb.TableName Any [ Field.NEX "Value" ] + let! noValue = conn.countByFields PostgresDb.TableName Any [ Field.NotExists "Value" ] Expect.equal noValue 1 "There should be 1 document without Value fields" } testTask "succeeds when a single field is removed" { @@ -939,9 +944,9 @@ let integrationTests = do! loadDocs conn do! conn.removeFieldsByJsonPath PostgresDb.TableName "$.NumValue ? (@ == 17)" [ "Sub" ] - let! noSubs = conn.countByFields PostgresDb.TableName Any [ Field.NEX "Sub" ] + let! noSubs = conn.countByFields PostgresDb.TableName Any [ Field.NotExists "Sub" ] Expect.equal noSubs 4 "There should now be 4 documents without Sub fields" - let! noValue = conn.countByFields PostgresDb.TableName Any [ Field.NEX "Value" ] + let! noValue = conn.countByFields PostgresDb.TableName Any [ Field.NotExists "Value" ] Expect.equal noValue 0 "There should be no documents without Value fields" } testTask "succeeds when a field is not removed" { @@ -986,7 +991,7 @@ let integrationTests = use conn = mkConn db do! loadDocs conn - do! conn.deleteByFields PostgresDb.TableName Any [ Field.EQ "Value" "purple" ] + do! conn.deleteByFields PostgresDb.TableName Any [ Field.Equal "Value" "purple" ] let! remaining = conn.countAll PostgresDb.TableName Expect.equal remaining 3 "There should have been 3 documents remaining" } @@ -995,7 +1000,7 @@ let integrationTests = use conn = mkConn db do! loadDocs conn - do! conn.deleteByFields PostgresDb.TableName Any [ Field.EQ "Value" "crimson" ] + do! conn.deleteByFields PostgresDb.TableName Any [ Field.Equal "Value" "crimson" ] let! remaining = conn.countAll PostgresDb.TableName Expect.equal remaining 5 "There should have been 5 documents remaining" } diff --git a/src/Tests/PostgresTests.fs b/src/Tests/PostgresTests.fs index 7e04c94..cd37e8c 100644 --- a/src/Tests/PostgresTests.fs +++ b/src/Tests/PostgresTests.fs @@ -83,14 +83,14 @@ let parametersTests = testList "Parameters" [ } testList "addFieldParams" [ test "succeeds when a parameter is added" { - let paramList = addFieldParams [ Field.EQ "it" "242" ] [] + let paramList = addFieldParams [ Field.Where "it" (Equal "242") ] [] Expect.hasLength paramList 1 "There should have been a parameter added" let name, value = Seq.head paramList Expect.equal name "@field0" "Field parameter name not correct" Expect.equal value (Sql.string "242") "Parameter value not correct" } test "succeeds when multiple independent parameters are added" { - let paramList = addFieldParams [ Field.EQ "me" "you"; Field.GT "us" "them" ] [ idParam 14 ] + let paramList = addFieldParams [ Field.Equal "me" "you"; Field.Greater "us" "them" ] [ idParam 14 ] Expect.hasLength paramList 3 "There should have been 2 parameters added" let p = Array.ofSeq paramList Expect.equal (fst p[0]) "@id" "First field parameter name not correct" @@ -101,11 +101,11 @@ let parametersTests = testList "Parameters" [ Expect.equal (snd p[2]) (Sql.string "them") "Third parameter value not correct" } test "succeeds when a parameter is not added" { - let paramList = addFieldParams [ Field.EX "tacos" ] [] + let paramList = addFieldParams [ Field.Exists "tacos" ] [] Expect.isEmpty paramList "There should not have been any parameters added" } test "succeeds when two parameters are added for one field" { - let paramList = addFieldParams [ { Field.BT "that" "eh" "zed" with ParameterName = Some "@test" } ] [] + let paramList = addFieldParams [ { Field.Between "that" "eh" "zed" with ParameterName = Some "@test" } ] [] Expect.hasLength paramList 2 "There should have been 2 parameters added" let name, value = Seq.head paramList Expect.equal name "@testmin" "Minimum field name not correct" @@ -137,55 +137,55 @@ let queryTests = testList "Query" [ testList "whereByFields" [ test "succeeds for a single field when a logical operator is passed" { Expect.equal - (Query.whereByFields Any [ { Field.GT "theField" "0" with ParameterName = Some "@test" } ]) + (Query.whereByFields Any [ { Field.Greater "theField" "0" with ParameterName = Some "@test" } ]) "data->>'theField' > @test" "WHERE clause not correct" } test "succeeds for a single field when an existence operator is passed" { Expect.equal - (Query.whereByFields Any [ Field.NEX "thatField" ]) + (Query.whereByFields Any [ Field.NotExists "thatField" ]) "data->>'thatField' IS NULL" "WHERE clause not correct" } test "succeeds for a single field when a between operator is passed with numeric values" { Expect.equal - (Query.whereByFields All [ { Field.BT "aField" 50 99 with ParameterName = Some "@range" } ]) + (Query.whereByFields All [ { Field.Between "aField" 50 99 with ParameterName = Some "@range" } ]) "(data->>'aField')::numeric BETWEEN @rangemin AND @rangemax" "WHERE clause not correct" } test "succeeds for a single field when a between operator is passed with non-numeric values" { Expect.equal - (Query.whereByFields Any [ { Field.BT "field0" "a" "b" with ParameterName = Some "@alpha" } ]) + (Query.whereByFields Any [ { Field.Between "field0" "a" "b" with ParameterName = Some "@alpha" } ]) "data->>'field0' BETWEEN @alphamin AND @alphamax" "WHERE clause not correct" } test "succeeds for all multiple fields with logical operators" { Expect.equal - (Query.whereByFields All [ Field.EQ "theFirst" "1"; Field.EQ "numberTwo" "2" ]) + (Query.whereByFields All [ Field.Equal "theFirst" "1"; Field.Equal "numberTwo" "2" ]) "data->>'theFirst' = @field0 AND data->>'numberTwo' = @field1" "WHERE clause not correct" } test "succeeds for any multiple fields with an existence operator" { Expect.equal - (Query.whereByFields Any [ Field.NEX "thatField"; Field.GE "thisField" 18 ]) + (Query.whereByFields Any [ Field.NotExists "thatField"; Field.GreaterOrEqual "thisField" 18 ]) "data->>'thatField' IS NULL OR (data->>'thisField')::numeric >= @field0" "WHERE clause not correct" } test "succeeds for all multiple fields with between operators" { Expect.equal - (Query.whereByFields All [ Field.BT "aField" 50 99; Field.BT "anotherField" "a" "b" ]) + (Query.whereByFields All [ Field.Between "aField" 50 99; Field.Between "anotherField" "a" "b" ]) "(data->>'aField')::numeric BETWEEN @field0min AND @field0max AND data->>'anotherField' BETWEEN @field1min AND @field1max" "WHERE clause not correct" } test "succeeds for a field with an IN operator with alphanumeric values" { Expect.equal - (Query.whereByFields All [ Field.IN "this" [ "a"; "b"; "c" ] ]) + (Query.whereByFields All [ Field.In "this" [ "a"; "b"; "c" ] ]) "data->>'this' IN (@field0_0, @field0_1, @field0_2)" "WHERE clause not correct" } test "succeeds for a field with an IN operator with numeric values" { Expect.equal - (Query.whereByFields All [ Field.IN "this" [ 7; 14; 21 ] ]) + (Query.whereByFields All [ Field.In "this" [ 7; 14; 21 ] ]) "(data->>'this')::numeric IN (@field0_0, @field0_1, @field0_2)" "WHERE clause not correct" } @@ -246,7 +246,7 @@ let queryTests = testList "Query" [ } test "byFields succeeds" { Expect.equal - (Query.byFields "unit" Any [ Field.GT "That" 14 ]) + (Query.byFields "unit" Any [ Field.Greater "That" 14 ]) "unit WHERE (data->>'That')::numeric > @field0" "By-Field query not correct" } @@ -544,14 +544,15 @@ let countTests = testList "Count" [ use db = PostgresDb.BuildDb() do! loadDocs () - let! theCount = Count.byFields PostgresDb.TableName Any [ Field.BT "NumValue" 15 20; Field.EQ "NumValue" 0 ] + let! theCount = + Count.byFields PostgresDb.TableName Any [ Field.Between "NumValue" 15 20; Field.Equal "NumValue" 0 ] Expect.equal theCount 3 "There should have been 3 matching documents" } testTask "succeeds when items are not found" { use db = PostgresDb.BuildDb() do! loadDocs () - let! theCount = Count.byFields PostgresDb.TableName All [ Field.EX "Sub"; Field.GT "NumValue" 100 ] + let! theCount = Count.byFields PostgresDb.TableName All [ Field.Exists "Sub"; Field.Greater "NumValue" 100 ] Expect.equal theCount 0 "There should have been no matching documents" } ] @@ -594,14 +595,14 @@ let existsTests = testList "Exists" [ use db = PostgresDb.BuildDb() do! loadDocs () - let! exists = Exists.byFields PostgresDb.TableName Any [ Field.EX "Sub"; Field.EX "Boo" ] + let! exists = Exists.byFields PostgresDb.TableName Any [ Field.Exists "Sub"; Field.Exists "Boo" ] Expect.isTrue exists "There should have been existing documents" } testTask "succeeds when documents do not exist" { use db = PostgresDb.BuildDb() do! loadDocs () - let! exists = Exists.byFields PostgresDb.TableName All [ Field.EQ "NumValue" "six"; Field.EX "Nope" ] + let! exists = Exists.byFields PostgresDb.TableName All [ Field.Equal "NumValue" "six"; Field.Exists "Nope" ] Expect.isFalse exists "There should not have been existing documents" } ] @@ -720,14 +721,14 @@ let findTests = testList "Find" [ let! docs = Find.byFields - PostgresDb.TableName All [ Field.IN "Value" [ "purple"; "blue" ]; Field.EX "Sub" ] + PostgresDb.TableName All [ Field.In "Value" [ "purple"; "blue" ]; Field.Exists "Sub" ] Expect.hasLength docs 1 "There should have been one document returned" } testTask "succeeds when documents are found using IN with numeric field" { use db = PostgresDb.BuildDb() do! loadDocs () - let! docs = Find.byFields PostgresDb.TableName All [ Field.IN "NumValue" [ 2; 4; 6; 8 ] ] + let! docs = Find.byFields PostgresDb.TableName All [ Field.In "NumValue" [ 2; 4; 6; 8 ] ] Expect.hasLength docs 1 "There should have been one document returned" } testTask "succeeds when documents are not found" { @@ -736,7 +737,7 @@ let findTests = testList "Find" [ let! docs = Find.byFields - PostgresDb.TableName All [ Field.EQ "Value" "mauve"; Field.NE "NumValue" 40 ] + PostgresDb.TableName All [ Field.Equal "Value" "mauve"; Field.NotEqual "NumValue" 40 ] Expect.isEmpty docs "There should have been no documents returned" } ] @@ -747,7 +748,7 @@ let findTests = testList "Find" [ let! docs = Find.byFieldsOrdered - PostgresDb.TableName All [ Field.EQ "Value" "purple" ] [ Field.Named "Id" ] + PostgresDb.TableName All [ Field.Equal "Value" "purple" ] [ Field.Named "Id" ] Expect.hasLength docs 2 "There should have been two documents returned" Expect.equal (docs |> List.map _.Id |> String.concat "|") "five|four" "Documents not ordered correctly" @@ -758,7 +759,7 @@ let findTests = testList "Find" [ let! docs = Find.byFieldsOrdered - PostgresDb.TableName All [ Field.EQ "Value" "purple" ] [ Field.Named "Id DESC" ] + PostgresDb.TableName All [ Field.Equal "Value" "purple" ] [ Field.Named "Id DESC" ] Expect.hasLength docs 2 "There should have been two documents returned" Expect.equal (docs |> List.map _.Id |> String.concat "|") "four|five" "Documents not ordered correctly" @@ -851,7 +852,7 @@ let findTests = testList "Find" [ use db = PostgresDb.BuildDb() do! loadDocs () - let! doc = Find.firstByFields PostgresDb.TableName Any [ Field.EQ "Value" "another" ] + let! doc = Find.firstByFields PostgresDb.TableName Any [ Field.Equal "Value" "another" ] Expect.isSome doc "There should have been a document returned" Expect.equal doc.Value.Id "two" "The incorrect document was returned" } @@ -859,7 +860,7 @@ let findTests = testList "Find" [ use db = PostgresDb.BuildDb() do! loadDocs () - let! doc = Find.firstByFields PostgresDb.TableName Any [ Field.EQ "Value" "purple" ] + let! doc = Find.firstByFields PostgresDb.TableName Any [ Field.Equal "Value" "purple" ] Expect.isSome doc "There should have been a document returned" Expect.contains [ "five"; "four" ] doc.Value.Id "An incorrect document was returned" } @@ -867,7 +868,7 @@ let findTests = testList "Find" [ use db = PostgresDb.BuildDb() do! loadDocs () - let! doc = Find.firstByFields PostgresDb.TableName Any [ Field.EQ "Value" "absent" ] + let! doc = Find.firstByFields PostgresDb.TableName Any [ Field.Equal "Value" "absent" ] Expect.isNone doc "There should not have been a document returned" } ] @@ -878,7 +879,7 @@ let findTests = testList "Find" [ let! doc = Find.firstByFieldsOrdered - PostgresDb.TableName Any [ Field.EQ "Value" "purple" ] [ Field.Named "Id" ] + PostgresDb.TableName Any [ Field.Equal "Value" "purple" ] [ Field.Named "Id" ] Expect.isSome doc "There should have been a document returned" Expect.equal "five" doc.Value.Id "An incorrect document was returned" } @@ -888,7 +889,7 @@ let findTests = testList "Find" [ let! doc = Find.firstByFieldsOrdered - PostgresDb.TableName Any [ Field.EQ "Value" "purple" ] [ Field.Named "Id DESC" ] + PostgresDb.TableName Any [ Field.Equal "Value" "purple" ] [ Field.Named "Id DESC" ] Expect.isSome doc "There should have been a document returned" Expect.equal "four" doc.Value.Id "An incorrect document was returned" } @@ -1065,8 +1066,8 @@ let patchTests = testList "Patch" [ use db = PostgresDb.BuildDb() do! loadDocs () - do! Patch.byFields PostgresDb.TableName Any [ Field.EQ "Value" "purple" ] {| NumValue = 77 |} - let! after = Count.byFields PostgresDb.TableName Any [ Field.EQ "NumValue" 77 ] + do! Patch.byFields PostgresDb.TableName Any [ Field.Equal "Value" "purple" ] {| NumValue = 77 |} + let! after = Count.byFields PostgresDb.TableName Any [ Field.Equal "NumValue" 77 ] Expect.equal after 2 "There should have been 2 documents returned" } testTask "succeeds when no document is updated" { @@ -1076,7 +1077,7 @@ let patchTests = testList "Patch" [ Expect.equal before 0 "There should have been no documents returned" // This not raising an exception is the test - do! Patch.byFields PostgresDb.TableName Any [ Field.EQ "Value" "burgundy" ] {| Foo = "green" |} + do! Patch.byFields PostgresDb.TableName Any [ Field.Equal "Value" "burgundy" ] {| Foo = "green" |} } ] testList "byContains" [ @@ -1127,9 +1128,9 @@ let removeFieldsTests = testList "RemoveFields" [ do! loadDocs () do! RemoveFields.byId PostgresDb.TableName "two" [ "Sub"; "Value" ] - let! noSubs = Count.byFields PostgresDb.TableName Any [ Field.NEX "Sub" ] + let! noSubs = Count.byFields PostgresDb.TableName Any [ Field.NotExists "Sub" ] Expect.equal noSubs 4 "There should now be 4 documents without Sub fields" - let! noValue = Count.byFields PostgresDb.TableName Any [ Field.NEX "Value" ] + let! noValue = Count.byFields PostgresDb.TableName Any [ Field.NotExists "Value" ] Expect.equal noValue 1 "There should be 1 document without Value fields" } testTask "succeeds when a single field is removed" { @@ -1137,9 +1138,9 @@ let removeFieldsTests = testList "RemoveFields" [ do! loadDocs () do! RemoveFields.byId PostgresDb.TableName "two" [ "Sub" ] - let! noSubs = Count.byFields PostgresDb.TableName Any [ Field.NEX "Sub" ] + let! noSubs = Count.byFields PostgresDb.TableName Any [ Field.NotExists "Sub" ] Expect.equal noSubs 4 "There should now be 4 documents without Sub fields" - let! noValue = Count.byFields PostgresDb.TableName Any [ Field.NEX "Value" ] + let! noValue = Count.byFields PostgresDb.TableName Any [ Field.NotExists "Value" ] Expect.equal noValue 0 "There should be no documents without Value fields" } testTask "succeeds when a field is not removed" { @@ -1161,20 +1162,20 @@ let removeFieldsTests = testList "RemoveFields" [ use db = PostgresDb.BuildDb() do! loadDocs () - do! RemoveFields.byFields PostgresDb.TableName Any [ Field.EQ "NumValue" "17" ] [ "Sub"; "Value" ] - let! noSubs = Count.byFields PostgresDb.TableName Any [ Field.NEX "Sub" ] + do! RemoveFields.byFields PostgresDb.TableName Any [ Field.Equal "NumValue" 17 ] [ "Sub"; "Value" ] + let! noSubs = Count.byFields PostgresDb.TableName Any [ Field.NotExists "Sub" ] Expect.equal noSubs 4 "There should now be 4 documents without Sub fields" - let! noValue = Count.byFields PostgresDb.TableName Any [ Field.NEX "Value" ] + let! noValue = Count.byFields PostgresDb.TableName Any [ Field.NotExists "Value" ] Expect.equal noValue 1 "There should be 1 document without Value fields" } testTask "succeeds when a single field is removed" { use db = PostgresDb.BuildDb() do! loadDocs () - do! RemoveFields.byFields PostgresDb.TableName Any [ Field.EQ "NumValue" "17" ] [ "Sub" ] - let! noSubs = Count.byFields PostgresDb.TableName Any [ Field.NEX "Sub" ] + do! RemoveFields.byFields PostgresDb.TableName Any [ Field.Equal "NumValue" 17 ] [ "Sub" ] + let! noSubs = Count.byFields PostgresDb.TableName Any [ Field.NotExists "Sub" ] Expect.equal noSubs 4 "There should now be 4 documents without Sub fields" - let! noValue = Count.byFields PostgresDb.TableName Any [ Field.NEX "Value" ] + let! noValue = Count.byFields PostgresDb.TableName Any [ Field.NotExists "Value" ] Expect.equal noValue 0 "There should be no documents without Value fields" } testTask "succeeds when a field is not removed" { @@ -1182,13 +1183,13 @@ let removeFieldsTests = testList "RemoveFields" [ do! loadDocs () // This not raising an exception is the test - do! RemoveFields.byFields PostgresDb.TableName Any [ Field.EQ "NumValue" "17" ] [ "Nothing" ] + do! RemoveFields.byFields PostgresDb.TableName Any [ Field.Equal "NumValue" 17 ] [ "Nothing" ] } testTask "succeeds when no document is matched" { use db = PostgresDb.BuildDb() // This not raising an exception is the test - do! RemoveFields.byFields PostgresDb.TableName Any [ Field.NE "Abracadabra" "apple" ] [ "Value" ] + do! RemoveFields.byFields PostgresDb.TableName Any [ Field.NotEqual "Abracadabra" "apple" ] [ "Value" ] } ] testList "byContains" [ @@ -1197,9 +1198,9 @@ let removeFieldsTests = testList "RemoveFields" [ do! loadDocs () do! RemoveFields.byContains PostgresDb.TableName {| NumValue = 17 |} [ "Sub"; "Value" ] - let! noSubs = Count.byFields PostgresDb.TableName Any [ Field.NEX "Sub" ] + let! noSubs = Count.byFields PostgresDb.TableName Any [ Field.NotExists "Sub" ] Expect.equal noSubs 4 "There should now be 4 documents without Sub fields" - let! noValue = Count.byFields PostgresDb.TableName Any [ Field.NEX "Value" ] + let! noValue = Count.byFields PostgresDb.TableName Any [ Field.NotExists "Value" ] Expect.equal noValue 1 "There should be 1 document without Value fields" } testTask "succeeds when a single field is removed" { @@ -1207,9 +1208,9 @@ let removeFieldsTests = testList "RemoveFields" [ do! loadDocs () do! RemoveFields.byContains PostgresDb.TableName {| NumValue = 17 |} [ "Sub" ] - let! noSubs = Count.byFields PostgresDb.TableName Any [ Field.NEX "Sub" ] + let! noSubs = Count.byFields PostgresDb.TableName Any [ Field.NotExists "Sub" ] Expect.equal noSubs 4 "There should now be 4 documents without Sub fields" - let! noValue = Count.byFields PostgresDb.TableName Any [ Field.NEX "Value" ] + let! noValue = Count.byFields PostgresDb.TableName Any [ Field.NotExists "Value" ] Expect.equal noValue 0 "There should be no documents without Value fields" } testTask "succeeds when a field is not removed" { @@ -1232,9 +1233,9 @@ let removeFieldsTests = testList "RemoveFields" [ do! loadDocs () do! RemoveFields.byJsonPath PostgresDb.TableName "$.NumValue ? (@ == 17)" [ "Sub"; "Value" ] - let! noSubs = Count.byFields PostgresDb.TableName Any [ Field.NEX "Sub" ] + let! noSubs = Count.byFields PostgresDb.TableName Any [ Field.NotExists "Sub" ] Expect.equal noSubs 4 "There should now be 4 documents without Sub fields" - let! noValue = Count.byFields PostgresDb.TableName Any [ Field.NEX "Value" ] + let! noValue = Count.byFields PostgresDb.TableName Any [ Field.NotExists "Value" ] Expect.equal noValue 1 "There should be 1 document without Value fields" } testTask "succeeds when a single field is removed" { @@ -1242,9 +1243,9 @@ let removeFieldsTests = testList "RemoveFields" [ do! loadDocs () do! RemoveFields.byJsonPath PostgresDb.TableName "$.NumValue ? (@ == 17)" [ "Sub" ] - let! noSubs = Count.byFields PostgresDb.TableName Any [ Field.NEX "Sub" ] + let! noSubs = Count.byFields PostgresDb.TableName Any [ Field.NotExists "Sub" ] Expect.equal noSubs 4 "There should now be 4 documents without Sub fields" - let! noValue = Count.byFields PostgresDb.TableName Any [ Field.NEX "Value" ] + let! noValue = Count.byFields PostgresDb.TableName Any [ Field.NotExists "Value" ] Expect.equal noValue 0 "There should be no documents without Value fields" } testTask "succeeds when a field is not removed" { @@ -1288,7 +1289,7 @@ let deleteTests = testList "Delete" [ use db = PostgresDb.BuildDb() do! loadDocs () - do! Delete.byFields PostgresDb.TableName Any [ Field.EQ "Value" "purple" ] + do! Delete.byFields PostgresDb.TableName Any [ Field.Equal "Value" "purple" ] let! remaining = Count.all PostgresDb.TableName Expect.equal remaining 3 "There should have been 3 documents remaining" } @@ -1296,7 +1297,7 @@ let deleteTests = testList "Delete" [ use db = PostgresDb.BuildDb() do! loadDocs () - do! Delete.byFields PostgresDb.TableName Any [ Field.EQ "Value" "crimson" ] + do! Delete.byFields PostgresDb.TableName Any [ Field.Equal "Value" "crimson" ] let! remaining = Count.all PostgresDb.TableName Expect.equal remaining 5 "There should have been 5 documents remaining" } diff --git a/src/Tests/SqliteExtensionTests.fs b/src/Tests/SqliteExtensionTests.fs index 51d4f39..8f0380e 100644 --- a/src/Tests/SqliteExtensionTests.fs +++ b/src/Tests/SqliteExtensionTests.fs @@ -118,7 +118,7 @@ let integrationTests = use conn = Configuration.dbConn () do! loadDocs () - let! theCount = conn.countByFields SqliteDb.TableName Any [ Field.EQ "Value" "purple" ] + let! theCount = conn.countByFields SqliteDb.TableName Any [ Field.Equal "Value" "purple" ] Expect.equal theCount 2L "There should have been 2 matching documents" } testList "existsById" [ @@ -145,7 +145,7 @@ let integrationTests = use conn = Configuration.dbConn () do! loadDocs () - let! exists = conn.existsByFields SqliteDb.TableName Any [ Field.EQ "NumValue" 10 ] + let! exists = conn.existsByFields SqliteDb.TableName Any [ Field.Equal "NumValue" 10 ] Expect.isTrue exists "There should have been existing documents" } testTask "succeeds when no matching documents exist" { @@ -153,7 +153,7 @@ let integrationTests = use conn = Configuration.dbConn () do! loadDocs () - let! exists = conn.existsByFields SqliteDb.TableName Any [ Field.EQ "Nothing" "none" ] + let! exists = conn.existsByFields SqliteDb.TableName Any [ Field.Equal "Nothing" "none" ] Expect.isFalse exists "There should not have been any existing documents" } ] @@ -244,7 +244,7 @@ let integrationTests = use conn = Configuration.dbConn () do! loadDocs () - let! docs = conn.findByFields SqliteDb.TableName Any [ Field.EQ "Sub.Foo" "green" ] + let! docs = conn.findByFields SqliteDb.TableName Any [ Field.Equal "Sub.Foo" "green" ] Expect.hasLength docs 2 "There should have been two documents returned" } testTask "succeeds when documents are not found" { @@ -252,7 +252,7 @@ let integrationTests = use conn = Configuration.dbConn () do! loadDocs () - let! docs = conn.findByFields SqliteDb.TableName Any [ Field.EQ "Value" "mauve" ] + let! docs = conn.findByFields SqliteDb.TableName Any [ Field.Equal "Value" "mauve" ] Expect.isEmpty docs "There should have been no documents returned" } ] @@ -264,7 +264,7 @@ let integrationTests = let! docs = conn.findByFieldsOrdered - SqliteDb.TableName Any [ Field.GT "NumValue" 15 ] [ Field.Named "Id" ] + SqliteDb.TableName Any [ Field.Greater "NumValue" 15 ] [ Field.Named "Id" ] Expect.equal (docs |> List.map _.Id |> String.concat "|") "five|four" "The documents were not ordered correctly" } @@ -275,7 +275,7 @@ let integrationTests = let! docs = conn.findByFieldsOrdered - SqliteDb.TableName Any [ Field.GT "NumValue" 15 ] [ Field.Named "Id DESC" ] + SqliteDb.TableName Any [ Field.Greater "NumValue" 15 ] [ Field.Named "Id DESC" ] Expect.equal (docs |> List.map _.Id |> String.concat "|") "four|five" "The documents were not ordered correctly" } @@ -286,7 +286,7 @@ let integrationTests = use conn = Configuration.dbConn () do! loadDocs () - let! doc = conn.findFirstByFields SqliteDb.TableName Any [ Field.EQ "Value" "another" ] + let! doc = conn.findFirstByFields SqliteDb.TableName Any [ Field.Equal "Value" "another" ] Expect.isSome doc "There should have been a document returned" Expect.equal doc.Value.Id "two" "The incorrect document was returned" } @@ -295,7 +295,7 @@ let integrationTests = use conn = Configuration.dbConn () do! loadDocs () - let! doc = conn.findFirstByFields SqliteDb.TableName Any [ Field.EQ "Sub.Foo" "green" ] + let! doc = conn.findFirstByFields SqliteDb.TableName Any [ Field.Equal "Sub.Foo" "green" ] Expect.isSome doc "There should have been a document returned" Expect.contains [ "two"; "four" ] doc.Value.Id "An incorrect document was returned" } @@ -304,7 +304,7 @@ let integrationTests = use conn = Configuration.dbConn () do! loadDocs () - let! doc = conn.findFirstByFields SqliteDb.TableName Any [ Field.EQ "Value" "absent" ] + let! doc = conn.findFirstByFields SqliteDb.TableName Any [ Field.Equal "Value" "absent" ] Expect.isNone doc "There should not have been a document returned" } ] @@ -316,7 +316,7 @@ let integrationTests = let! doc = conn.findFirstByFieldsOrdered - SqliteDb.TableName Any [ Field.EQ "Sub.Foo" "green" ] [ Field.Named "Sub.Bar" ] + SqliteDb.TableName Any [ Field.Equal "Sub.Foo" "green" ] [ Field.Named "Sub.Bar" ] Expect.isSome doc "There should have been a document returned" Expect.equal "two" doc.Value.Id "An incorrect document was returned" } @@ -327,7 +327,7 @@ let integrationTests = let! doc = conn.findFirstByFieldsOrdered - SqliteDb.TableName Any [ Field.EQ "Sub.Foo" "green" ] [ Field.Named "Sub.Bar DESC" ] + SqliteDb.TableName Any [ Field.Equal "Sub.Foo" "green" ] [ Field.Named "Sub.Bar DESC" ] Expect.isSome doc "There should have been a document returned" Expect.equal "four" doc.Value.Id "An incorrect document was returned" } @@ -416,8 +416,8 @@ let integrationTests = use conn = Configuration.dbConn () do! loadDocs () - do! conn.patchByFields SqliteDb.TableName Any [ Field.EQ "Value" "purple" ] {| NumValue = 77 |} - let! after = conn.countByFields SqliteDb.TableName Any [ Field.EQ "NumValue" 77 ] + do! conn.patchByFields SqliteDb.TableName Any [ Field.Equal "Value" "purple" ] {| NumValue = 77 |} + let! after = conn.countByFields SqliteDb.TableName Any [ Field.Equal "NumValue" 77 ] Expect.equal after 2L "There should have been 2 documents returned" } testTask "succeeds when no document is updated" { @@ -428,7 +428,7 @@ let integrationTests = Expect.isEmpty before "There should have been no documents returned" // This not raising an exception is the test - do! conn.patchByFields SqliteDb.TableName Any [ Field.EQ "Value" "burgundy" ] {| Foo = "green" |} + do! conn.patchByFields SqliteDb.TableName Any [ Field.Equal "Value" "burgundy" ] {| Foo = "green" |} } ] testList "removeFieldsById" [ @@ -467,7 +467,7 @@ let integrationTests = use conn = Configuration.dbConn () do! loadDocs () - do! conn.removeFieldsByFields SqliteDb.TableName Any [ Field.EQ "NumValue" 17 ] [ "Sub" ] + do! conn.removeFieldsByFields SqliteDb.TableName Any [ Field.Equal "NumValue" 17 ] [ "Sub" ] try let! _ = conn.findById SqliteDb.TableName "four" Expect.isTrue false "The updated document should have failed to parse" @@ -481,14 +481,15 @@ let integrationTests = do! loadDocs () // This not raising an exception is the test - do! conn.removeFieldsByFields SqliteDb.TableName Any [ Field.EQ "NumValue" 17 ] [ "Nothing" ] + do! conn.removeFieldsByFields SqliteDb.TableName Any [ Field.Equal "NumValue" 17 ] [ "Nothing" ] } testTask "succeeds when no document is matched" { use! db = SqliteDb.BuildDb() use conn = Configuration.dbConn () // This not raising an exception is the test - do! conn.removeFieldsByFields SqliteDb.TableName Any [ Field.NE "Abracadabra" "apple" ] [ "Value" ] + do! conn.removeFieldsByFields + SqliteDb.TableName Any [ Field.NotEqual "Abracadabra" "apple" ] [ "Value" ] } ] testList "deleteById" [ @@ -517,7 +518,7 @@ let integrationTests = use conn = Configuration.dbConn () do! loadDocs () - do! conn.deleteByFields SqliteDb.TableName Any [ Field.NE "Value" "purple" ] + do! conn.deleteByFields SqliteDb.TableName Any [ Field.NotEqual "Value" "purple" ] let! remaining = conn.countAll SqliteDb.TableName Expect.equal remaining 2L "There should have been 2 documents remaining" } @@ -526,7 +527,7 @@ let integrationTests = use conn = Configuration.dbConn () do! loadDocs () - do! conn.deleteByFields SqliteDb.TableName Any [ Field.EQ "Value" "crimson" ] + do! conn.deleteByFields SqliteDb.TableName Any [ Field.Equal "Value" "crimson" ] let! remaining = conn.countAll SqliteDb.TableName Expect.equal remaining 5L "There should have been 5 documents remaining" } diff --git a/src/Tests/SqliteTests.fs b/src/Tests/SqliteTests.fs index de02b9f..c2139af 100644 --- a/src/Tests/SqliteTests.fs +++ b/src/Tests/SqliteTests.fs @@ -17,43 +17,43 @@ let queryTests = testList "Query" [ testList "whereByFields" [ test "succeeds for a single field when a logical operator is passed" { Expect.equal - (Query.whereByFields Any [ { Field.GT "theField" 0 with ParameterName = Some "@test" } ]) + (Query.whereByFields Any [ { Field.Greater "theField" 0 with ParameterName = Some "@test" } ]) "data->>'theField' > @test" "WHERE clause not correct" } test "succeeds for a single field when an existence operator is passed" { Expect.equal - (Query.whereByFields Any [ Field.NEX "thatField" ]) + (Query.whereByFields Any [ Field.NotExists "thatField" ]) "data->>'thatField' IS NULL" "WHERE clause not correct" } test "succeeds for a single field when a between operator is passed" { Expect.equal - (Query.whereByFields All [ { Field.BT "aField" 50 99 with ParameterName = Some "@range" } ]) + (Query.whereByFields All [ { Field.Between "aField" 50 99 with ParameterName = Some "@range" } ]) "data->>'aField' BETWEEN @rangemin AND @rangemax" "WHERE clause not correct" } test "succeeds for all multiple fields with logical operators" { Expect.equal - (Query.whereByFields All [ Field.EQ "theFirst" "1"; Field.EQ "numberTwo" "2" ]) + (Query.whereByFields All [ Field.Equal "theFirst" "1"; Field.Equal "numberTwo" "2" ]) "data->>'theFirst' = @field0 AND data->>'numberTwo' = @field1" "WHERE clause not correct" } test "succeeds for any multiple fields with an existence operator" { Expect.equal - (Query.whereByFields Any [ Field.NEX "thatField"; Field.GE "thisField" 18 ]) + (Query.whereByFields Any [ Field.NotExists "thatField"; Field.GreaterOrEqual "thisField" 18 ]) "data->>'thatField' IS NULL OR data->>'thisField' >= @field0" "WHERE clause not correct" } test "succeeds for all multiple fields with between operators" { Expect.equal - (Query.whereByFields All [ Field.BT "aField" 50 99; Field.BT "anotherField" "a" "b" ]) + (Query.whereByFields All [ Field.Between "aField" 50 99; Field.Between "anotherField" "a" "b" ]) "data->>'aField' BETWEEN @field0min AND @field0max AND data->>'anotherField' BETWEEN @field1min AND @field1max" "WHERE clause not correct" } test "succeeds for a field with an IN operator" { Expect.equal - (Query.whereByFields All [ Field.IN "this" [ "a"; "b"; "c" ] ]) + (Query.whereByFields All [ Field.In "this" [ "a"; "b"; "c" ] ]) "data->>'this' IN (@field0_0, @field0_1, @field0_2)" "WHERE clause not correct" } @@ -78,7 +78,7 @@ let queryTests = testList "Query" [ } test "byFields succeeds" { Expect.equal - (Query.byFields "unit" Any [ Field.GT "That" 14 ]) + (Query.byFields "unit" Any [ Field.Greater "That" 14 ]) "unit WHERE data->>'That' > @field0" "By-Field query not correct" } @@ -104,14 +104,14 @@ let parametersTests = testList "Parameters" [ } testList "addFieldParam" [ test "succeeds when adding a parameter" { - let paramList = addFieldParam "@field" (Field.EQ "it" 99) [] + let paramList = addFieldParam "@field" (Field.Equal "it" 99) [] Expect.hasLength paramList 1 "There should have been a parameter added" let theParam = Seq.head paramList Expect.equal theParam.ParameterName "@field" "The parameter name is incorrect" Expect.equal theParam.Value 99 "The parameter value is incorrect" } test "succeeds when not adding a parameter" { - let paramList = addFieldParam "@it" (Field.NEX "Coffee") [] + let paramList = addFieldParam "@it" (Field.NotExists "Coffee") [] Expect.isEmpty paramList "There should not have been any parameters added" } ] @@ -386,14 +386,14 @@ let countTests = testList "Count" [ use! db = SqliteDb.BuildDb() do! loadDocs () - let! theCount = Count.byFields SqliteDb.TableName Any [ Field.BT "NumValue" 10 20 ] + let! theCount = Count.byFields SqliteDb.TableName Any [ Field.Between "NumValue" 10 20 ] Expect.equal theCount 3L "There should have been 3 matching documents" } testTask "succeeds for a non-numeric range" { use! db = SqliteDb.BuildDb() do! loadDocs () - let! theCount = Count.byFields SqliteDb.TableName Any [ Field.BT "Value" "aardvark" "apple" ] + let! theCount = Count.byFields SqliteDb.TableName Any [ Field.Between "Value" "aardvark" "apple" ] Expect.equal theCount 1L "There should have been 1 matching document" } ] @@ -422,14 +422,14 @@ let existsTests = testList "Exists" [ use! db = SqliteDb.BuildDb() do! loadDocs () - let! exists = Exists.byFields SqliteDb.TableName Any [ Field.EQ "NumValue" 10 ] + let! exists = Exists.byFields SqliteDb.TableName Any [ Field.Equal "NumValue" 10 ] Expect.isTrue exists "There should have been existing documents" } testTask "succeeds when no matching documents exist" { use! db = SqliteDb.BuildDb() do! loadDocs () - let! exists = Exists.byFields SqliteDb.TableName Any [ Field.LT "Nothing" "none" ] + let! exists = Exists.byFields SqliteDb.TableName Any [ Field.Less "Nothing" "none" ] Expect.isFalse exists "There should not have been any existing documents" } ] @@ -514,21 +514,21 @@ let findTests = testList "Find" [ use! db = SqliteDb.BuildDb() do! loadDocs () - let! docs = Find.byFields SqliteDb.TableName Any [ Field.GT "NumValue" 15 ] + let! docs = Find.byFields SqliteDb.TableName Any [ Field.Greater "NumValue" 15 ] Expect.hasLength docs 2 "There should have been two documents returned" } testTask "succeeds when documents are found using IN with numeric field" { use! db = SqliteDb.BuildDb() do! loadDocs () - let! docs = Find.byFields SqliteDb.TableName All [ Field.IN "NumValue" [ 2; 4; 6; 8 ] ] + let! docs = Find.byFields SqliteDb.TableName All [ Field.In "NumValue" [ 2; 4; 6; 8 ] ] Expect.hasLength docs 1 "There should have been one document returned" } testTask "succeeds when documents are not found" { use! db = SqliteDb.BuildDb() do! loadDocs () - let! docs = Find.byFields SqliteDb.TableName Any [ Field.GT "NumValue" 100 ] + let! docs = Find.byFields SqliteDb.TableName Any [ Field.Greater "NumValue" 100 ] Expect.isTrue (List.isEmpty docs) "There should have been no documents returned" } ] @@ -539,7 +539,7 @@ let findTests = testList "Find" [ let! docs = Find.byFieldsOrdered - SqliteDb.TableName Any [ Field.GT "NumValue" 15 ] [ Field.Named "Id" ] + SqliteDb.TableName Any [ Field.Greater "NumValue" 15 ] [ Field.Named "Id" ] Expect.hasLength docs 2 "There should have been two documents returned" Expect.equal (docs |> List.map _.Id |> String.concat "|") "five|four" "The documents were not ordered correctly" @@ -550,7 +550,7 @@ let findTests = testList "Find" [ let! docs = Find.byFieldsOrdered - SqliteDb.TableName Any [ Field.GT "NumValue" 15 ] [ Field.Named "Id DESC" ] + SqliteDb.TableName Any [ Field.Greater "NumValue" 15 ] [ Field.Named "Id DESC" ] Expect.hasLength docs 2 "There should have been two documents returned" Expect.equal (docs |> List.map _.Id |> String.concat "|") "four|five" "The documents were not ordered correctly" @@ -561,7 +561,7 @@ let findTests = testList "Find" [ let! docs = Find.byFieldsOrdered - SqliteDb.TableName All [ Field.LE "NumValue" 10 ] [ Field.Named "Value" ] + SqliteDb.TableName All [ Field.LessOrEqual "NumValue" 10 ] [ Field.Named "Value" ] Expect.hasLength docs 3 "There should have been three documents returned" Expect.equal (docs |> List.map _.Id |> String.concat "|") "three|one|two" "Documents not ordered correctly" @@ -572,7 +572,7 @@ let findTests = testList "Find" [ let! docs = Find.byFieldsOrdered - SqliteDb.TableName All [ Field.LE "NumValue" 10 ] [ Field.Named "i:Value" ] + SqliteDb.TableName All [ Field.LessOrEqual "NumValue" 10 ] [ Field.Named "i:Value" ] Expect.hasLength docs 3 "There should have been three documents returned" Expect.equal (docs |> List.map _.Id |> String.concat "|") "three|two|one" "Documents not ordered correctly" @@ -583,7 +583,7 @@ let findTests = testList "Find" [ use! db = SqliteDb.BuildDb() do! loadDocs () - let! doc = Find.firstByFields SqliteDb.TableName Any [ Field.EQ "Value" "another" ] + let! doc = Find.firstByFields SqliteDb.TableName Any [ Field.Equal "Value" "another" ] Expect.isSome doc "There should have been a document returned" Expect.equal doc.Value.Id "two" "The incorrect document was returned" } @@ -591,7 +591,7 @@ let findTests = testList "Find" [ use! db = SqliteDb.BuildDb() do! loadDocs () - let! doc = Find.firstByFields SqliteDb.TableName Any [ Field.EQ "Sub.Foo" "green" ] + let! doc = Find.firstByFields SqliteDb.TableName Any [ Field.Equal "Sub.Foo" "green" ] Expect.isSome doc "There should have been a document returned" Expect.contains [ "two"; "four" ] doc.Value.Id "An incorrect document was returned" } @@ -599,7 +599,7 @@ let findTests = testList "Find" [ use! db = SqliteDb.BuildDb() do! loadDocs () - let! doc = Find.firstByFields SqliteDb.TableName Any [ Field.EQ "Value" "absent" ] + let! doc = Find.firstByFields SqliteDb.TableName Any [ Field.Equal "Value" "absent" ] Expect.isNone doc "There should not have been a document returned" } ] @@ -610,7 +610,7 @@ let findTests = testList "Find" [ let! doc = Find.firstByFieldsOrdered - SqliteDb.TableName Any [ Field.EQ "Sub.Foo" "green" ] [ Field.Named "Sub.Bar" ] + SqliteDb.TableName Any [ Field.Equal "Sub.Foo" "green" ] [ Field.Named "Sub.Bar" ] Expect.isSome doc "There should have been a document returned" Expect.equal "two" doc.Value.Id "An incorrect document was returned" } @@ -620,7 +620,7 @@ let findTests = testList "Find" [ let! doc = Find.firstByFieldsOrdered - SqliteDb.TableName Any [ Field.EQ "Sub.Foo" "green" ] [ Field.Named "Sub.Bar DESC" ] + SqliteDb.TableName Any [ Field.Equal "Sub.Foo" "green" ] [ Field.Named "Sub.Bar DESC" ] Expect.isSome doc "There should have been a document returned" Expect.equal "four" doc.Value.Id "An incorrect document was returned" } @@ -703,8 +703,8 @@ let patchTests = testList "Patch" [ use! db = SqliteDb.BuildDb() do! loadDocs () - do! Patch.byFields SqliteDb.TableName Any [ Field.EQ "Value" "purple" ] {| NumValue = 77 |} - let! after = Count.byFields SqliteDb.TableName Any [ Field.EQ "NumValue" 77 ] + do! Patch.byFields SqliteDb.TableName Any [ Field.Equal "Value" "purple" ] {| NumValue = 77 |} + let! after = Count.byFields SqliteDb.TableName Any [ Field.Equal "NumValue" 77 ] Expect.equal after 2L "There should have been 2 documents returned" } testTask "succeeds when no document is updated" { @@ -714,7 +714,7 @@ let patchTests = testList "Patch" [ Expect.isEmpty before "There should have been no documents returned" // This not raising an exception is the test - do! Patch.byFields SqliteDb.TableName Any [ Field.EQ "Value" "burgundy" ] {| Foo = "green" |} + do! Patch.byFields SqliteDb.TableName Any [ Field.Equal "Value" "burgundy" ] {| Foo = "green" |} } ] ] @@ -753,7 +753,7 @@ let removeFieldsTests = testList "RemoveFields" [ use! db = SqliteDb.BuildDb() do! loadDocs () - do! RemoveFields.byFields SqliteDb.TableName Any [ Field.EQ "NumValue" 17 ] [ "Sub" ] + do! RemoveFields.byFields SqliteDb.TableName Any [ Field.Equal "NumValue" 17 ] [ "Sub" ] try let! _ = Find.byId SqliteDb.TableName "four" Expect.isTrue false "The updated document should have failed to parse" @@ -766,13 +766,13 @@ let removeFieldsTests = testList "RemoveFields" [ do! loadDocs () // This not raising an exception is the test - do! RemoveFields.byFields SqliteDb.TableName Any [ Field.EQ "NumValue" 17 ] [ "Nothing" ] + do! RemoveFields.byFields SqliteDb.TableName Any [ Field.Equal "NumValue" 17 ] [ "Nothing" ] } testTask "succeeds when no document is matched" { use! db = SqliteDb.BuildDb() // This not raising an exception is the test - do! RemoveFields.byFields SqliteDb.TableName Any [ Field.NE "Abracadabra" "apple" ] [ "Value" ] + do! RemoveFields.byFields SqliteDb.TableName Any [ Field.NotEqual "Abracadabra" "apple" ] [ "Value" ] } ] ] @@ -802,7 +802,7 @@ let deleteTests = testList "Delete" [ use! db = SqliteDb.BuildDb() do! loadDocs () - do! Delete.byFields SqliteDb.TableName Any [ Field.NE "Value" "purple" ] + do! Delete.byFields SqliteDb.TableName Any [ Field.NotEqual "Value" "purple" ] let! remaining = Count.all SqliteDb.TableName Expect.equal remaining 2L "There should have been 2 documents remaining" } @@ -810,7 +810,7 @@ let deleteTests = testList "Delete" [ use! db = SqliteDb.BuildDb() do! loadDocs () - do! Delete.byFields SqliteDb.TableName Any [ Field.EQ "Value" "crimson" ] + do! Delete.byFields SqliteDb.TableName Any [ Field.Equal "Value" "crimson" ] let! remaining = Count.all SqliteDb.TableName Expect.equal remaining 5L "There should have been 5 documents remaining" }