Fix mapping for char(x)

char(x) were wrongly getting scaffolded as char instead of string.

Fixes #370

Author: roji

src/EFCore.PG/Storage/Internal/NpgsqlEFTypeMapper.cs

@@ -81,6 +81,7 @@ void AddCustomizedMappings()
             // Mappings where we need literal string generation
             _baseClrMappings[typeof(string)] = _storeTypeMappings["text"] = new NpgsqlStringTypeMapping("text", NpgsqlDbType.Text);
             _storeTypeMappings["varchar"] = new NpgsqlStringTypeMapping("varchar", NpgsqlDbType.Varchar);
+            _storeTypeMappings["char"] = new NpgsqlStringTypeMapping("char", NpgsqlDbType.Char);
             _storeTypeMappings["citext"] = new NpgsqlStringTypeMapping("citext", NpgsqlDbType.Citext);
             _storeTypeMappings["json"] = new NpgsqlStringTypeMapping("json", NpgsqlDbType.Json);
             _storeTypeMappings["jsonb"] = new NpgsqlStringTypeMapping("jsonb", NpgsqlDbType.Jsonb);

test/EFCore.PG.Design.FunctionalTests/ReverseEngineering/Expected/AllFluentApi/AllDataTypes.expected

@@ -14,7 +14,7 @@ namespace E2ETest.Namespace
         public decimal DecimalColumn { get; set; }
         public decimal NumericColumn { get; set; }
         public decimal MoneyColumn { get; set; }
-        public char? CharColumn { get; set; }
+        public string CharColumn { get; set; }
         public string TextColumn { get; set; }
         public string VarcharColumn { get; set; }
         public DateTime DateColumn { get; set; }

test/EFCore.PG.Design.FunctionalTests/ReverseEngineering/Expected/Attributes/AllDataTypes.expected

@@ -26,7 +26,7 @@ namespace E2ETest.Namespace
         [Column("moneyColumn", TypeName = "money")]
         public decimal MoneyColumn { get; set; }
         [Column("charColumn", TypeName = "char(1)")]
-        public char? CharColumn { get; set; }
+        public string CharColumn { get; set; }
         [Column("textColumn")]
         public string TextColumn { get; set; }
         [Column("varcharColumn", TypeName = "varchar")]