Add tests for passwords#edit when redirect_whitelist is set

[MaicolBen]

Missing tests from #1228

[MaicolBen]

this was wrong lol, I am glad that I did this

[dks17]

You could replace these parts by get_reset_token, they look the same:

        describe 'case-sensitive email' do
          before do
            post :create,
                 params: { email: @resource.email,
                           redirect_url: @redirect_url }

            @mail = ActionMailer::Base.deliveries.last
            @resource.reload
            @data = JSON.parse(response.body)

            @mail_config_name  = CGI.unescape(@mail.body.match(/config=([^&]*)&/)[1])
            @mail_redirect_url = CGI.unescape(@mail.body.match(/redirect_url=([^&]*)&/)[1])
            @mail_reset_token  = @mail.body.match(/reset_password_token=(.*)\"/)[1]
          end

    describe 'Alternate user class' do
      before do
        @resource = create(:mang_user, :confirmed)
        @redirect_url = 'http://ng-token-auth.dev'

        post :create, params: { email: @resource.email,
                                redirect_url: @redirect_url }

        @mail = ActionMailer::Base.deliveries.last
        @resource.reload

        @mail_config_name  = CGI.unescape(@mail.body.match(/config=([^&]*)&/)[1])
        @mail_redirect_url = CGI.unescape(@mail.body.match(/redirect_url=([^&]*)&/)[1])
        @mail_reset_token  = @mail.body.match(/reset_password_token=(.*)\"/)[1]
      end

    describe 'alternate user type' do
      before do
        @resource = create(:user, :confirmed)
        @config_name  = 'altUser'

        post :create, params: { email: @resource.email,
                                redirect_url: @redirect_url,
                                config_name: @config_name }

        @mail = ActionMailer::Base.deliveries.last
        @resource.reload

        @mail_config_name  = CGI.unescape(@mail.body.match(/config=([^&]*)&/)[1])
        @mail_redirect_url = CGI.unescape(@mail.body.match(/redirect_url=([^&]*)&/)[1])
        @mail_reset_token  = @mail.body.match(/reset_password_token=(.*)\"/)[1]
      end

[dks17]

You could also delete this:

Suggested change

@auth_headers = @resource.create_new_auth_token

because it appears in another before block which is above:

describe 'not redirect_url should return 401' do
  before do
    @auth_headers = @resource.create_new_auth_token
    @new_password = Faker::Internet.password
  end

[dks17]

In general, everything looks good.

[dks17]

Some tests failed. I have released PR #1249 to fix it.