json: Implement JsonNullable.toJson and readFromJsonAssertingPresent

Author: chrisbobbe

lib/api/model/json.dart

@@ -22,6 +22,11 @@ class JsonNullable<T extends Object> {
     return map.containsKey(key) ? JsonNullable(map[key] as T?) : null;
   }
 
+  static T? readFromJsonAssertingPresent<T extends Object>(
+      Map<dynamic, dynamic> map, String key) {
+    return readFromJson<T>(map, key)!.value;
+  }
+
   @override
   bool operator ==(Object other) {
     if (other is! JsonNullable) return false;
@@ -30,6 +35,8 @@ class JsonNullable<T extends Object> {
 
   @override
   int get hashCode => Object.hash('JsonNullable', value);
+
+  T? toJson() => value;
 }
 
 class IdentityJsonConverter<T> extends JsonConverter<T, T> {

test/api/model/json_test.dart

@@ -2,6 +2,8 @@ import 'package:checks/checks.dart';
 import 'package:test/scaffolding.dart';
 import 'package:zulip/api/model/json.dart';
 
+import '../../stdlib_checks.dart';
+
 void main() {
   group('JsonNullable', () {
     test('value', () {
@@ -18,6 +20,23 @@ void main() {
       check(JsonNullable.readFromJson({'a': 3},    'a')).equals(const JsonNullable(3));
     });
 
+    test('readFromJsonAssertingPresent', () {
+      check(() => JsonNullable.readFromJsonAssertingPresent({},          'a'))
+        .throws<TypeError>();
+      check(() => JsonNullable.readFromJsonAssertingPresent({'a': null}, 'a'))
+        .returnsNormally().isNull();
+      check(() => JsonNullable.readFromJsonAssertingPresent({'a': 3},    'a'))
+        .returnsNormally().equals(3);
+    });
+
+    test('toJson / round-tripping', () {
+      check(JsonNullable<int>(3).toJson()).equals(3);
+      check(JsonNullable<int>(null).toJson()).equals(null);
+
+      check(JsonNullable.readFromJson({'a': null}, 'a')).jsonEquals(null);
+      check(JsonNullable.readFromJson({'a': 3},    'a')).jsonEquals(3);
+    });
+
     test('==/hashCode', () {
       // ignore: prefer_const_constructors
       check(JsonNullable<int>(null)).equals(JsonNullable(null));