codeninjasllc/discourse
1
0
Fork 0
mirror of https://github.com/codeninjasllc/discourse.git synced 2024-12-18 03:25:31 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #3065 from fantasticfears/fix-visits-report

Browse source 
FIX: today visits not shown
This commit is contained in:
Régis Hanol 2014-12-29 21:23:47 +01:00
commit a04f44d65a
2 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/models/report.rb
View file

@ -46,7 +46,7 @@ class Report
end end
def self.report_visits(report) def self.report_visits(report)
basic_report_about report, UserVisit, :by_day, report.start_date, report.end_date basic_report_about report, UserVisit, :by_day, report.start_date, report.end_date.change(hour: 24)
end end
def self.report_signups(report) def self.report_signups(report)

5
spec/models/report_spec.rb
View file

@ -15,6 +15,7 @@ describe Report do
let(:user) { Fabricate(:user) } let(:user) { Fabricate(:user) }
before do before do
user.user_visits.create(visited_at: 1.hour.ago)
user.user_visits.create(visited_at: 1.day.ago) user.user_visits.create(visited_at: 1.day.ago)
user.user_visits.create(visited_at: 2.days.ago) user.user_visits.create(visited_at: 2.days.ago)
end end
@ -22,6 +23,10 @@ describe Report do
it "returns a report with data" do it "returns a report with data" do
report.data.should be_present report.data.should be_present
end end
it "return today's visit" do
report.data.select { |v| v[:x].today? }.should be_present
end
end end
end end