wmlele / devise-otp

Two Factors authentication for Devise using Time Based OTP/rfc6238 tokens.
MIT License
210 stars 40 forks source link

Switch test suite to default Rack Test #82

Closed strouptl closed 5 months ago

strouptl commented 5 months ago

The test suite utilizes the cuprite driver, which causes compatibility issues with the Warden redirect solution in PR #80 (produces a "Chrome data error" when querying the current_url). In addition, the current testing approach does not test the actual content of the recovery codes download (only that something was downloaded).

This PR switches the test suite to the default rack_test driver, which resolves the test failures in PR #80, adds tests for the actual content of the recovery codes download, and simplifies the codebase by removing the custom Capybara coniguration and DowloadHelper module.