Tweaks from review

Author: ajcvickers

src/EFCore.Relational/Metadata/Conventions/RelationalMapToJsonConvention.cs

@@ -59,7 +59,7 @@ public virtual void ProcessModelFinalizing(
                          .Where(p => p.ClrType.UnwrapNullableType().IsEnum))
             {
                 // If the enum is mapped with no conversion, then use the reader/writer that handles legacy string values and warns.
-                if (enumProperty.GetValueConverter()?.ProviderClrType == null
+                if (enumProperty.GetValueConverter() == null
                     && enumProperty.GetProviderClrType() == null)
                 {
                     enumProperty.SetJsonValueReaderWriterType(

test/EFCore.Relational.Specification.Tests/Query/JsonQueryAdHocTestBase.cs

@@ -704,123 +704,6 @@ public MyJsonEntityShadowPropertiesWithCtor(string name)
 
     #endregion
 
-    //#region EnumLegacyValues
-
-    //[ConditionalTheory]
-    //[MemberData(nameof(IsAsyncData))]
-    //public virtual async Task Read_enum_property_with_legacy_values(bool async)
-    //{
-    //    var contextFactory = await InitializeAsync<MyContextEnumLegacyValues>(
-    //        seed: SeedEnumLegacyValues);
-
-    //    using (var context = contextFactory.CreateContext())
-    //    {
-    //        var query = context.Entities.Select(x => new
-    //        {
-    //            x.Reference.IntEnum,
-    //            x.Reference.ByteEnum,
-    //            x.Reference.LongEnum,
-    //            x.Reference.NullableEnum
-    //        });
-
-    //        var result = async
-    //            ? await query.ToListAsync()
-    //            : query.ToList();
-
-    //        Assert.Equal(1, result.Count);
-    //        //Assert.Equal("Foo", result[0].ShadowString);
-    //        //Assert.Equal(143, result[0].ShadowInt);
-    //    }
-    //}
-
-    //[ConditionalTheory]
-    //[MemberData(nameof(IsAsyncData))]
-    //public virtual async Task Read_json_entity_with_enum_properties_with_legacy_values(bool async)
-    //{
-    //    var contextFactory = await InitializeAsync<MyContextEnumLegacyValues>(
-    //        seed: SeedEnumLegacyValues);
-
-    //    using (var context = contextFactory.CreateContext())
-    //    {
-    //        var query = context.Entities.Select(x => x.Reference).AsNoTracking();
-
-    //        var result = async
-    //            ? await query.ToListAsync()
-    //            : query.ToList();
-
-    //        Assert.Equal(1, result.Count);
-    //        //Assert.Equal("Foo", result[0].ShadowString);
-    //        //Assert.Equal(143, result[0].ShadowInt);
-    //    }
-    //}
-
-    //protected abstract void SeedEnumLegacyValues(MyContextEnumLegacyValues ctx);
-
-    //protected class MyContextEnumLegacyValues : DbContext
-    //{
-    //    public MyContextEnumLegacyValues(DbContextOptions options)
-    //        : base(options)
-    //    {
-    //    }
-
-    //    public DbSet<MyEntityEnumLegacyValues> Entities { get; set; }
-
-    //    protected override void OnModelCreating(ModelBuilder modelBuilder)
-    //    {
-    //        modelBuilder.Entity<MyEntityEnumLegacyValues>().Property(x => x.Id).ValueGeneratedNever();
-    //        modelBuilder.Entity<MyEntityEnumLegacyValues>().OwnsOne(x => x.Reference, b =>
-    //        {
-    //            b.ToJson();
-    //        });
-    //        modelBuilder.Entity<MyEntityEnumLegacyValues>().OwnsMany(x => x.Collection, b =>
-    //        {
-    //            b.ToJson();
-    //        });
-    //    }
-    //}
-
-    //public class MyEntityEnumLegacyValues
-    //{
-    //    public int Id { get; set; }
-    //    public string Name { get; set; }
-
-    //    public MyJsonEntityEnumLegacyValues Reference { get; set; }
-    //    public List<MyJsonEntityEnumLegacyValues> Collection { get; set; }
-    //}
-
-    //public class MyJsonEntityEnumLegacyValues
-    //{
-    //    public string Name { get; set; }
-
-    //    public IntEnumLegacyValues IntEnum { get; set; }
-    //    public ByteEnumLegacyValues ByteEnum { get; set; }
-    //    public LongEnumLegacyValues LongEnum { get; set; }
-    //    public IntEnumLegacyValues? NullableEnum { get; set; }
-    //}
-
-    //public enum IntEnumLegacyValues
-    //{
-    //    Foo,
-    //    Bar,
-    //    Baz,
-    //}
-
-    //public enum ByteEnumLegacyValues : byte
-    //{
-    //    Seattle,
-    //    Redmond,
-    //    Bellevue,
-    //}
-
-    //public enum LongEnumLegacyValues : long
-    //{
-    //    One,
-    //    Two,
-    //    Three,
-    //}
-
-    //#endregion
-
     protected TestSqlLoggerFactory TestSqlLoggerFactory
         => (TestSqlLoggerFactory)ListLoggerFactory;
 }