summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStan Hu <stanhu@gmail.com>2016-07-27 16:42:38 -0700
committerStan Hu <stanhu@gmail.com>2016-07-27 16:42:38 -0700
commitd27e36f35af0c2850c5370a3348d30ce4bcf1a68 (patch)
tree537e289c3fb5d151fe5c477fe62c27ef6badfa05
parent8a9fc2b67e5bb6aeabdffa1f91115ac7613a505a (diff)
downloadgitlab-ce-d27e36f35af0c2850c5370a3348d30ce4bcf1a68.tar.gz
Add specs for caching commit author
-rw-r--r--app/models/commit.rb22
-rw-r--r--spec/models/commit_spec.rb20
2 files changed, 35 insertions, 7 deletions
diff --git a/app/models/commit.rb b/app/models/commit.rb
index 6a0d32d406e..486ad6714d9 100644
--- a/app/models/commit.rb
+++ b/app/models/commit.rb
@@ -178,14 +178,18 @@ class Commit
end
def author
- key = "commit_author:#{author_email}"
-
- # nil is a valid value since no author may exist in the system
- unless RequestStore.store.has_key?(key)
- RequestStore.store[key] = User.find_by_any_email(author_email.downcase)
+ if RequestStore.active?
+ key = "commit_author:#{author_email.downcase}"
+ # nil is a valid value since no author may exist in the system
+ if RequestStore.store.has_key?(key)
+ @author = RequestStore.store[key]
+ else
+ @author = find_author_by_any_email
+ RequestStore.store[key] = @author
+ end
+ else
+ @author ||= find_author_by_any_email
end
-
- @author ||= RequestStore.store[key]
end
def committer
@@ -313,6 +317,10 @@ class Commit
private
+ def find_author_by_any_email
+ User.find_by_any_email(author_email.downcase)
+ end
+
def repo_changes
changes = { added: [], modified: [], removed: [] }
diff --git a/spec/models/commit_spec.rb b/spec/models/commit_spec.rb
index ec1544bf815..c3392ee7440 100644
--- a/spec/models/commit_spec.rb
+++ b/spec/models/commit_spec.rb
@@ -13,6 +13,26 @@ describe Commit, models: true do
it { is_expected.to include_module(StaticModel) }
end
+ describe '#author' do
+ it 'looks up the author in a case-insensitive way' do
+ user = create(:user, email: commit.author_email.upcase)
+ expect(commit.author).to eq(user)
+ end
+
+ it 'caches the author' do
+ user = create(:user, email: commit.author_email)
+ expect(RequestStore).to receive(:active?).twice.and_return(true)
+ expect_any_instance_of(Commit).to receive(:find_author_by_any_email).and_call_original
+
+ expect(commit.author).to eq(user)
+ key = "commit_author:#{commit.author_email}"
+ expect(RequestStore.store[key]).to eq(user)
+
+ expect(commit.author).to eq(user)
+ RequestStore.store.clear
+ end
+ end
+
describe '#to_reference' do
it 'returns a String reference to the object' do
expect(commit.to_reference).to eq commit.id