Commit 9505d1bf4223bcb4d698b88d693a0b4ddbdf561c
Committed by
Carlos Purificação
1 parent
e618194c
Exists in
theme-brasil-digital-from-staging
and in
9 other branches
Fix test
Showing
2 changed files
with
2 additions
and
1 deletions
Show diff stats
lib/noosfero/api/helpers.rb
@@ -279,7 +279,7 @@ | @@ -279,7 +279,7 @@ | ||
279 | end | 279 | end |
280 | end | 280 | end |
281 | if d[:provider] == 'serpro' | 281 | if d[:provider] == 'serpro' |
282 | - d[:verify_uri] ||= 'http://captcha2.servicoscorporativos.serpro.gov.br/captchavalidar/1.0.0/validar' | 282 | + raise ArgumentError, "Environment api_captcha_settings verify_uri not defined" if d[:verify_uri].nil? |
283 | return verify_serpro_captcha(d[:serpro_client_id], params[:txtToken_captcha_serpro_gov_br], params[:captcha_text], d[:verify_uri]) | 283 | return verify_serpro_captcha(d[:serpro_client_id], params[:txtToken_captcha_serpro_gov_br], params[:captcha_text], d[:verify_uri]) |
284 | end | 284 | end |
285 | raise ArgumentError, "Environment api_captcha_settings provider not defined" | 285 | raise ArgumentError, "Environment api_captcha_settings provider not defined" |
test/unit/api/helpers_test.rb
@@ -207,6 +207,7 @@ class APIHelpersTest < ActiveSupport::TestCase | @@ -207,6 +207,7 @@ class APIHelpersTest < ActiveSupport::TestCase | ||
207 | enabled: true, | 207 | enabled: true, |
208 | provider: 'serpro', | 208 | provider: 'serpro', |
209 | serpro_client_id: '0000000000000000', | 209 | serpro_client_id: '0000000000000000', |
210 | + verify_uri: 'http://localhost/api/verify', | ||
210 | } | 211 | } |
211 | assert_equal test_captcha("127.0.0.1", {}, environment), "Missing captcha data" | 212 | assert_equal test_captcha("127.0.0.1", {}, environment), "Missing captcha data" |
212 | end | 213 | end |