diff --git a/src/tests/http-data/krate_yank b/src/tests/http-data/krate_yank_works_as_intended similarity index 100% rename from src/tests/http-data/krate_yank rename to src/tests/http-data/krate_yank_works_as_intended diff --git a/src/tests/krate.rs b/src/tests/krate.rs index efcf11c6d4f..0d8650cddf8 100644 --- a/src/tests/krate.rs +++ b/src/tests/krate.rs @@ -1386,7 +1386,7 @@ fn following() { } #[test] -fn yank() { +fn yank_works_as_intended() { let (app, anon, _, token) = TestApp::full().with_token(); // Upload a new crate, putting it in the git index @@ -1424,19 +1424,21 @@ fn yank() { } #[test] -fn yank_not_owner() { +fn yank_by_a_non_owner_fails() { let (app, _, _, token) = TestApp::full().with_token(); let another_user = app.db_new_user("bar"); let another_user = another_user.as_model(); app.db(|conn| { - CrateBuilder::new("foo_not", another_user.id).expect_build(conn); + CrateBuilder::new("foo_not", another_user.id) + .version("1.0.0") + .expect_build(conn); }); let json = token.yank("foo_not", "1.0.0").bad_with_status(200); assert_eq!( json.errors[0].detail, - "crate `foo_not` does not have a version `1.0.0`" + "must already be an owner to yank or unyank" ); }