Skip to content

Add tests for SSO + user interactive authentication.#845

Merged
clokep merged 10 commits intodevelopfrom
clokep/ui-interactive-auth-for-cas
Apr 17, 2020
Merged

Add tests for SSO + user interactive authentication.#845
clokep merged 10 commits intodevelopfrom
clokep/ui-interactive-auth-for-cas

Conversation

@clokep
Copy link
Member

@clokep clokep commented Mar 31, 2020

This adds tests for using SSO in the user interactive authentication flow. It requires matrix-org/synapse#7186.

@clokep clokep requested a review from richvdh April 3, 2020 19:37
@clokep clokep requested a review from richvdh April 9, 2020 18:10
@clokep clokep force-pushed the clokep/ui-interactive-auth-for-cas branch from 9480cf3 to cb5991e Compare April 15, 2020 12:14
@clokep
Copy link
Member Author

clokep commented Apr 15, 2020

Looks like rebasing this got the other test passing. 👍 This is ready for review!

Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

Comment on lines +6 to +7
# A convenience function which wraps await_http_request. It returns a successful
# CAS response when queried for a particular path.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it'd be nice to document what the params can be, but 🤷‍♂️

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added some.

@clokep clokep merged commit a22ca2d into develop Apr 17, 2020
@clokep clokep deleted the clokep/ui-interactive-auth-for-cas branch April 17, 2020 11:57
anoadragon453 added a commit that referenced this pull request Apr 27, 2020
…rite_whitelist_annotation

* 'develop' of github.com:matrix-org/sytest: (42 commits)
  Fix OFFLINE support for the sytest-synapse docker image (#858)
  Add support for running worker mode against redis (#857)
  route GET account_data requests to client_reader (#855)
  Test that users receive updates for their own devices (#854)
  Add tests for SSO + user interactive authentication. (#845)
  Modify login tests to be spec compliant
  Make message history pagination test less flaky (#851)
  Small comment fixes for consistency (#808)
  Change room_fixture to run matrix_create_room_synced by default (#848)
  Update dendrite docs with new whitelist/blacklist info (#849)
  Add test for querying remote public rooms list with local server_name (#847)
  Add more logging to "Guest user can set display names" (#846)
  Add test for updating room aliases on upgraded room join (#844)
  Add tests to ensure that the operation cannot be modified during interactive auth. (#830)
  fix the CAS login test (#829)
  Update NOT_FOUND error to BAD_ALIAS error to be spec compliant. (#837)
  Missing bits
  Use r0 instead of 'unstable' for joined_members|rooms
  Add test to optionally keep other sessions when modifying a user's password. (#832)
  Remove tests that call query_auth endpoint. (#827)
  ...
anoadragon453 added a commit that referenced this pull request Jun 10, 2020
…into dinsic

* 'release-v1.13.0' of https://github.com/matrix-org/sytest: (23 commits)
  Dendrite: Show sytest compliance report after sytest runs (#864)
  Provide some examples of running sytest with worker mode synapse (#862)
  Fix build, don't build into source directory
  Maybe this time
  Better
  Just build the monolith server when testing
  Test for sending to_device messages to multiple recipients at once (#856)
  Mention required debian and cpan dependencies (#859)
  Add whitelist information to dendrite buildkite annotation (#803)
  Fix OFFLINE support for the sytest-synapse docker image (#858)
  Add support for running worker mode against redis (#857)
  Add tests for SSO + user interactive authentication. (#845)
  Modify login tests to be spec compliant
  Make message history pagination test less flaky (#851)
  Small comment fixes for consistency (#808)
  Change room_fixture to run matrix_create_room_synced by default (#848)
  Update dendrite docs with new whitelist/blacklist info (#849)
  Add test for querying remote public rooms list with local server_name (#847)
  Add more logging to "Guest user can set display names" (#846)
  Add test for updating room aliases on upgraded room join (#844)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants