2 files changed, 73 insertions, 14 deletions

diff --git a/test/test_reader_writer.c b/test/test_reader_writer.c
index 61874bc7..829caff5 100644
--- a/test/test_reader_writer.c
+++ b/test/test_reader_writer.c
@@ -168,15 +168,19 @@ test_writer(const char* const path)
 
   const uint8_t buf[] = {0x80, 0, 0, 0, 0};
 
-  SerdNode s = serd_node_from_string(SERD_URI, USTR(""));
-  SerdNode p = serd_node_from_string(SERD_URI, USTR("http://example.org/pred"));
-  SerdNode o = serd_node_from_string(SERD_LITERAL, buf);
+  const SerdNode s = serd_node_from_string(SERD_URI, USTR(""));
+  const SerdNode p =
+    serd_node_from_string(SERD_URI, USTR("http://example.org/pred"));
+  const SerdNode o = serd_node_from_string(SERD_LITERAL, buf);
+  const SerdNode t = serd_node_from_string(SERD_URI, USTR("urn:Type"));
+  const SerdNode l = serd_node_from_string(SERD_LITERAL, USTR("en"));
 
   // Attempt to write invalid statements (should write nothing)
   const SerdNode* junk[][5] = {{&s, &p, &SERD_NODE_NULL, NULL, NULL},
                                {&s, &SERD_NODE_NULL, &o, NULL, NULL},
                                {&SERD_NODE_NULL, &p, &o, NULL, NULL},
                                {&s, &o, &o, NULL, NULL},
+                               {&s, &o, &o, &t, &l},
                                {&o, &p, &o, NULL, NULL},
                                {&s, &p, &SERD_NODE_NULL, NULL, NULL}};
   for (size_t i = 0; i < sizeof(junk) / (sizeof(SerdNode*) * 5); ++i) {
@@ -191,14 +195,11 @@ test_writer(const char* const path)
   }
 
   // Write some valid statements
-  const SerdNode  t         = serd_node_from_string(SERD_URI, USTR("urn:Type"));
-  const SerdNode  l         = serd_node_from_string(SERD_LITERAL, USTR("en"));
   const SerdNode* good[][5] = {{&s, &p, &o, NULL, NULL},
                                {&s, &p, &lit, NULL, NULL},
                                {&s, &p, &o, &SERD_NODE_NULL, &SERD_NODE_NULL},
                                {&s, &p, &o, &t, NULL},
                                {&s, &p, &o, NULL, &l},
-                               {&s, &p, &o, &t, &l},
                                {&s, &p, &o, &t, &SERD_NODE_NULL},
                                {&s, &p, &o, &SERD_NODE_NULL, &l},
                                {&s, &p, &o, NULL, &SERD_NODE_NULL},
@@ -260,7 +261,7 @@ test_reader(const char* const path)
 
   const SerdStatus st = serd_reader_read_file(reader, USTR(path));
   assert(!st);
-  assert(rt->n_statement == 13);
+  assert(rt->n_statement == 12);
   assert(rt->graph && rt->graph->buf &&
          !strcmp((const char*)rt->graph->buf, "http://example.org/"));
 
diff --git a/test/test_writer.c b/test/test_writer.c
index 27ce6138..7f8d2f94 100644
--- a/test/test_writer.c
+++ b/test/test_writer.c
@@ -309,17 +309,74 @@ test_write_nothing_node(void)
   SerdNode s = serd_node_from_string(SERD_URI, USTR(""));
   SerdNode p = serd_node_from_string(SERD_URI, USTR("http://example.org/pred"));
   SerdNode o = serd_node_from_string(SERD_NOTHING, USTR(""));
-  assert(!serd_writer_write_statement(writer, 0, NULL, &s, &p, &o, NULL, NULL));
-
-  assert(
-    !strncmp((const char*)chunk.buf, "<>\n\t<http://example.org/pred> ", 30));
+  assert(serd_writer_write_statement(writer, 0, NULL, &s, &p, &o, NULL, NULL) ==
+         SERD_ERR_BAD_ARG);
 
+  assert(!chunk.buf);
   serd_writer_free(writer);
+  serd_env_free(env);
+}
 
-  uint8_t* const out = serd_chunk_sink_finish(&chunk);
-  assert(!strcmp((const char*)out, "<>\n\t<http://example.org/pred>  .\n"));
-  serd_free(out);
+static void
+test_write_bad_statement(void)
+{
+  SerdEnv* const env = serd_env_new(NULL);
+  assert(env);
 
+  SerdChunk         chunk  = {NULL, 0};
+  SerdWriter* const writer = serd_writer_new(
+    SERD_TURTLE, (SerdStyle)0, env, NULL, serd_chunk_sink, &chunk);
+  assert(writer);
+
+  SerdNode s = serd_node_from_string(SERD_URI, USTR("http://example.org/s"));
+  SerdNode p = serd_node_from_string(SERD_URI, USTR("http://example.org/p"));
+  SerdNode o = serd_node_from_string(SERD_URI, USTR("http://example.org/o"));
+  SerdNode l = serd_node_from_string(SERD_LITERAL, USTR("lang"));
+
+  assert(serd_writer_write_statement(
+           writer,
+           (SerdStatementFlags)(SERD_ANON_S_BEGIN | SERD_LIST_S_BEGIN),
+           NULL,
+           &s,
+           &p,
+           &o,
+           NULL,
+           NULL) == SERD_ERR_BAD_ARG);
+
+  assert(serd_writer_write_statement(
+           writer,
+           (SerdStatementFlags)(SERD_EMPTY_S | SERD_LIST_S_BEGIN),
+           NULL,
+           &s,
+           &p,
+           &o,
+           NULL,
+           NULL) == SERD_ERR_BAD_ARG);
+
+  assert(serd_writer_write_statement(
+           writer,
+           (SerdStatementFlags)(SERD_ANON_O_BEGIN | SERD_LIST_O_BEGIN),
+           NULL,
+           &s,
+           &p,
+           &o,
+           NULL,
+           NULL) == SERD_ERR_BAD_ARG);
+
+  assert(serd_writer_write_statement(
+           writer,
+           (SerdStatementFlags)(SERD_EMPTY_O | SERD_LIST_O_BEGIN),
+           NULL,
+           &s,
+           &p,
+           &o,
+           NULL,
+           NULL) == SERD_ERR_BAD_ARG);
+
+  assert(serd_writer_write_statement(writer, 0U, NULL, &s, &p, &o, &o, &l) ==
+         SERD_ERR_BAD_ARG);
+
+  serd_writer_free(writer);
   serd_env_free(env);
 }
 
@@ -334,6 +391,7 @@ main(void)
   test_write_error();
   test_chunk_sink();
   test_write_nothing_node();
+  test_write_bad_statement();
 
   return 0;
 }