Add missing type mapping for timetz

OffsetTime wasn't properly mapped to timetz
npgsql · May 21, 2018 · fe483cd · fe483cd
1 parent 55a22ba
commit fe483cd
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/Npgsql.NodaTime/NpgsqlNodatimeExtensions.cs b/src/Npgsql.NodaTime/NpgsqlNodatimeExtensions.cs
@@ -77,6 +77,7 @@ public static INpgsqlTypeMapper UseNodatime(this INpgsqlTypeMapper mapper)
                 {
                     PgTypeName = "timetz",
                     NpgsqlDbType = NpgsqlDbType.TimeTz,
+                    ClrTypes = new[] { typeof(OffsetTime) },
                     TypeHandlerFactory = new TimeTzHandlerFactory()
                 }.Build())
                 .AddMapping(new NpgsqlTypeMappingBuilder

diff --git a/test/Npgsql.PluginTests/NodaTimeTests.cs b/test/Npgsql.PluginTests/NodaTimeTests.cs
@@ -310,9 +310,10 @@ public void TimeTz()
                 var offset = Offset.FromHoursAndMinutes(3, 30) + Offset.FromSeconds(5);
                 var expected = new OffsetTime(time, offset);
 
-                using (var cmd = new NpgsqlCommand("SELECT @p", conn))
+                using (var cmd = new NpgsqlCommand("SELECT @p1, @p2", conn))
                 {
-                    cmd.Parameters.Add(new NpgsqlParameter("p", NpgsqlDbType.TimeTz) { Value = expected });
+                    cmd.Parameters.Add(new NpgsqlParameter("p1", NpgsqlDbType.TimeTz) { Value = expected });
+                    cmd.Parameters.Add(new NpgsqlParameter { ParameterName = "p2", Value = expected });
                     using (var reader = cmd.ExecuteReader())
                     {
                         reader.Read();