summaryrefslogtreecommitdiff
path: root/spec/lib/votes_spec.rb
blob: df243a26008dca1ab7fafe6907b95d21fc49717a (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
require 'spec_helper'

describe Issue, 'Votes' do
  let(:issue) { create(:issue) }

  describe "#upvotes" do
    it "with no notes has a 0/0 score" do
      expect(issue.upvotes).to eq(0)
    end

    it "should recognize non-+1 notes" do
      add_note "No +1 here"
      expect(issue.notes.size).to eq(1)
      expect(issue.notes.first.upvote?).to be_falsey
      expect(issue.upvotes).to eq(0)
    end

    it "should recognize a single +1 note" do
      add_note "+1 This is awesome"
      expect(issue.upvotes).to eq(1)
    end

    it 'should recognize multiple +1 notes' do
      add_note '+1 This is awesome', create(:user)
      add_note '+1 I want this', create(:user)
      expect(issue.upvotes).to eq(2)
    end

    it 'should not count 2 +1 votes from the same user' do
      add_note '+1 This is awesome'
      add_note '+1 I want this'
      expect(issue.upvotes).to eq(1)
    end
  end

  describe "#downvotes" do
    it "with no notes has a 0/0 score" do
      expect(issue.downvotes).to eq(0)
    end

    it "should recognize non--1 notes" do
      add_note "Almost got a -1"
      expect(issue.notes.size).to eq(1)
      expect(issue.notes.first.downvote?).to be_falsey
      expect(issue.downvotes).to eq(0)
    end

    it "should recognize a single -1 note" do
      add_note "-1 This is bad"
      expect(issue.downvotes).to eq(1)
    end

    it "should recognize multiple -1 notes" do
      add_note('-1 This is bad', create(:user))
      add_note('-1 Away with this', create(:user))
      expect(issue.downvotes).to eq(2)
    end
  end

  describe "#votes_count" do
    it "with no notes has a 0/0 score" do
      expect(issue.votes_count).to eq(0)
    end

    it "should recognize non notes" do
      add_note "No +1 here"
      expect(issue.notes.size).to eq(1)
      expect(issue.votes_count).to eq(0)
    end

    it "should recognize a single +1 note" do
      add_note "+1 This is awesome"
      expect(issue.votes_count).to eq(1)
    end

    it "should recognize a single -1 note" do
      add_note "-1 This is bad"
      expect(issue.votes_count).to eq(1)
    end

    it "should recognize multiple notes" do
      add_note('+1 This is awesome', create(:user))
      add_note('-1 This is bad', create(:user))
      add_note('+1 I want this', create(:user))
      expect(issue.votes_count).to eq(3)
    end

    it 'should not count 2 -1 votes from the same user' do
      add_note '-1 This is suspicious'
      add_note '-1 This is bad'
      expect(issue.votes_count).to eq(1)
    end
  end

  describe "#upvotes_in_percent" do
    it "with no notes has a 0% score" do
      expect(issue.upvotes_in_percent).to eq(0)
    end

    it "should count a single 1 note as 100%" do
      add_note "+1 This is awesome"
      expect(issue.upvotes_in_percent).to eq(100)
    end

    it 'should count multiple +1 notes as 100%' do
      add_note('+1 This is awesome', create(:user))
      add_note('+1 I want this', create(:user))
      expect(issue.upvotes_in_percent).to eq(100)
    end

    it 'should count fractions for multiple +1 and -1 notes correctly' do
      add_note('+1 This is awesome', create(:user))
      add_note('+1 I want this', create(:user))
      add_note('-1 This is bad', create(:user))
      add_note('+1 me too', create(:user))
      expect(issue.upvotes_in_percent).to eq(75)
    end
  end

  describe "#downvotes_in_percent" do
    it "with no notes has a 0% score" do
      expect(issue.downvotes_in_percent).to eq(0)
    end

    it "should count a single -1 note as 100%" do
      add_note "-1 This is bad"
      expect(issue.downvotes_in_percent).to eq(100)
    end

    it 'should count multiple -1 notes as 100%' do
      add_note('-1 This is bad', create(:user))
      add_note('-1 Away with this', create(:user))
      expect(issue.downvotes_in_percent).to eq(100)
    end

    it 'should count fractions for multiple +1 and -1 notes correctly' do
      add_note('+1 This is awesome', create(:user))
      add_note('+1 I want this', create(:user))
      add_note('-1 This is bad', create(:user))
      add_note('+1 me too', create(:user))
      expect(issue.downvotes_in_percent).to eq(25)
    end
  end

  describe '#filter_superceded_votes' do

    it 'should count a users vote only once amongst multiple votes' do
      add_note('-1 This needs work before I will accept it')
      add_note('+1 I want this', create(:user))
      add_note('+1 This is is awesome', create(:user))
      add_note('+1 this looks good now')
      add_note('+1 This is awesome', create(:user))
      add_note('+1 me too', create(:user))
      expect(issue.downvotes).to eq(0)
      expect(issue.upvotes).to eq(5)
    end

    it 'should count each users vote only once' do
      add_note '-1 This needs work before it will be accepted'
      add_note '+1 I like this'
      add_note '+1 I still like this'
      add_note '+1 I really like this'
      add_note '+1 Give me this now!!!!'
      expect(issue.downvotes).to eq(0)
      expect(issue.upvotes).to eq(1)
    end

    it 'should count a users vote only once without caring about comments' do
      add_note '-1 This needs work before it will be accepted'
      add_note 'Comment 1'
      add_note 'Another comment'
      add_note '+1 vote'
      add_note 'final comment'
      expect(issue.downvotes).to eq(0)
      expect(issue.upvotes).to eq(1)
    end

  end

  def add_note(text, author = issue.author)
    created_at = Time.now - 1.hour + Note.count.seconds
    issue.notes << create(:note, note: text, project: issue.project,
                          author_id: author.id, created_at: created_at)
  end
end