add: some test case

broker_test.go

@@ -116,6 +116,19 @@ func Test_fetchRemoteArticle(t *testing.T) {
 	}
 }
 
+func Test_fetchRemoteArticleNoID(t *testing.T) {
+	broker, _, _, teardown := setup()
+	defer teardown()
+
+	_, err := broker.fetchRemoteArticle(&Article{
+		ArticleHeader: &ArticleHeader{ID: ""},
+	})
+	if err == nil {
+		t.Errorf("expected error occured if no article ID")
+		return
+	}
+}
+
 func Test_fetchRemoteArticles(t *testing.T) {
 	broker, mux, _, teardown := setup()
 	t.Cleanup(func() {
@@ -814,6 +827,17 @@ func TestPatchArticle(t *testing.T) {
 	}
 }
 
+func TestPatchArticleNoID(t *testing.T) {
+	broker, _, _, teardown := setup()
+	defer teardown()
+
+	err := broker.patchArticle(&PostItem{ID: ""})
+	if err == nil {
+		t.Errorf("expected error occured if no article ID")
+		return
+	}
+}
+
 func Test_fetchLocalArticles(t *testing.T) {
 	updateAt := time.Date(2020, 4, 22, 16, 59, 59, 0, time.UTC)