Refactor builder usage

[sankethkatta]

There were a lot of changes to the build system, testing files, and code interface in this PR. I pulled the Javadoc into a separate PR to make this cleaner to read.

I have not tried pulling this into an actual application and testing end-to-end. So DO NOT MERGE until the testing is complete!

Public Interface Changes

• Upgraded everything to target Android 29, but have minimum compatibility with Android 21
• getAuthUrl() has been removed completely in favor of using AuthUrlBuilder()
• a new method called authUrlBuilder() was added to make it easier to create a builder.
• launchAuthFlow(..., String authUrl) now can take in a url built by the builder
• addClickHandler(..., String authUrl) now can take in a url built by the builder
• Both the above methods have an overloaded version that takes no authUrl and is just builds the default url with no additional parameters (i.e. scope, single select, ...)
• SmartcarAuth() has only 2 overloaded constructors now (default live mode / test mode). I removed the feature of providing scope as a string as all of our other SDKs only accept it as an array.

Internal Changes

By setting state directly on SmartcarAuth and removing Helpers that are only used a single time we were able to remove many classes and create a simpler structure. The following classes and according tests were removed:

• Helper: All methods were trivial and only used once
• ApprovalPrompt: Defined an enum that is no longer relevant
• ResponseType: Defined an enum that is no longer relevant
• SmartcarAuthRequest: Stored state that was moved onto SmartcarAuth
• WebViewActivity: We now use Chrome customtabs

Build

• Since we don't actually run any tests which require then Android emulator, we don't need to run an Android emulator on Travis. (builds are now down to 5 mins!)
• We had some repetitive gradle commands being run, I've cleaned it so now we only use 2:
  • build - Assembles and tests the project
  • jacocoTestReport - Generates Jacoco coverage reports

End To End Testing

• [X] Successfully retrieve code
• [X] Use Make Bypass
• [x] Use Single Select
• [x] Retrieve access_denied error message
• [X] Retrieve vehicle_incompatible with vehicle information

[codecov[bot]]

Codecov Report

Merging #22 into master will increase coverage by 23.55%. The diff coverage is 90%.

@@             Coverage Diff             @@
##           master      #22       +/-   ##
===========================================
+ Coverage   67.59%   91.15%   +23.55%     
===========================================
  Files           9        4        -5     
  Lines         250      147      -103     
  Branches       18        9        -9     
===========================================
- Hits          169      134       -35     
+ Misses         76       11       -65     
+ Partials        5        2        -3

Impacted Files	Coverage Δ
...in/java/com/smartcar/sdk/SmartcarCodeReceiver.java	0% <ø> (ø)	:arrow_up:
...th/src/main/java/com/smartcar/sdk/VehicleInfo.java	95.23% <0%> (-4.77%)	:arrow_down:
...c/main/java/com/smartcar/sdk/SmartcarResponse.java	96% <0%> (-4%)	:arrow_down:
...h/src/main/java/com/smartcar/sdk/SmartcarAuth.java	95.74% <94.73%> (+26.4%)	:arrow_up:

---

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 6f7ae3b...a4d41ca. Read the comment docs.