100% test coverage

.github/workflows/dart.yml

@@ -25,4 +25,4 @@ jobs:
       - name: Tests
         run: dart test --coverage=.coverage -j1
       - name: Coverage
-        run: dart run coverage:format_coverage -l -c -i .coverage --report-on=lib | dart run check_coverage:check_coverage 98
+        run: dart run coverage:format_coverage -l -c -i .coverage --report-on=lib | dart run check_coverage:check_coverage

lib/src/document/new_resource.dart

@@ -61,7 +61,6 @@ class NewResource {
     if (r is NewToMany) {
       return ToMany(r.map((identifier) => _toIdentifier(identifier, id)));
     }
-    // coverage:ignore-line
     throw StateError('Unexpected relationship type: ${r.runtimeType}');
   }
 

test/unit/document/inbound_document_test.dart

@@ -188,6 +188,20 @@ void main() {
         expect(() => InboundDocument(payload.many).asToOne(),
             throwsFormatException);
       });
+
+      test('throws on invalid new relationship', () {
+        expect(
+            () => InboundDocument(payload.newResourceInvalidRelationship)
+                .dataAsNewResource(),
+            throwsFormatException);
+      });
+
+      test('throws on incomplete new relationship', () {
+        expect(
+            () => InboundDocument(payload.newResourceIncompleteRelationship)
+                .dataAsNewResource(),
+            throwsFormatException);
+      });
     });
   });
 }

test/unit/document/new_resource_test.dart

@@ -5,6 +5,30 @@ import 'package:test/test.dart';
 
 void main() {
   group('NewResource', () {
+    test('toResource throws on unmatched local id in "many"', () {
+      final resource = NewResource('test_type', id: 'test_id', lid: 'test_lid')
+        ..relationships['many'] = NewToMany([
+          LocalIdentifier('test_type', 'test_lid2'),
+        ]);
+
+      expect(() => resource.toResource(() => 'my-test-id'), throwsStateError);
+    });
+
+    test('toResource throws on unmatched local id in "one"', () {
+      final resource = NewResource('test_type', id: 'test_id', lid: 'test_lid')
+        ..relationships['one'] =
+            NewToOne(LocalIdentifier('test_type', 'test_lid2'));
+
+      expect(() => resource.toResource(() => 'my-test-id'), throwsStateError);
+    });
+
+    test('toResource throws on invalid relationship', () {
+      final resource = NewResource('test_type', id: 'test_id')
+        ..relationships['many'] = NewRelationship();
+
+      expect(() => resource.toResource(() => 'my-test-id'), throwsStateError);
+    });
+
     test('json encoding', () {
       expect(jsonEncode(NewResource('test_type')),
           jsonEncode({'type': 'test_type'}));

test/unit/document/new_to_one_test.dart

@@ -0,0 +1,15 @@
+import 'package:json_api/document.dart';
+import 'package:test/expect.dart';
+import 'package:test/scaffolding.dart';
+
+void main() {
+  group('NewToOne', () {
+    test('can be iterated', () {
+      final id = Identifier('books', '123');
+      final r = NewToOne(id);
+      final list = <NewIdentifier>[];
+      list.addAll(r);
+      expect(list.single, equals(id));
+    });
+  });
+}

test/unit/document/payload.dart

@@ -147,3 +147,27 @@ final resource = {
     }
   }
 };
+
+final newResourceInvalidRelationship = {
+  'data': {
+    'type': 'articles',
+    'attributes': {'title': 'JSON:API paints my bikeshed!'},
+    'relationships': {
+      'author': {
+        'data': true,
+      },
+    }
+  }
+};
+
+final newResourceIncompleteRelationship = {
+  'data': {
+    'type': 'articles',
+    'attributes': {'title': 'JSON:API paints my bikeshed!'},
+    'relationships': {
+      'author': {
+        'data': {'type': 'person'},
+      },
+    }
+  }
+};