Include `NULL` constraints in node->sql

drift/test/generated/custom_tables.g.dart

@@ -115,7 +115,7 @@ class WithDefaults extends Table with TableInfo<WithDefaults, WithDefault> {
       'b', aliasedName, true,
       type: DriftSqlType.int,
       requiredDuringInsert: false,
-      $customConstraints: 'UNIQUE');
+      $customConstraints: 'UNIQUE NULL');
   @override
   List<GeneratedColumn> get $columns => [a, b];
   @override

drift/test/generated/tables.drift

@@ -7,7 +7,7 @@ CREATE TABLE no_ids (
 
 CREATE TABLE with_defaults (
     a `const CustomTextType()` JSON KEY customJsonName DEFAULT 'something',
-    b INT UNIQUE
+    b INT UNIQUE NULL
 );
 
 CREATE TABLE with_constraints (

drift/test/integration_tests/drift_files_test.dart

@@ -11,7 +11,7 @@ const _createNoIds =
     'WITHOUT ROWID;';
 
 const _createWithDefaults = 'CREATE TABLE IF NOT EXISTS "with_defaults" ('
-    "\"a\" MY_TEXT DEFAULT 'something', \"b\" INTEGER UNIQUE);";
+    "\"a\" MY_TEXT DEFAULT 'something', \"b\" INTEGER UNIQUE NULL);";
 
 const _createWithConstraints = 'CREATE TABLE IF NOT EXISTS "with_constraints" ('
     '"a" TEXT, "b" INTEGER NOT NULL, "c" REAL, '

drift_dev/CHANGELOG.md

@@ -1,3 +1,14 @@
+## 2.15.0-dev
+
+- Potentially __breaking change__: Fix a bug causing `NULL` column constraints
+  not to show up in generated table definitions.
+  This is a breaking change drift's migration tooling is used, since they will
+  not expect the `NULL` column constraint to be there.
+  After upgrading, you can either drop the `NULL` constraint from your drift
+  files to make it consistent with the schema (since `NULL` has no effect in
+  sqlite3), or increment the schema version and run a `TableMigration` on
+  affected tables to make column constraints consistent.
+
 ## 2.14.1
 
 - Fix inconsistencies when generating `Variable` instances for columns with

examples/migrations_example/lib/tables.drift

@@ -4,7 +4,7 @@ import 'tables.dart';
 CREATE TABLE "groups" (
     id INTEGER NOT NULL,
     title TEXT NOT NULL,
-    deleted BOOLEAN DEFAULT FALSE,
+    deleted BOOLEAN NULL DEFAULT FALSE,
     owner INTEGER NOT NULL REFERENCES users (id),
 
     PRIMARY KEY (id)

sqlparser/CHANGELOG.md

@@ -1,3 +1,8 @@
+## 0.33.1
+
+- Fix explicit `NULL` column constraints being dropped when converting nodes
+  to SQL.
+
 ## 0.33.0
 
 - Support the [column-name-list](https://sqlite.org/syntax/column-name-list.html)

sqlparser/lib/utils/node_to_text.dart

@@ -204,6 +204,9 @@ class NodeSqlBuilder extends AstVisitor<void, void> {
         keyword(TokenType.$null);
         _conflictClause(notNull.onConflict);
       },
+      nullable: (nullable) {
+        keyword(TokenType.$null);
+      },
       unique: (unique) {
         keyword(TokenType.unique);
         _conflictClause(unique.onConflict);

sqlparser/test/utils/node_to_text_test.dart

@@ -165,6 +165,14 @@ CREATE TABLE foo (bar INTEGER NOT NULL PRIMARY KEY) With FooData.myConstructor;
       test('virtual', () {
         testFormat('CREATE VIRTUAL TABLE foo USING bar(a, b, c);');
       });
+
+      test('not null', () {
+        testFormat('CREATE TABLE foo (bar TEXT NOT NULL);');
+      });
+
+      test('null constraint', () {
+        testFormat('CREATE TABLE foo (bar TEXT NULL);');
+      });
     });
 
     test('index', () {