Remove ColumnProperty.Null if primary key is set.

src/Migrator.Tests/Providers/SQLServer/SQLServerTransformationProvider_GetColumns_DefaultValues_Tests.cs

@@ -33,6 +33,7 @@ public void GetColumns_DefaultValues_Succeeds()
         const string decimalColumnName1 = "decimalcolumn";
         const string guidColumnName1 = "guidcolumn1";
         const string booleanColumnName1 = "booleancolumn1";
+        const string booleanColumnName2 = "booleancolumn2";
         const string int32ColumnName1 = "int32column1";
         const string int64ColumnName1 = "int64column1";
         const string int64ColumnName2 = "int64column2";
@@ -50,6 +51,7 @@ public void GetColumns_DefaultValues_Succeeds()
 
             // other boolean default values are tested in another test
             new Column(booleanColumnName1, DbType.Boolean, true),
+            new Column(booleanColumnName2, DbType.Boolean, false),
 
             new Column(int32ColumnName1, DbType.Int32, defaultValue: 43),
             new Column(int64ColumnName1, DbType.Int64, defaultValue: 88),
@@ -69,6 +71,7 @@ public void GetColumns_DefaultValues_Succeeds()
         var decimalColumn1 = columns.Single(x => x.Name.Equals(decimalColumnName1, StringComparison.OrdinalIgnoreCase));
         var guidColumn1 = columns.Single(x => x.Name.Equals(guidColumnName1, StringComparison.OrdinalIgnoreCase));
         var booleanColumn1 = columns.Single(x => x.Name.Equals(booleanColumnName1, StringComparison.OrdinalIgnoreCase));
+        var booleanColumn2 = columns.Single(x => x.Name.Equals(booleanColumnName2, StringComparison.OrdinalIgnoreCase));
         var int32Column1 = columns.Single(x => x.Name.Equals(int32ColumnName1, StringComparison.OrdinalIgnoreCase));
         var int64Column1 = columns.Single(x => x.Name.Equals(int64ColumnName1, StringComparison.OrdinalIgnoreCase));
         var int64Column2 = columns.Single(x => x.Name.Equals(int64ColumnName2, StringComparison.OrdinalIgnoreCase));
@@ -82,6 +85,7 @@ public void GetColumns_DefaultValues_Succeeds()
         Assert.That(decimalColumn1.DefaultValue, Is.EqualTo(decimalDefaultValue));
         Assert.That(guidColumn1.DefaultValue, Is.EqualTo(guidDefaultValue));
         Assert.That(booleanColumn1.DefaultValue, Is.True);
+        Assert.That(booleanColumn2.DefaultValue, Is.False);
         Assert.That(int32Column1.DefaultValue, Is.EqualTo(43));
         Assert.That(int64Column1.DefaultValue, Is.EqualTo(88));
         Assert.That(stringColumn1.DefaultValue, Is.EqualTo("Hello"));

src/Migrator.Tests/Providers/SQLite/SQLiteTransformationProvider_AddColumnTests.cs

@@ -58,4 +58,40 @@ public void AddColumn_HavingColumnPropertyUniqueAndIndex_RebuildSucceeds()
         Assert.That(indexAfter.Name, Is.EqualTo(indexName));
         CollectionAssert.AreEquivalent(indexAfter.KeyColumns, new string[] { propertyName1, propertyName2 });
     }
+
+    [Test]
+    public void AddColumn_HavingNullInPrimaryKey_Succeds()
+    {
+        // Arrange/Act
+        Provider.ExecuteNonQuery("CREATE TABLE Common_Language (LanguageID TEXT PRIMARY KEY)");
+
+        Provider.AddColumn("Common_Language", "Enabled", DbType.Boolean);
+
+        var tableInfo = ((SQLiteTransformationProvider)Provider).GetSQLiteTableInfo("Common_Language");
+        var script = ((SQLiteTransformationProvider)Provider).GetSqlCreateTableScript("Common_Language");
+
+        var columnProperty = tableInfo.Columns.Single(x => x.Name == "LanguageID").ColumnProperty;
+        var hasNull = columnProperty.IsSet(ColumnProperty.Null);
+
+        // Assert        
+        Assert.That(hasNull, Is.False);
+    }
+
+    [Test]
+    public void AddColumn_HavingNotNullInPrimaryKey_Succeds()
+    {
+        // Arrange/Act
+        Provider.ExecuteNonQuery("CREATE TABLE Common_Language (LanguageID INT NOT NULL PRIMARY KEY)");
+
+        Provider.AddColumn("Common_Language", "Enabled", DbType.Boolean);
+
+        var tableInfo = ((SQLiteTransformationProvider)Provider).GetSQLiteTableInfo("Common_Language");
+        var script = ((SQLiteTransformationProvider)Provider).GetSqlCreateTableScript("Common_Language");
+
+        var columnProperty = tableInfo.Columns.Single(x => x.Name == "LanguageID").ColumnProperty;
+        var hasNull = columnProperty.IsSet(ColumnProperty.Null);
+
+        // Assert        
+        Assert.That(hasNull, Is.False);
+    }
 }

src/Migrator/Providers/Impl/SQLite/SQLiteColumnPropertiesMapper.cs

@@ -1,4 +1,3 @@
-using System;
 using System.Collections.Generic;
 using DotNetProjects.Migrator.Framework;
 
@@ -22,24 +21,8 @@
         }
     }
 
-    protected override void AddNotNull(Column column, List<string> vals)
-    {
-        var isPrimaryKeySelected = PropertySelected(column.ColumnProperty, ColumnProperty.PrimaryKey);
-        var isNullSelected = PropertySelected(column.ColumnProperty, ColumnProperty.Null);
-        var isNotNullSelected = PropertySelected(column.ColumnProperty, ColumnProperty.NotNull);
-
-        if (isNullSelected && isPrimaryKeySelected)
-        {
-            throw new Exception("This is currently not supported by the migrator see issue #44. You need to use NOT NULL for a PK column.");
-        }
-
-        if (isNotNullSelected || isPrimaryKeySelected)
-        {
-            AddValueIfSelected(column, ColumnProperty.NotNull, vals);
-        }
-    }
 
     protected virtual void AddValueIfSelected(Column column, ColumnProperty property, ICollection<string> vals)
     {
         vals.Add(_Dialect.SqlForProperty(property, column));
     }

src/Migrator/Providers/Impl/SQLite/SQLiteTransformationProvider.cs

@@ -893,7 +893,7 @@
     }
 
     [Obsolete]
     public override void AddTable(string table, string engine, string columns)
     {
         throw new NotSupportedException();
     }
@@ -1213,6 +1213,11 @@
             if (pragmaTableInfoItem.Pk > 0)
             {
                 column.ColumnProperty |= ColumnProperty.PrimaryKey;
+
+                if (column.ColumnProperty.IsSet(ColumnProperty.Null))
+                {
+                    column.ColumnProperty = column.ColumnProperty.Clear(ColumnProperty.Null);
+                }
             }
 
             var indexListItems = GetPragmaIndexListItems(tableName);