Fix error check logic and modify test_ccs to include response_mode

Author: Emmanuel Oche

msal/oauth2cli/oidc.py

@@ -185,8 +185,8 @@ def initiate_auth_code_flow(
             # they should simply go with oauth2.Client.
             _scope.append("openid")
         response_mode = kwargs.get("response_mode")
-        if response_mode is not None and (response_mode != "form_post" or response_mode != "query"):
-            raise ValueError('response_mode="form_post" or response_mode="query"  is allowed')
+        if response_mode is not None and response_mode != "form_post" and response_mode != "query":
+            raise ValueError('only response_mode="form_post" or response_mode="query" is allowed. default is query')
         nonce = "".join(random.sample(string.ascii_letters, 16))
         flow = super(Client, self).initiate_auth_code_flow(
             scope=_scope, nonce=_nonce_hash(nonce), **kwargs)

tests/test_ccs.py

@@ -16,7 +16,7 @@ def test_acquire_token_by_auth_code_flow(self):
         app = msal.ClientApplication("client_id")
         state = "foo"
         flow = app.initiate_auth_code_flow(
-            ["some", "scope"], login_hint="[email protected]", state=state)
+            ["some", "scope"], login_hint="[email protected]", state=state, response_mode="form_post")
         with patch.object(app.http_client, "post", return_value=MinimalResponse(
                 status_code=400, text='{"error": "mock"}')) as mocked_method:
             app.acquire_token_by_auth_code_flow(flow, {