AMT-Cheif
/
drift
mirror of https://github.com/AMT-Cheif/drift.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Ensure BoolType maps null values correctly 

Fixes #106
This commit is contained in:
Simon Binder 2019-08-19 16:16:34 +02:00
parent 40dafa1f2e
commit 17aabbe446
No known key found for this signature in database
GPG Key ID: 7891917E4147B8C0
2 changed files with 23 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
moor/lib/src/types/sql_types.dart
View File

@ -26,16 +26,24 @@ class BoolType extends SqlType<bool> {
@override @override
bool mapFromDatabaseResponse(response) { bool mapFromDatabaseResponse(response) {
// ignore: avoid_returning_null
if (response == null) return null;
return response != 0; return response != 0;
} }
@override @override
String mapToSqlConstant(bool content) { String mapToSqlConstant(bool content) {
if (content == null) {
return 'NULL';
}
return content ? '1' : '0'; return content ? '1' : '0';
} }
@override @override
mapToSqlVariable(bool content) { mapToSqlVariable(bool content) {
if (content == null) {
return null;
}
return content ? 1 : 0; return content ? 1 : 0;
} }
} }

15
moor/test/types/sql_type_test.dart Normal file
View File

@ -0,0 +1,15 @@
import 'package:moor/moor.dart' hide isNull;
import 'package:test/test.dart';
void main() {
test('types map null values to null', () {
final typeSystem = const SqlTypeSystem.withDefaults();
for (var type in typeSystem.types) {
expect(type.mapToSqlVariable(null), isNull,
reason: '$type should map null to null variables');
expect(type.mapFromDatabaseResponse(null), isNull,
reason: '$type should map null response to null value');
}
});
}