]> git.openstreetmap.org Git - rails.git/blob - test/functional/notes_controller_test.rb
Make the API reject changes to closed 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 => -100.0, :lon => -1.0, :text => "This is a comment"}
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 => -1.0, :lon => -200.0, :text => "This is a comment"}
186       end
187     end
188     assert_response :bad_request
189   end
190
191   def test_note_comment_create_success
192     assert_difference('NoteComment.count') do
193       post :comment, {:id => notes(:open_note_with_comment).id, :text => "This is an additional comment", :format => "json"}
194     end
195     assert_response :success
196     js = ActiveSupport::JSON.decode(@response.body)
197     assert_not_nil js
198     assert_equal "Feature", js["type"]
199     assert_equal notes(:open_note_with_comment).id, js["properties"]["id"]
200     assert_equal "open", js["properties"]["status"]
201     assert_equal 3, js["properties"]["comments"].count
202     assert_equal "commented", js["properties"]["comments"].last["action"]
203     assert_equal "This is an additional comment", js["properties"]["comments"].last["text"]
204     assert_nil js["properties"]["comments"].last["user"]
205
206     get :show, {:id => notes(:open_note_with_comment).id, :format => "json"}
207     assert_response :success
208     js = ActiveSupport::JSON.decode(@response.body)
209     assert_not_nil js
210     assert_equal "Feature", js["type"]
211     assert_equal notes(:open_note_with_comment).id, js["properties"]["id"]
212     assert_equal "open", js["properties"]["status"]
213     assert_equal 3, js["properties"]["comments"].count
214     assert_equal "commented", js["properties"]["comments"].last["action"]
215     assert_equal "This is an additional comment", js["properties"]["comments"].last["text"]
216     assert_nil js["properties"]["comments"].last["user"]
217   end
218
219   def test_note_comment_create_fail
220     assert_no_difference('NoteComment.count') do
221       post :comment, {:text => "This is an additional comment"}
222     end
223     assert_response :bad_request
224
225     assert_no_difference('NoteComment.count') do
226       post :comment, {:id => notes(:open_note_with_comment).id}
227     end
228     assert_response :bad_request
229
230     assert_no_difference('NoteComment.count') do
231       post :comment, {:id => 12345, :text => "This is an additional comment"}
232     end
233     assert_response :not_found
234
235     assert_no_difference('NoteComment.count') do
236       post :comment, {:id => notes(:hidden_note_with_comment).id, :text => "This is an additional comment"}
237     end
238     assert_response :gone
239
240     assert_no_difference('NoteComment.count') do
241       post :comment, {:id => notes(:closed_note_with_comment).id, :text => "This is an additional comment"}
242     end
243     assert_response :conflict
244   end
245
246   def test_note_close_success
247     post :close, {:id => notes(:open_note_with_comment).id, :text => "This is a close comment", :format => "json"}
248     assert_response :success
249     js = ActiveSupport::JSON.decode(@response.body)
250     assert_not_nil js
251     assert_equal "Feature", js["type"]
252     assert_equal notes(:open_note_with_comment).id, js["properties"]["id"]
253     assert_equal "closed", js["properties"]["status"]
254     assert_equal 3, js["properties"]["comments"].count
255     assert_equal "closed", js["properties"]["comments"].last["action"]
256     assert_equal "This is a close comment", js["properties"]["comments"].last["text"]
257     assert_nil js["properties"]["comments"].last["user"]
258
259     get :show, {:id => notes(:open_note_with_comment).id, :format => "json"}
260     assert_response :success
261     js = ActiveSupport::JSON.decode(@response.body)
262     assert_not_nil js
263     assert_equal "Feature", js["type"]
264     assert_equal notes(:open_note_with_comment).id, js["properties"]["id"]
265     assert_equal "closed", js["properties"]["status"]
266     assert_equal 3, js["properties"]["comments"].count
267     assert_equal "closed", js["properties"]["comments"].last["action"]
268     assert_equal "This is a close comment", js["properties"]["comments"].last["text"]
269     assert_nil js["properties"]["comments"].last["user"]
270   end
271
272   def test_note_close_fail
273     post :close
274     assert_response :bad_request
275
276     post :close, {:id => 12345}
277     assert_response :not_found
278
279     post :close, {:id => notes(:hidden_note_with_comment).id}
280     assert_response :gone
281
282     post :close, {:id => notes(:closed_note_with_comment).id}
283     assert_response :conflict
284   end
285
286   def test_note_read_success
287     get :show, {:id => notes(:open_note).id, :format => "xml"}
288     assert_response :success
289     assert_equal "application/xml", @response.content_type
290
291     get :show, {:id => notes(:open_note).id, :format => "rss"}
292     assert_response :success
293     assert_equal "application/rss+xml", @response.content_type
294
295     get :show, {:id => notes(:open_note).id, :format => "json"}
296     assert_response :success
297     assert_equal "application/json", @response.content_type
298
299     get :show, {:id => notes(:open_note).id, :format => "gpx"}
300     assert_response :success
301     assert_equal "application/gpx+xml", @response.content_type
302   end
303
304   def test_note_read_hidden_comment
305     get :show, {:id => notes(:note_with_hidden_comment).id, :format => "json"}
306     assert_response :success
307     js = ActiveSupport::JSON.decode(@response.body)
308     assert_not_nil js
309     assert_equal notes(:note_with_hidden_comment).id, js["properties"]["id"]
310     assert_equal 2, js["properties"]["comments"].count
311     assert_equal "Valid comment for note 5", js["properties"]["comments"][0]["text"]
312     assert_equal "Another valid comment for note 5", js["properties"]["comments"][1]["text"]
313   end
314
315   def test_note_read_fail
316     get :show, {:id => 12345}
317     assert_response :not_found
318
319     get :show, {:id => notes(:hidden_note_with_comment).id}
320     assert_response :gone
321   end
322
323   def test_note_delete_success
324     delete :destroy, {:id => notes(:open_note_with_comment).id}
325     assert_response :success
326
327     get :show, {:id => notes(:open_note_with_comment).id, :format => 'json'}
328     assert_response :gone
329   end
330
331   def test_note_delete_fail
332     delete :destroy, {:id => 12345}
333     assert_response :not_found
334
335     delete :destroy, {:id => notes(:hidden_note_with_comment).id}
336     assert_response :gone
337   end
338
339   def test_get_notes_success
340 #    get :index, {:bbox => '1,1,1.2,1.2'}
341 #    assert_response :success
342 #    assert_equal "text/javascript", @response.content_type
343
344     get :index, {:bbox => '1,1,1.2,1.2', :format => 'rss'}
345     assert_response :success
346     assert_equal "application/rss+xml", @response.content_type
347
348     get :index, {:bbox => '1,1,1.2,1.2', :format => 'json'}
349     assert_response :success
350     assert_equal "application/json", @response.content_type
351
352     get :index, {:bbox => '1,1,1.2,1.2', :format => 'xml'}
353     assert_response :success
354     assert_equal "application/xml", @response.content_type
355
356     get :index, {:bbox => '1,1,1.2,1.2', :format => 'gpx'}
357     assert_response :success
358     assert_equal "application/gpx+xml", @response.content_type
359   end
360
361   def test_get_notes_large_area
362 #    get :index, {:bbox => '-2.5,-2.5,2.5,2.5'}
363 #    assert_response :success
364
365 #    get :index, {:l => '-2.5', :b => '-2.5', :r => '2.5', :t => '2.5'}
366 #    assert_response :success
367
368     get :index, {:bbox => '-10,-10,12,12'}
369     assert_response :bad_request
370
371     get :index, {:l => '-10', :b => '-10', :r => '12', :t => '12'}
372     assert_response :bad_request
373   end
374
375   def test_get_notes_closed
376     get :index, {:bbox => '1,1,1.7,1.7', :closed => '7', :format => 'json'}
377     assert_response :success
378     assert_equal "application/json", @response.content_type
379     js = ActiveSupport::JSON.decode(@response.body)
380     assert_not_nil js
381     assert_equal "FeatureCollection", js["type"]
382     assert_equal 4, js["features"].count
383
384     get :index, {:bbox => '1,1,1.7,1.7', :closed => '0', :format => 'json'}
385     assert_response :success
386     assert_equal "application/json", @response.content_type
387     js = ActiveSupport::JSON.decode(@response.body)
388     assert_not_nil js
389     assert_equal "FeatureCollection", js["type"]
390     assert_equal 4, js["features"].count
391
392     get :index, {:bbox => '1,1,1.7,1.7', :closed => '-1', :format => 'json'}
393     assert_response :success
394     assert_equal "application/json", @response.content_type
395     js = ActiveSupport::JSON.decode(@response.body)
396     assert_not_nil js
397     assert_equal "FeatureCollection", js["type"]
398     assert_equal 6, js["features"].count
399   end
400
401   def test_get_notes_bad_params
402     get :index, {:bbox => '-2.5,-2.5,2.5'}
403     assert_response :bad_request
404
405     get :index, {:bbox => '-2.5,-2.5,2.5,2.5,2.5'}
406     assert_response :bad_request
407
408     get :index, {:b => '-2.5', :r => '2.5', :t => '2.5'}
409     assert_response :bad_request
410
411     get :index, {:l => '-2.5', :r => '2.5', :t => '2.5'}
412     assert_response :bad_request
413
414     get :index, {:l => '-2.5', :b => '-2.5', :t => '2.5'}
415     assert_response :bad_request
416
417     get :index, {:l => '-2.5', :b => '-2.5', :r => '2.5'}
418     assert_response :bad_request
419   end
420
421   def test_search_success
422     get :search, {:q => 'note 1', :format => 'xml'}
423     assert_response :success
424     assert_equal "application/xml", @response.content_type
425
426     get :search, {:q => 'note 1', :format => 'json'}
427     assert_response :success
428     assert_equal "application/json", @response.content_type
429
430     get :search, {:q => 'note 1', :format => 'rss'}
431     assert_response :success
432     assert_equal "application/rss+xml", @response.content_type
433
434     get :search, {:q => 'note 1', :format => 'gpx'}
435     assert_response :success
436     assert_equal "application/gpx+xml", @response.content_type
437   end
438
439   def test_search_bad_params
440     get :search
441     assert_response :bad_request
442   end
443
444   def test_rss_success
445     get :feed, {:format => "rss"}
446     assert_response :success
447     assert_equal "application/rss+xml", @response.content_type
448
449     get :feed, {:bbox => "1,1,1.2,1.2", :format => "rss"}
450     assert_response :success    
451     assert_equal "application/rss+xml", @response.content_type
452   end
453
454   def test_rss_fail
455     get :feed, {:bbox => "1,1,1.2"}
456     assert_response :bad_request
457
458     get :feed, {:bbox => "1,1,1.2,1.2,1.2"}
459     assert_response :bad_request
460   end
461
462   def test_user_notes_success
463     get :mine, {:display_name => "test"}
464     assert_response :success
465
466     get :mine, {:display_name => "pulibc_test2"}
467     assert_response :success
468
469     get :mine, {:display_name => "non-existent"}
470     assert_response :not_found  
471   end
472 end