spring-security-samples icon indicating copy to clipboard operation
spring-security-samples copied to clipboard

OAuth Sample Do not Test Actual Configuration

Open rwinch opened this issue 7 years ago • 6 comments

Summary

As it stands the tests that are in Spring Security's OAuth Samples do not test the actual configuration. They are instead copying and modifying the samples to allow testing.

This might seem innocent enough, but we don't want our users to follow this pattern. We need to demonstrate how the existing configuration can be used with perhaps mocking a small piece of the application or running a MockWebServer instance

rwinch avatar May 25 '18 17:05 rwinch

@rwinch I'm not sure I understand the issue with the test(s). Can you please provide more detail.

jgrandja avatar Jan 15 '19 21:01 jgrandja

OAuth2LoginApplicationTests currently uses SecurityTestConfig for the integration tests. We should remove SecurityTestConfig so that we're actually testing the WebSecurityConfigurerAdapter that the samples uses.

We also want the integration tests to follow the same pattern as in FormJcTests where pages are defined.

jgrandja avatar Jan 16 '19 17:01 jgrandja

@raphaelDL Would you be interested in taking this on?

jgrandja avatar Jan 19 '19 11:01 jgrandja

sure

raphaelDL avatar Jan 19 '19 14:01 raphaelDL

@raphaelDL have you had a chance to look into this?

eleftherias avatar Oct 16 '19 18:10 eleftherias

apologies, can someone take over this?

raphaelDL avatar Nov 13 '19 23:11 raphaelDL