test: fix graphql_can_remove_subgraph test

The test was silently passing because remove_subgraph returned Ok(vecfor non-existent subgraphs. Now that SubgraphNotFound error is properly
returned, the test needs to actually create the subgraph first.

Signed-off-by: Maksim Dimitrov <dimitrov.maksim@gmail.com>

Author: dimitrovmaksim

server/graphman/tests/deployment_mutation.rs

@@ -6,8 +6,8 @@ use graph::components::store::SubgraphStore;
 use graph::prelude::DeploymentHash;
 use serde::Deserialize;
 use serde_json::json;
-use test_store::create_test_subgraph;
 use test_store::SUBGRAPH_STORE;
+use test_store::{create_subgraph_name, create_test_subgraph};
 use tokio::time::sleep;
 
 use self::util::client::send_graphql_request;
@@ -358,6 +358,8 @@ fn graphql_cannot_create_new_subgraph_with_invalid_name() {
 #[test]
 fn graphql_can_remove_subgraph() {
     run_test(|| async {
+        create_subgraph_name("subgraph_1").await.unwrap();
+
         let resp = send_graphql_request(
             json!({
                 "query": r#"mutation RemoveSubgraph {

store/test-store/src/store.rs

@@ -259,6 +259,12 @@ pub async fn create_test_subgraph_with_features(
     locator
 }
 
+pub async fn create_subgraph_name(name: &str) -> Result<(), StoreError> {
+    let subgraph_name = SubgraphName::new_unchecked(name.to_string());
+    SUBGRAPH_STORE.create_subgraph(subgraph_name).await?;
+    Ok(())
+}
+
 pub async fn remove_subgraph(id: &DeploymentHash) {
     let name = SubgraphName::new_unchecked(id.to_string());
     SUBGRAPH_STORE.remove_subgraph(name).await.unwrap();