The original code used boolValue

[echo-branch]

Reference

SDK-2250

Summary

Swap unused string check from doubleValue to boolValue

Motivation

Be consistent with the code it is intended to replace.

Type Of Change

• Bug fix (non-breaking change which fixes an issue)

Testing Instructions

Unit test has been updated. Note this utility code isn't actually used by the SDK.

cc @BranchMetrics/saas-sdk-devs for visibility.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 51.27%. Comparing base (16aae6a) to head (1b03a81).
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1356      +/-   ##
==========================================
- Coverage   51.51%   51.27%   -0.24%     
==========================================
  Files          66       66              
  Lines       10092    10092              
  Branches     3698     3697       -1     
==========================================
- Hits         5199     5175      -24     
- Misses       4628     4654      +26     
+ Partials      265      263       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.