]> git.openstreetmap.org Git - rails.git/blob - test/functional/notes_controller_test.rb
Merge branch 'master' into notes
[rails.git] / test / functional / notes_controller_test.rb
1 require File.dirname(__FILE__) + '/../test_helper'
2
3 class NotesControllerTest < ActionController::TestCase
4   fixtures :users, :notes, :note_comments
5
6   ##
7   # test all routes which lead to this controller
8   def test_routes
9     assert_routing(
10       { :path => "/api/0.6/notes", :method => :post },
11       { :controller => "notes", :action => "create", :format => "xml" }
12     )
13     assert_routing(
14       { :path => "/api/0.6/notes/1", :method => :get },
15       { :controller => "notes", :action => "show", :id => "1", :format => "xml" }
16     )
17     assert_recognizes(
18       { :controller => "notes", :action => "show", :id => "1", :format => "xml" },
19       { :path => "/api/0.6/notes/1.xml", :method => :get }
20     )
21     assert_routing(
22       { :path => "/api/0.6/notes/1.rss", :method => :get },
23       { :controller => "notes", :action => "show", :id => "1", :format => "rss" }
24     )
25     assert_routing(
26       { :path => "/api/0.6/notes/1.json", :method => :get },
27       { :controller => "notes", :action => "show", :id => "1", :format => "json" }
28     )
29     assert_routing(
30       { :path => "/api/0.6/notes/1.gpx", :method => :get },
31       { :controller => "notes", :action => "show", :id => "1", :format => "gpx" }
32     )
33     assert_routing(
34       { :path => "/api/0.6/notes/1/comment", :method => :post },
35       { :controller => "notes", :action => "comment", :id => "1", :format => "xml" }
36     )
37     assert_routing(
38       { :path => "/api/0.6/notes/1/close", :method => :post },
39       { :controller => "notes", :action => "close", :id => "1", :format => "xml" }
40     )
41     assert_routing(
42       { :path => "/api/0.6/notes/1", :method => :delete },
43       { :controller => "notes", :action => "destroy", :id => "1", :format => "xml" }
44     )
45
46     assert_routing(
47       { :path => "/api/0.6/notes", :method => :get },
48       { :controller => "notes", :action => "index", :format => "xml" }
49     )
50     assert_recognizes(
51       { :controller => "notes", :action => "index", :format => "xml" },
52       { :path => "/api/0.6/notes.xml", :method => :get }
53     )
54     assert_routing(
55       { :path => "/api/0.6/notes.rss", :method => :get },
56       { :controller => "notes", :action => "index", :format => "rss" }
57     )
58     assert_routing(
59       { :path => "/api/0.6/notes.json", :method => :get },
60       { :controller => "notes", :action => "index", :format => "json" }
61     )
62     assert_routing(
63       { :path => "/api/0.6/notes.gpx", :method => :get },
64       { :controller => "notes", :action => "index", :format => "gpx" }
65     )
66
67     assert_routing(
68       { :path => "/api/0.6/notes/search", :method => :get },
69       { :controller => "notes", :action => "search", :format => "xml" }
70     )
71     assert_recognizes(
72       { :controller => "notes", :action => "search", :format => "xml" },
73       { :path => "/api/0.6/notes/search.xml", :method => :get }
74     )
75     assert_routing(
76       { :path => "/api/0.6/notes/search.rss", :method => :get },
77       { :controller => "notes", :action => "search", :format => "rss" }
78     )
79     assert_routing(
80       { :path => "/api/0.6/notes/search.json", :method => :get },
81       { :controller => "notes", :action => "search", :format => "json" }
82     )
83     assert_routing(
84       { :path => "/api/0.6/notes/search.gpx", :method => :get },
85       { :controller => "notes", :action => "search", :format => "gpx" }
86     )
87
88     assert_routing(
89       { :path => "/api/0.6/notes/feed", :method => :get },
90       { :controller => "notes", :action => "feed", :format => "rss" }
91     )
92
93     assert_recognizes(
94       { :controller => "notes", :action => "create" },
95       { :path => "/api/0.6/notes/addPOIexec", :method => :post }
96     )
97     assert_recognizes(
98       { :controller => "notes", :action => "close" },
99       { :path => "/api/0.6/notes/closePOIexec", :method => :post }
100     )
101     assert_recognizes(
102       { :controller => "notes", :action => "comment" },
103       { :path => "/api/0.6/notes/editPOIexec", :method => :post }
104     )
105     assert_recognizes(
106       { :controller => "notes", :action => "index", :format => "gpx" },
107       { :path => "/api/0.6/notes/getGPX", :method => :get }
108     )
109     assert_recognizes(
110       { :controller => "notes", :action => "feed", :format => "rss" },
111       { :path => "/api/0.6/notes/getRSSfeed", :method => :get }
112     )
113
114     assert_routing(
115       { :path => "/user/username/notes", :method => :get },
116       { :controller => "notes", :action => "mine", :display_name => "username" }
117     )
118   end
119
120   def test_note_create_success
121     assert_difference('Note.count') do
122       assert_difference('NoteComment.count') do
123         post :create, {:lat => -1.0, :lon => -1.0, :text => "This is a comment", :format => "json"}
124       end
125     end
126     assert_response :success
127     js = ActiveSupport::JSON.decode(@response.body)
128     assert_not_nil js
129     assert_equal "Feature", js["type"]
130     assert_equal "Point", js["geometry"]["type"]
131     assert_equal [-1.0, -1.0], js["geometry"]["coordinates"]
132     assert_equal "open", js["properties"]["status"]
133     assert_equal 1, js["properties"]["comments"].count
134     assert_equal "opened", js["properties"]["comments"].last["action"]
135     assert_equal "This is a comment", js["properties"]["comments"].last["text"]
136     assert_nil js["properties"]["comments"].last["user"]
137     id = js["properties"]["id"]
138
139     get :show, {:id => id, :format => "json"}
140     assert_response :success
141     js = ActiveSupport::JSON.decode(@response.body)
142     assert_not_nil js
143     assert_equal "Feature", js["type"]
144     assert_equal "Point", js["geometry"]["type"]
145     assert_equal [-1.0, -1.0], js["geometry"]["coordinates"]
146     assert_equal id, js["properties"]["id"]
147     assert_equal "open", js["properties"]["status"]
148     assert_equal 1, js["properties"]["comments"].count
149     assert_equal "opened", js["properties"]["comments"].last["action"]
150     assert_equal "This is a comment", js["properties"]["comments"].last["text"]
151     assert_nil js["properties"]["comments"].last["user"]
152   end
153
154   def test_note_create_fail
155     assert_no_difference('Note.count') do
156       assert_no_difference('NoteComment.count') do
157         post :create, {:lon => -1.0, :text => "This is a comment"}
158       end
159     end
160     assert_response :bad_request
161
162     assert_no_difference('Note.count') do
163       assert_no_difference('NoteComment.count') do
164         post :create, {:lat => -1.0, :text => "This is a comment"}
165       end
166     end
167     assert_response :bad_request
168
169     assert_no_difference('Note.count') do
170       assert_no_difference('NoteComment.count') do
171         post :create, {:lat => -1.0, :lon => -1.0}
172       end
173     end
174     assert_response :bad_request
175
176     assert_no_difference('Note.count') do
177       assert_no_difference('NoteComment.count') do
178         post :create, {:lat => -1.0, :lon => -1.0, :text => ""}
179       end
180     end
181     assert_response :bad_request
182
183     assert_no_difference('Note.count') do
184       assert_no_difference('NoteComment.count') do
185         post :create, {:lat => -100.0, :lon => -1.0, :text => "This is a comment"}
186       end
187     end
188     assert_response :bad_request
189
190     assert_no_difference('Note.count') do
191       assert_no_difference('NoteComment.count') do
192         post :create, {:lat => -1.0, :lon => -200.0, :text => "This is a comment"}
193       end
194     end
195     assert_response :bad_request
196   end
197
198   def test_note_comment_create_success
199     assert_difference('NoteComment.count') do
200       post :comment, {:id => notes(:open_note_with_comment).id, :text => "This is an additional comment", :format => "json"}
201     end
202     assert_response :success
203     js = ActiveSupport::JSON.decode(@response.body)
204     assert_not_nil js
205     assert_equal "Feature", js["type"]
206     assert_equal notes(:open_note_with_comment).id, js["properties"]["id"]
207     assert_equal "open", js["properties"]["status"]
208     assert_equal 3, js["properties"]["comments"].count
209     assert_equal "commented", js["properties"]["comments"].last["action"]
210     assert_equal "This is an additional comment", js["properties"]["comments"].last["text"]
211     assert_nil js["properties"]["comments"].last["user"]
212
213     get :show, {:id => notes(:open_note_with_comment).id, :format => "json"}
214     assert_response :success
215     js = ActiveSupport::JSON.decode(@response.body)
216     assert_not_nil js
217     assert_equal "Feature", js["type"]
218     assert_equal notes(:open_note_with_comment).id, js["properties"]["id"]
219     assert_equal "open", js["properties"]["status"]
220     assert_equal 3, js["properties"]["comments"].count
221     assert_equal "commented", js["properties"]["comments"].last["action"]
222     assert_equal "This is an additional comment", js["properties"]["comments"].last["text"]
223     assert_nil js["properties"]["comments"].last["user"]
224   end
225
226   def test_note_comment_create_fail
227     assert_no_difference('NoteComment.count') do
228       post :comment, {:text => "This is an additional comment"}
229     end
230     assert_response :bad_request
231
232     assert_no_difference('NoteComment.count') do
233       post :comment, {:id => notes(:open_note_with_comment).id}
234     end
235     assert_response :bad_request
236
237     assert_no_difference('NoteComment.count') do
238       post :comment, {:id => notes(:open_note_with_comment).id, :text => ""}
239     end
240     assert_response :bad_request
241
242     assert_no_difference('NoteComment.count') do
243       post :comment, {:id => 12345, :text => "This is an additional comment"}
244     end
245     assert_response :not_found
246
247     assert_no_difference('NoteComment.count') do
248       post :comment, {:id => notes(:hidden_note_with_comment).id, :text => "This is an additional comment"}
249     end
250     assert_response :gone
251
252     assert_no_difference('NoteComment.count') do
253       post :comment, {:id => notes(:closed_note_with_comment).id, :text => "This is an additional comment"}
254     end
255     assert_response :conflict
256   end
257
258   def test_note_close_success
259     post :close, {:id => notes(:open_note_with_comment).id, :text => "This is a close comment", :format => "json"}
260     assert_response :unauthorized
261
262     basic_authorization(users(:public_user).email, "test")
263
264     post :close, {:id => notes(:open_note_with_comment).id, :text => "This is a close comment", :format => "json"}
265     assert_response :success
266     js = ActiveSupport::JSON.decode(@response.body)
267     assert_not_nil js
268     assert_equal "Feature", js["type"]
269     assert_equal notes(:open_note_with_comment).id, js["properties"]["id"]
270     assert_equal "closed", js["properties"]["status"]
271     assert_equal 3, js["properties"]["comments"].count
272     assert_equal "closed", js["properties"]["comments"].last["action"]
273     assert_equal "This is a close comment", js["properties"]["comments"].last["text"]
274     assert_equal "test2", js["properties"]["comments"].last["user"]
275
276     get :show, {:id => notes(:open_note_with_comment).id, :format => "json"}
277     assert_response :success
278     js = ActiveSupport::JSON.decode(@response.body)
279     assert_not_nil js
280     assert_equal "Feature", js["type"]
281     assert_equal notes(:open_note_with_comment).id, js["properties"]["id"]
282     assert_equal "closed", js["properties"]["status"]
283     assert_equal 3, js["properties"]["comments"].count
284     assert_equal "closed", js["properties"]["comments"].last["action"]
285     assert_equal "This is a close comment", js["properties"]["comments"].last["text"]
286     assert_equal "test2", js["properties"]["comments"].last["user"]
287   end
288
289   def test_note_close_fail
290     post :close
291     assert_response :unauthorized
292
293     basic_authorization(users(:public_user).email, "test")
294
295     post :close
296     assert_response :bad_request
297
298     post :close, {:id => 12345}
299     assert_response :not_found
300
301     post :close, {:id => notes(:hidden_note_with_comment).id}
302     assert_response :gone
303
304     post :close, {:id => notes(:closed_note_with_comment).id}
305     assert_response :conflict
306   end
307
308   def test_note_read_success
309     get :show, {:id => notes(:open_note).id, :format => "xml"}
310     assert_response :success
311     assert_equal "application/xml", @response.content_type
312
313     get :show, {:id => notes(:open_note).id, :format => "rss"}
314     assert_response :success
315     assert_equal "application/rss+xml", @response.content_type
316
317     get :show, {:id => notes(:open_note).id, :format => "json"}
318     assert_response :success
319     assert_equal "application/json", @response.content_type
320
321     get :show, {:id => notes(:open_note).id, :format => "gpx"}
322     assert_response :success
323     assert_equal "application/gpx+xml", @response.content_type
324   end
325
326   def test_note_read_hidden_comment
327     get :show, {:id => notes(:note_with_hidden_comment).id, :format => "json"}
328     assert_response :success
329     js = ActiveSupport::JSON.decode(@response.body)
330     assert_not_nil js
331     assert_equal notes(:note_with_hidden_comment).id, js["properties"]["id"]
332     assert_equal 2, js["properties"]["comments"].count
333     assert_equal "Valid comment for note 5", js["properties"]["comments"][0]["text"]
334     assert_equal "Another valid comment for note 5", js["properties"]["comments"][1]["text"]
335   end
336
337   def test_note_read_fail
338     get :show, {:id => 12345}
339     assert_response :not_found
340
341     get :show, {:id => notes(:hidden_note_with_comment).id}
342     assert_response :gone
343   end
344
345   def test_note_delete_success
346     delete :destroy, {:id => notes(:open_note_with_comment).id, :text => "This is a hide comment", :format => "json"}
347     assert_response :unauthorized
348
349     basic_authorization(users(:public_user).email, "test")
350
351     delete :destroy, {:id => notes(:open_note_with_comment).id, :text => "This is a hide comment", :format => "json"}
352     assert_response :forbidden
353
354     basic_authorization(users(:moderator_user).email, "test")
355
356     delete :destroy, {:id => notes(:open_note_with_comment).id, :text => "This is a hide comment", :format => "json"}
357     assert_response :success
358     js = ActiveSupport::JSON.decode(@response.body)
359     assert_not_nil js
360     assert_equal "Feature", js["type"]
361     assert_equal notes(:open_note_with_comment).id, js["properties"]["id"]
362     assert_equal "hidden", js["properties"]["status"]
363     assert_equal 3, js["properties"]["comments"].count
364     assert_equal "hidden", js["properties"]["comments"].last["action"]
365     assert_equal "This is a hide comment", js["properties"]["comments"].last["text"]
366     assert_equal "moderator", js["properties"]["comments"].last["user"]
367
368     get :show, {:id => notes(:open_note_with_comment).id, :format => 'json'}
369     assert_response :gone
370   end
371
372   def test_note_delete_fail
373     delete :destroy, {:id => 12345, :format => "json"}
374     assert_response :unauthorized
375
376     basic_authorization(users(:public_user).email, "test")
377
378     delete :destroy, {:id => 12345, :format => "json"}
379     assert_response :forbidden
380
381     basic_authorization(users(:moderator_user).email, "test")
382
383     delete :destroy, {:id => 12345, :format => "json"}
384     assert_response :not_found
385
386     delete :destroy, {:id => notes(:hidden_note_with_comment).id, :format => "json"}
387     assert_response :gone
388   end
389
390   def test_get_notes_success
391 #    get :index, {:bbox => '1,1,1.2,1.2'}
392 #    assert_response :success
393 #    assert_equal "text/javascript", @response.content_type
394
395     get :index, {:bbox => '1,1,1.2,1.2', :format => 'rss'}
396     assert_response :success
397     assert_equal "application/rss+xml", @response.content_type
398
399     get :index, {:bbox => '1,1,1.2,1.2', :format => 'json'}
400     assert_response :success
401     assert_equal "application/json", @response.content_type
402
403     get :index, {:bbox => '1,1,1.2,1.2', :format => 'xml'}
404     assert_response :success
405     assert_equal "application/xml", @response.content_type
406
407     get :index, {:bbox => '1,1,1.2,1.2', :format => 'gpx'}
408     assert_response :success
409     assert_equal "application/gpx+xml", @response.content_type
410   end
411
412   def test_get_notes_large_area
413 #    get :index, {:bbox => '-2.5,-2.5,2.5,2.5'}
414 #    assert_response :success
415
416 #    get :index, {:l => '-2.5', :b => '-2.5', :r => '2.5', :t => '2.5'}
417 #    assert_response :success
418
419     get :index, {:bbox => '-10,-10,12,12'}
420     assert_response :bad_request
421
422     get :index, {:l => '-10', :b => '-10', :r => '12', :t => '12'}
423     assert_response :bad_request
424   end
425
426   def test_get_notes_closed
427     get :index, {:bbox => '1,1,1.7,1.7', :closed => '7', :format => 'json'}
428     assert_response :success
429     assert_equal "application/json", @response.content_type
430     js = ActiveSupport::JSON.decode(@response.body)
431     assert_not_nil js
432     assert_equal "FeatureCollection", js["type"]
433     assert_equal 4, js["features"].count
434
435     get :index, {:bbox => '1,1,1.7,1.7', :closed => '0', :format => 'json'}
436     assert_response :success
437     assert_equal "application/json", @response.content_type
438     js = ActiveSupport::JSON.decode(@response.body)
439     assert_not_nil js
440     assert_equal "FeatureCollection", js["type"]
441     assert_equal 4, js["features"].count
442
443     get :index, {:bbox => '1,1,1.7,1.7', :closed => '-1', :format => 'json'}
444     assert_response :success
445     assert_equal "application/json", @response.content_type
446     js = ActiveSupport::JSON.decode(@response.body)
447     assert_not_nil js
448     assert_equal "FeatureCollection", js["type"]
449     assert_equal 6, js["features"].count
450   end
451
452   def test_get_notes_bad_params
453     get :index, {:bbox => '-2.5,-2.5,2.5'}
454     assert_response :bad_request
455
456     get :index, {:bbox => '-2.5,-2.5,2.5,2.5,2.5'}
457     assert_response :bad_request
458
459     get :index, {:b => '-2.5', :r => '2.5', :t => '2.5'}
460     assert_response :bad_request
461
462     get :index, {:l => '-2.5', :r => '2.5', :t => '2.5'}
463     assert_response :bad_request
464
465     get :index, {:l => '-2.5', :b => '-2.5', :t => '2.5'}
466     assert_response :bad_request
467
468     get :index, {:l => '-2.5', :b => '-2.5', :r => '2.5'}
469     assert_response :bad_request
470   end
471
472   def test_search_success
473     get :search, {:q => 'note 1', :format => 'xml'}
474     assert_response :success
475     assert_equal "application/xml", @response.content_type
476
477     get :search, {:q => 'note 1', :format => 'json'}
478     assert_response :success
479     assert_equal "application/json", @response.content_type
480
481     get :search, {:q => 'note 1', :format => 'rss'}
482     assert_response :success
483     assert_equal "application/rss+xml", @response.content_type
484
485     get :search, {:q => 'note 1', :format => 'gpx'}
486     assert_response :success
487     assert_equal "application/gpx+xml", @response.content_type
488   end
489
490   def test_search_bad_params
491     get :search
492     assert_response :bad_request
493   end
494
495   def test_rss_success
496     get :feed, {:format => "rss"}
497     assert_response :success
498     assert_equal "application/rss+xml", @response.content_type
499
500     get :feed, {:bbox => "1,1,1.2,1.2", :format => "rss"}
501     assert_response :success    
502     assert_equal "application/rss+xml", @response.content_type
503   end
504
505   def test_rss_fail
506     get :feed, {:bbox => "1,1,1.2"}
507     assert_response :bad_request
508
509     get :feed, {:bbox => "1,1,1.2,1.2,1.2"}
510     assert_response :bad_request
511   end
512
513   def test_user_notes_success
514     get :mine, {:display_name => "test"}
515     assert_response :success
516
517     get :mine, {:display_name => "pulibc_test2"}
518     assert_response :success
519
520     get :mine, {:display_name => "non-existent"}
521     assert_response :not_found  
522   end
523 end