xwp / unsplash-wp

GNU General Public License v2.0
9 stars 3 forks source link

Feature: Add authentication flow & tests #107

Closed derekherman closed 4 years ago

derekherman commented 4 years ago

Summary

This pull request adds the completed authentication flow with full test coverage and is intended to be merged into the 1.0-release-prep branch before it gets merged.

Coverage goes us to 93.13% with this PR and all linting passes.

Implements #108

Checklist

spacedmonkey commented 4 years ago

This PR looks great!

There are a couple of nit picks, but nothing that blocks this PR from being merge. Just things we may want to look at later.