1 # Pleroma: A lightweight social networking server
2 # Copyright © 2017-2018 Pleroma Authors <https://pleroma.social/>
3 # SPDX-License-Identifier: AGPL-3.0-only
5 defmodule Pleroma.Web.OAuth.OAuthControllerTest do
6 use Pleroma.Web.ConnCase
10 alias Pleroma.Web.OAuth.Authorization
11 alias Pleroma.Web.OAuth.OAuthController
12 alias Pleroma.Web.OAuth.Token
14 @oauth_config_path [:oauth2, :issue_new_refresh_token]
18 signing_salt: "cooldude"
21 describe "in OAuth consumer mode, " do
23 oauth_consumer_strategies_path = [:auth, :oauth_consumer_strategies]
24 oauth_consumer_strategies = Pleroma.Config.get(oauth_consumer_strategies_path)
25 Pleroma.Config.put(oauth_consumer_strategies_path, ~w(twitter facebook))
28 Pleroma.Config.put(oauth_consumer_strategies_path, oauth_consumer_strategies)
32 app: insert(:oauth_app),
35 |> Plug.Session.call(Plug.Session.init(@session_opts))
40 test "GET /oauth/authorize renders auth forms, including OAuth consumer form", %{
49 "response_type" => "code",
50 "client_id" => app.client_id,
51 "redirect_uri" => OAuthController.default_redirect_uri(app),
56 assert response = html_response(conn, 200)
57 assert response =~ "Sign in with Twitter"
58 assert response =~ o_auth_path(conn, :prepare_request)
61 test "GET /oauth/prepare_request encodes parameters as `state` and redirects", %{
68 "/oauth/prepare_request",
70 "provider" => "twitter",
72 "scope" => "read follow",
73 "client_id" => app.client_id,
74 "redirect_uri" => OAuthController.default_redirect_uri(app),
80 assert response = html_response(conn, 302)
82 redirect_query = URI.parse(redirected_to(conn)).query
83 assert %{"state" => state_param} = URI.decode_query(redirect_query)
84 assert {:ok, state_components} = Poison.decode(state_param)
86 expected_client_id = app.client_id
87 expected_redirect_uri = app.redirect_uris
90 "scope" => "read follow",
91 "client_id" => ^expected_client_id,
92 "redirect_uri" => ^expected_redirect_uri,
97 test "with user-bound registration, GET /oauth/<provider>/callback redirects to `redirect_uri` with `code`",
98 %{app: app, conn: conn} do
99 registration = insert(:registration)
100 redirect_uri = OAuthController.default_redirect_uri(app)
103 "scope" => Enum.join(app.scopes, " "),
104 "client_id" => app.client_id,
105 "redirect_uri" => redirect_uri,
111 |> assign(:ueberauth_auth, %{provider: registration.provider, uid: registration.uid})
113 "/oauth/twitter/callback",
115 "oauth_token" => "G-5a3AAAAAAAwMH9AAABaektfSM",
116 "oauth_verifier" => "QZl8vUqNvXMTKpdmUnGejJxuHG75WWWs",
117 "provider" => "twitter",
118 "state" => Poison.encode!(state_params)
122 assert response = html_response(conn, 302)
123 assert redirected_to(conn) =~ ~r/#{redirect_uri}\?code=.+/
126 test "with user-unbound registration, GET /oauth/<provider>/callback renders registration_details page",
127 %{app: app, conn: conn} do
131 "scope" => "read write",
132 "client_id" => app.client_id,
133 "redirect_uri" => OAuthController.default_redirect_uri(app),
139 |> assign(:ueberauth_auth, %{
142 info: %{nickname: user.nickname, email: user.email, name: user.name, description: nil}
145 "/oauth/twitter/callback",
147 "oauth_token" => "G-5a3AAAAAAAwMH9AAABaektfSM",
148 "oauth_verifier" => "QZl8vUqNvXMTKpdmUnGejJxuHG75WWWs",
149 "provider" => "twitter",
150 "state" => Poison.encode!(state_params)
154 assert response = html_response(conn, 200)
155 assert response =~ ~r/name="op" type="submit" value="register"/
156 assert response =~ ~r/name="op" type="submit" value="connect"/
157 assert response =~ user.email
158 assert response =~ user.nickname
161 test "on authentication error, GET /oauth/<provider>/callback redirects to `redirect_uri`", %{
166 "scope" => Enum.join(app.scopes, " "),
167 "client_id" => app.client_id,
168 "redirect_uri" => OAuthController.default_redirect_uri(app),
174 |> assign(:ueberauth_failure, %{errors: [%{message: "(error description)"}]})
176 "/oauth/twitter/callback",
178 "oauth_token" => "G-5a3AAAAAAAwMH9AAABaektfSM",
179 "oauth_verifier" => "QZl8vUqNvXMTKpdmUnGejJxuHG75WWWs",
180 "provider" => "twitter",
181 "state" => Poison.encode!(state_params)
185 assert response = html_response(conn, 302)
186 assert redirected_to(conn) == app.redirect_uris
187 assert get_flash(conn, :error) == "Failed to authenticate: (error description)."
190 test "GET /oauth/registration_details renders registration details form", %{
197 "/oauth/registration_details",
199 "authorization" => %{
200 "scopes" => app.scopes,
201 "client_id" => app.client_id,
202 "redirect_uri" => OAuthController.default_redirect_uri(app),
203 "state" => "a_state",
205 "email" => "john@doe.com"
210 assert response = html_response(conn, 200)
211 assert response =~ ~r/name="op" type="submit" value="register"/
212 assert response =~ ~r/name="op" type="submit" value="connect"/
215 test "with valid params, POST /oauth/register?op=register redirects to `redirect_uri` with `code`",
220 registration = insert(:registration, user: nil, info: %{"nickname" => nil, "email" => nil})
221 redirect_uri = OAuthController.default_redirect_uri(app)
225 |> put_session(:registration_id, registration.id)
230 "authorization" => %{
231 "scopes" => app.scopes,
232 "client_id" => app.client_id,
233 "redirect_uri" => redirect_uri,
234 "state" => "a_state",
235 "nickname" => "availablenick",
236 "email" => "available@email.com"
241 assert response = html_response(conn, 302)
242 assert redirected_to(conn) =~ ~r/#{redirect_uri}\?code=.+/
245 test "with unlisted `redirect_uri`, POST /oauth/register?op=register results in HTTP 401",
250 registration = insert(:registration, user: nil, info: %{"nickname" => nil, "email" => nil})
251 unlisted_redirect_uri = "http://cross-site-request.com"
255 |> put_session(:registration_id, registration.id)
260 "authorization" => %{
261 "scopes" => app.scopes,
262 "client_id" => app.client_id,
263 "redirect_uri" => unlisted_redirect_uri,
264 "state" => "a_state",
265 "nickname" => "availablenick",
266 "email" => "available@email.com"
271 assert response = html_response(conn, 401)
274 test "with invalid params, POST /oauth/register?op=register renders registration_details page",
279 another_user = insert(:user)
280 registration = insert(:registration, user: nil, info: %{"nickname" => nil, "email" => nil})
284 "authorization" => %{
285 "scopes" => app.scopes,
286 "client_id" => app.client_id,
287 "redirect_uri" => OAuthController.default_redirect_uri(app),
288 "state" => "a_state",
289 "nickname" => "availablenickname",
290 "email" => "available@email.com"
294 for {bad_param, bad_param_value} <-
295 [{"nickname", another_user.nickname}, {"email", another_user.email}] do
296 bad_registration_attrs = %{
297 "authorization" => Map.put(params["authorization"], bad_param, bad_param_value)
300 bad_params = Map.merge(params, bad_registration_attrs)
304 |> put_session(:registration_id, registration.id)
305 |> post("/oauth/register", bad_params)
307 assert html_response(conn, 403) =~ ~r/name="op" type="submit" value="register"/
308 assert get_flash(conn, :error) == "Error: #{bad_param} has already been taken."
312 test "with valid params, POST /oauth/register?op=connect redirects to `redirect_uri` with `code`",
317 user = insert(:user, password_hash: Comeonin.Pbkdf2.hashpwsalt("testpassword"))
318 registration = insert(:registration, user: nil)
319 redirect_uri = OAuthController.default_redirect_uri(app)
323 |> put_session(:registration_id, registration.id)
328 "authorization" => %{
329 "scopes" => app.scopes,
330 "client_id" => app.client_id,
331 "redirect_uri" => redirect_uri,
332 "state" => "a_state",
333 "name" => user.nickname,
334 "password" => "testpassword"
339 assert response = html_response(conn, 302)
340 assert redirected_to(conn) =~ ~r/#{redirect_uri}\?code=.+/
343 test "with unlisted `redirect_uri`, POST /oauth/register?op=connect results in HTTP 401`",
348 user = insert(:user, password_hash: Comeonin.Pbkdf2.hashpwsalt("testpassword"))
349 registration = insert(:registration, user: nil)
350 unlisted_redirect_uri = "http://cross-site-request.com"
354 |> put_session(:registration_id, registration.id)
359 "authorization" => %{
360 "scopes" => app.scopes,
361 "client_id" => app.client_id,
362 "redirect_uri" => unlisted_redirect_uri,
363 "state" => "a_state",
364 "name" => user.nickname,
365 "password" => "testpassword"
370 assert response = html_response(conn, 401)
373 test "with invalid params, POST /oauth/register?op=connect renders registration_details page",
379 registration = insert(:registration, user: nil)
383 "authorization" => %{
384 "scopes" => app.scopes,
385 "client_id" => app.client_id,
386 "redirect_uri" => OAuthController.default_redirect_uri(app),
387 "state" => "a_state",
388 "name" => user.nickname,
389 "password" => "wrong password"
395 |> put_session(:registration_id, registration.id)
396 |> post("/oauth/register", params)
398 assert html_response(conn, 401) =~ ~r/name="op" type="submit" value="connect"/
399 assert get_flash(conn, :error) == "Invalid Username/Password"
403 describe "GET /oauth/authorize" do
406 app: insert(:oauth_app, redirect_uris: "https://redirect.url"),
409 |> Plug.Session.call(Plug.Session.init(@session_opts))
414 test "renders authentication page", %{app: app, conn: conn} do
420 "response_type" => "code",
421 "client_id" => app.client_id,
422 "redirect_uri" => OAuthController.default_redirect_uri(app),
427 assert html_response(conn, 200) =~ ~s(type="submit")
430 test "properly handles internal calls with `authorization`-wrapped params", %{
439 "authorization" => %{
440 "response_type" => "code",
441 "client_id" => app.client_id,
442 "redirect_uri" => OAuthController.default_redirect_uri(app),
448 assert html_response(conn, 200) =~ ~s(type="submit")
451 test "renders authentication page if user is already authenticated but `force_login` is tru-ish",
452 %{app: app, conn: conn} do
453 token = insert(:oauth_token, app_id: app.id)
457 |> put_session(:oauth_token, token.token)
461 "response_type" => "code",
462 "client_id" => app.client_id,
463 "redirect_uri" => OAuthController.default_redirect_uri(app),
465 "force_login" => "true"
469 assert html_response(conn, 200) =~ ~s(type="submit")
472 test "with existing authentication and non-OOB `redirect_uri`, redirects to app with `token` and `state` params",
477 token = insert(:oauth_token, app_id: app.id)
481 |> put_session(:oauth_token, token.token)
485 "response_type" => "code",
486 "client_id" => app.client_id,
487 "redirect_uri" => OAuthController.default_redirect_uri(app),
488 "state" => "specific_client_state",
493 assert URI.decode(redirected_to(conn)) ==
494 "https://redirect.url?access_token=#{token.token}&state=specific_client_state"
497 test "with existing authentication and unlisted non-OOB `redirect_uri`, redirects without credentials",
502 unlisted_redirect_uri = "http://cross-site-request.com"
503 token = insert(:oauth_token, app_id: app.id)
507 |> put_session(:oauth_token, token.token)
511 "response_type" => "code",
512 "client_id" => app.client_id,
513 "redirect_uri" => unlisted_redirect_uri,
514 "state" => "specific_client_state",
519 assert redirected_to(conn) == unlisted_redirect_uri
522 test "with existing authentication and OOB `redirect_uri`, redirects to app with `token` and `state` params",
527 token = insert(:oauth_token, app_id: app.id)
531 |> put_session(:oauth_token, token.token)
535 "response_type" => "code",
536 "client_id" => app.client_id,
537 "redirect_uri" => "urn:ietf:wg:oauth:2.0:oob",
542 assert html_response(conn, 200) =~ "Authorization exists"
546 describe "POST /oauth/authorize" do
547 test "redirects with oauth authorization" do
549 app = insert(:oauth_app, scopes: ["read", "write", "follow"])
550 redirect_uri = OAuthController.default_redirect_uri(app)
554 |> post("/oauth/authorize", %{
555 "authorization" => %{
556 "name" => user.nickname,
557 "password" => "test",
558 "client_id" => app.client_id,
559 "redirect_uri" => redirect_uri,
560 "scope" => "read write",
561 "state" => "statepassed"
565 target = redirected_to(conn)
566 assert target =~ redirect_uri
568 query = URI.parse(target).query |> URI.query_decoder() |> Map.new()
570 assert %{"state" => "statepassed", "code" => code} = query
571 auth = Repo.get_by(Authorization, token: code)
573 assert auth.scopes == ["read", "write"]
576 test "returns 401 for wrong credentials", %{conn: conn} do
578 app = insert(:oauth_app)
579 redirect_uri = OAuthController.default_redirect_uri(app)
583 |> post("/oauth/authorize", %{
584 "authorization" => %{
585 "name" => user.nickname,
586 "password" => "wrong",
587 "client_id" => app.client_id,
588 "redirect_uri" => redirect_uri,
589 "state" => "statepassed",
590 "scope" => Enum.join(app.scopes, " ")
593 |> html_response(:unauthorized)
596 assert result =~ app.client_id
597 assert result =~ redirect_uri
600 assert result =~ "Invalid Username/Password"
603 test "returns 401 for missing scopes", %{conn: conn} do
605 app = insert(:oauth_app)
606 redirect_uri = OAuthController.default_redirect_uri(app)
610 |> post("/oauth/authorize", %{
611 "authorization" => %{
612 "name" => user.nickname,
613 "password" => "test",
614 "client_id" => app.client_id,
615 "redirect_uri" => redirect_uri,
616 "state" => "statepassed",
620 |> html_response(:unauthorized)
623 assert result =~ app.client_id
624 assert result =~ redirect_uri
627 assert result =~ "This action is outside the authorized scopes"
630 test "returns 401 for scopes beyond app scopes", %{conn: conn} do
632 app = insert(:oauth_app, scopes: ["read", "write"])
633 redirect_uri = OAuthController.default_redirect_uri(app)
637 |> post("/oauth/authorize", %{
638 "authorization" => %{
639 "name" => user.nickname,
640 "password" => "test",
641 "client_id" => app.client_id,
642 "redirect_uri" => redirect_uri,
643 "state" => "statepassed",
644 "scope" => "read write follow"
647 |> html_response(:unauthorized)
650 assert result =~ app.client_id
651 assert result =~ redirect_uri
654 assert result =~ "This action is outside the authorized scopes"
658 describe "POST /oauth/token" do
659 test "issues a token for an all-body request" do
661 app = insert(:oauth_app, scopes: ["read", "write"])
663 {:ok, auth} = Authorization.create_authorization(app, user, ["write"])
667 |> post("/oauth/token", %{
668 "grant_type" => "authorization_code",
669 "code" => auth.token,
670 "redirect_uri" => OAuthController.default_redirect_uri(app),
671 "client_id" => app.client_id,
672 "client_secret" => app.client_secret
675 assert %{"access_token" => token, "me" => ap_id} = json_response(conn, 200)
677 token = Repo.get_by(Token, token: token)
679 assert token.scopes == auth.scopes
680 assert user.ap_id == ap_id
683 test "issues a token for `password` grant_type with valid credentials, with full permissions by default" do
684 password = "testpassword"
685 user = insert(:user, password_hash: Comeonin.Pbkdf2.hashpwsalt(password))
687 app = insert(:oauth_app, scopes: ["read", "write"])
689 # Note: "scope" param is intentionally omitted
692 |> post("/oauth/token", %{
693 "grant_type" => "password",
694 "username" => user.nickname,
695 "password" => password,
696 "client_id" => app.client_id,
697 "client_secret" => app.client_secret
700 assert %{"access_token" => token} = json_response(conn, 200)
702 token = Repo.get_by(Token, token: token)
704 assert token.scopes == app.scopes
707 test "issues a token for request with HTTP basic auth client credentials" do
709 app = insert(:oauth_app, scopes: ["scope1", "scope2", "scope3"])
711 {:ok, auth} = Authorization.create_authorization(app, user, ["scope1", "scope2"])
712 assert auth.scopes == ["scope1", "scope2"]
715 (URI.encode_www_form(app.client_id) <> ":" <> URI.encode_www_form(app.client_secret))
720 |> put_req_header("authorization", "Basic " <> app_encoded)
721 |> post("/oauth/token", %{
722 "grant_type" => "authorization_code",
723 "code" => auth.token,
724 "redirect_uri" => OAuthController.default_redirect_uri(app)
727 assert %{"access_token" => token, "scope" => scope} = json_response(conn, 200)
729 assert scope == "scope1 scope2"
731 token = Repo.get_by(Token, token: token)
733 assert token.scopes == ["scope1", "scope2"]
736 test "issue a token for client_credentials grant type" do
737 app = insert(:oauth_app, scopes: ["read", "write"])
741 |> post("/oauth/token", %{
742 "grant_type" => "client_credentials",
743 "client_id" => app.client_id,
744 "client_secret" => app.client_secret
747 assert %{"access_token" => token, "refresh_token" => refresh, "scope" => scope} =
748 json_response(conn, 200)
751 token_from_db = Repo.get_by(Token, token: token)
754 assert scope == "read write"
757 test "rejects token exchange with invalid client credentials" do
759 app = insert(:oauth_app)
761 {:ok, auth} = Authorization.create_authorization(app, user)
765 |> put_req_header("authorization", "Basic JTIxOiVGMCU5RiVBNCVCNwo=")
766 |> post("/oauth/token", %{
767 "grant_type" => "authorization_code",
768 "code" => auth.token,
769 "redirect_uri" => OAuthController.default_redirect_uri(app)
772 assert resp = json_response(conn, 400)
773 assert %{"error" => _} = resp
774 refute Map.has_key?(resp, "access_token")
777 test "rejects token exchange for valid credentials belonging to unconfirmed user and confirmation is required" do
778 setting = Pleroma.Config.get([:instance, :account_activation_required])
781 Pleroma.Config.put([:instance, :account_activation_required], true)
782 on_exit(fn -> Pleroma.Config.put([:instance, :account_activation_required], setting) end)
785 password = "testpassword"
786 user = insert(:user, password_hash: Comeonin.Pbkdf2.hashpwsalt(password))
787 info_change = Pleroma.User.Info.confirmation_changeset(user.info, need_confirmation: true)
791 |> Ecto.Changeset.change()
792 |> Ecto.Changeset.put_embed(:info, info_change)
795 refute Pleroma.User.auth_active?(user)
797 app = insert(:oauth_app)
801 |> post("/oauth/token", %{
802 "grant_type" => "password",
803 "username" => user.nickname,
804 "password" => password,
805 "client_id" => app.client_id,
806 "client_secret" => app.client_secret
809 assert resp = json_response(conn, 403)
810 assert %{"error" => _} = resp
811 refute Map.has_key?(resp, "access_token")
814 test "rejects token exchange for valid credentials belonging to deactivated user" do
815 password = "testpassword"
819 password_hash: Comeonin.Pbkdf2.hashpwsalt(password),
820 info: %{deactivated: true}
823 app = insert(:oauth_app)
827 |> post("/oauth/token", %{
828 "grant_type" => "password",
829 "username" => user.nickname,
830 "password" => password,
831 "client_id" => app.client_id,
832 "client_secret" => app.client_secret
835 assert resp = json_response(conn, 403)
836 assert %{"error" => _} = resp
837 refute Map.has_key?(resp, "access_token")
840 test "rejects an invalid authorization code" do
841 app = insert(:oauth_app)
845 |> post("/oauth/token", %{
846 "grant_type" => "authorization_code",
847 "code" => "Imobviouslyinvalid",
848 "redirect_uri" => OAuthController.default_redirect_uri(app),
849 "client_id" => app.client_id,
850 "client_secret" => app.client_secret
853 assert resp = json_response(conn, 400)
854 assert %{"error" => _} = json_response(conn, 400)
855 refute Map.has_key?(resp, "access_token")
859 describe "POST /oauth/token - refresh token" do
861 oauth_token_config = Pleroma.Config.get(@oauth_config_path)
864 Pleroma.Config.get(@oauth_config_path, oauth_token_config)
868 test "issues a new access token with keep fresh token" do
869 Pleroma.Config.put(@oauth_config_path, true)
871 app = insert(:oauth_app, scopes: ["read", "write"])
873 {:ok, auth} = Authorization.create_authorization(app, user, ["write"])
874 {:ok, token} = Token.exchange_token(app, auth)
878 |> post("/oauth/token", %{
879 "grant_type" => "refresh_token",
880 "refresh_token" => token.refresh_token,
881 "client_id" => app.client_id,
882 "client_secret" => app.client_secret
884 |> json_response(200)
891 "token_type" => "Bearer",
894 "refresh_token" => _,
900 refute Repo.get_by(Token, token: token.token)
901 new_token = Repo.get_by(Token, token: response["access_token"])
902 assert new_token.refresh_token == token.refresh_token
903 assert new_token.scopes == auth.scopes
904 assert new_token.user_id == user.id
905 assert new_token.app_id == app.id
908 test "issues a new access token with new fresh token" do
909 Pleroma.Config.put(@oauth_config_path, false)
911 app = insert(:oauth_app, scopes: ["read", "write"])
913 {:ok, auth} = Authorization.create_authorization(app, user, ["write"])
914 {:ok, token} = Token.exchange_token(app, auth)
918 |> post("/oauth/token", %{
919 "grant_type" => "refresh_token",
920 "refresh_token" => token.refresh_token,
921 "client_id" => app.client_id,
922 "client_secret" => app.client_secret
924 |> json_response(200)
931 "token_type" => "Bearer",
934 "refresh_token" => _,
940 refute Repo.get_by(Token, token: token.token)
941 new_token = Repo.get_by(Token, token: response["access_token"])
942 refute new_token.refresh_token == token.refresh_token
943 assert new_token.scopes == auth.scopes
944 assert new_token.user_id == user.id
945 assert new_token.app_id == app.id
948 test "returns 400 if we try use access token" do
950 app = insert(:oauth_app, scopes: ["read", "write"])
952 {:ok, auth} = Authorization.create_authorization(app, user, ["write"])
953 {:ok, token} = Token.exchange_token(app, auth)
957 |> post("/oauth/token", %{
958 "grant_type" => "refresh_token",
959 "refresh_token" => token.token,
960 "client_id" => app.client_id,
961 "client_secret" => app.client_secret
963 |> json_response(400)
965 assert %{"error" => "Invalid credentials"} == response
968 test "returns 400 if refresh_token invalid" do
969 app = insert(:oauth_app, scopes: ["read", "write"])
973 |> post("/oauth/token", %{
974 "grant_type" => "refresh_token",
975 "refresh_token" => "token.refresh_token",
976 "client_id" => app.client_id,
977 "client_secret" => app.client_secret
979 |> json_response(400)
981 assert %{"error" => "Invalid credentials"} == response
984 test "issues a new token if token expired" do
986 app = insert(:oauth_app, scopes: ["read", "write"])
988 {:ok, auth} = Authorization.create_authorization(app, user, ["write"])
989 {:ok, token} = Token.exchange_token(app, auth)
992 Ecto.Changeset.change(
994 %{valid_until: NaiveDateTime.add(NaiveDateTime.utc_now(), -86_400 * 30)}
997 {:ok, access_token} = Repo.update(change)
1001 |> post("/oauth/token", %{
1002 "grant_type" => "refresh_token",
1003 "refresh_token" => access_token.refresh_token,
1004 "client_id" => app.client_id,
1005 "client_secret" => app.client_secret
1007 |> json_response(200)
1014 "token_type" => "Bearer",
1015 "expires_in" => 600,
1016 "access_token" => _,
1017 "refresh_token" => _,
1023 refute Repo.get_by(Token, token: token.token)
1024 token = Repo.get_by(Token, token: response["access_token"])
1026 assert token.scopes == auth.scopes
1027 assert token.user_id == user.id
1028 assert token.app_id == app.id
1032 describe "POST /oauth/token - bad request" do
1033 test "returns 500" do
1036 |> post("/oauth/token", %{})
1037 |> json_response(500)
1039 assert %{"error" => "Bad request"} == response
1043 describe "POST /oauth/revoke - bad request" do
1044 test "returns 500" do
1047 |> post("/oauth/revoke", %{})
1048 |> json_response(500)
1050 assert %{"error" => "Bad request"} == response