Add empty feature name test and rename tests

invalid_feature and invalid_feature_name both for test invalid feature name.
So we can use the same prefix for them.

Signed-off-by: hi-rustin <rustin.liu@gmail.com>

Author: 0xPoe

src/tests/krate/publish/features.rs

@@ -35,7 +35,17 @@ fn feature_name_with_dot() {
 }
 
 #[test]
-fn invalid_feature_name() {
+fn empty_feature_name() {
+    let (app, _, _, token) = TestApp::full().with_token();
+    let crate_to_publish = PublishBuilder::new("foo", "1.0.0").feature("", &[]);
+    let response = token.publish_crate(crate_to_publish);
+    assert_eq!(response.status(), StatusCode::OK);
+    assert_json_snapshot!(response.into_json());
+    assert!(app.stored_files().is_empty());
+}
+
+#[test]
+fn invalid_feature_name1() {
     let (app, _, _, token) = TestApp::full().with_token();
 
     let crate_to_publish = PublishBuilder::new("foo", "1.0.0").feature("~foo", &[]);
@@ -46,7 +56,7 @@ fn invalid_feature_name() {
 }
 
 #[test]
-fn invalid_feature() {
+fn invalid_feature_name2() {
     let (app, _, _, token) = TestApp::full().with_token();
 
     let crate_to_publish = PublishBuilder::new("foo", "1.0.0").feature("foo", &["!bar"]);

src/tests/krate/publish/snapshots/all__krate__publish__features__empty_feature_name.snap

@@ -0,0 +1,11 @@
+---
+source: src/tests/krate/publish/features.rs
+expression: response.into_json()
+---
+{
+  "errors": [
+    {
+      "detail": "feature cannot be an empty"
+    }
+  ]
+}