Fix variable names in tests

parent 61aaf019
Pipeline #140114965 canceled with stages
in 7 minutes and 50 seconds
...@@ -42,7 +42,7 @@ describe Users::DestroyService do ...@@ -42,7 +42,7 @@ describe Users::DestroyService do
it 'calls the bulk snippet destroy service for the user personal snippets' do it 'calls the bulk snippet destroy service for the user personal snippets' do
repo1 = create(:personal_snippet, :repository, author: user).snippet_repository repo1 = create(:personal_snippet, :repository, author: user).snippet_repository
repo3 = create(:project_snippet, :repository, project: project, author: user).snippet_repository repo2 = create(:project_snippet, :repository, project: project, author: user).snippet_repository
aggregate_failures do aggregate_failures do
expect(gitlab_shell.repository_exists?(repo1.shard_name, repo1.disk_path + '.git')).to be_truthy expect(gitlab_shell.repository_exists?(repo1.shard_name, repo1.disk_path + '.git')).to be_truthy
...@@ -68,7 +68,7 @@ describe Users::DestroyService do ...@@ -68,7 +68,7 @@ describe Users::DestroyService do
end end
it 'does not delete project snippets that the user is the author of' do it 'does not delete project snippets that the user is the author of' do
repo2 = create(:project_snippet, :repository, author: user).snippet_repository repo = create(:project_snippet, :repository, author: user).snippet_repository
service.execute(user) service.execute(user)
expect(gitlab_shell.repository_exists?(repo2.shard_name, repo2.disk_path + '.git')).to be_truthy expect(gitlab_shell.repository_exists?(repo2.shard_name, repo2.disk_path + '.git')).to be_truthy
expect(User.ghost.snippets).to include(repo2.snippet) expect(User.ghost.snippets).to include(repo2.snippet)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment