Tweak data source warning

[fractalwrench]

Goal

Tweaks the log level & text shown when the data capture limit is reached in DataSourceImpl. This caused a bit of confusion so a lower severity level & clearer message is desirable. I've also altered the message to include the class name that has run into the limit.

Testing

Relied on existing test coverage & also tested out message manually in the test app:

Data capture limit reached for io.embrace.android.embracesdk.capture.crumbs.BreadcrumbDataSource. Ignoring to keep payload size reasonable - other data types will still be captured normally.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.92%. Comparing base (c7297cc) to head (21a67a8).
Report is 3 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #876   +/-   ##
=======================================
  Coverage   80.91%   80.92%           
=======================================
  Files         434      434           
  Lines       11502    11502           
  Branches     1753     1753           
=======================================
+ Hits         9307     9308    +1     
+ Misses       1427     1422    -5     
- Partials      768      772    +4     

Files	Coverage Δ
...ndroid/embracesdk/anr/sigquit/SigquitDataSource.kt	36.66% <100.00%> (ø)
...droid/embracesdk/arch/datasource/DataSourceImpl.kt	94.73% <100.00%> (+0.29%)	⬆️
...ndroid/embracesdk/arch/limits/UpToLimitStrategy.kt	100.00% <ø> (ø)
...ndroid/embracesdk/capture/aei/AeiDataSourceImpl.kt	85.36% <100.00%> (ø)
...dk/capture/connectivity/NetworkStatusDataSource.kt	96.15% <100.00%> (ø)
.../embracesdk/capture/crumbs/BreadcrumbDataSource.kt	92.85% <100.00%> (ø)
...cesdk/capture/crumbs/PushNotificationDataSource.kt	87.09% <100.00%> (ø)
...id/embracesdk/capture/crumbs/RnActionDataSource.kt	85.00% <100.00%> (ø)
...android/embracesdk/capture/crumbs/TapDataSource.kt	70.83% <100.00%> (ø)
...ndroid/embracesdk/capture/crumbs/ViewDataSource.kt	81.25% <100.00%> (ø)
... and 5 more

... and 4 files with indirect coverage changes

[bidetofevil]

Do we want to output this multiple times or just once per session?

[fractalwrench]

IMO let's do it multiple times for now, then following feedback from customers change to once per session if it's too spammy. I'm hoping that setting the log level & increasing descriptiveness of message will be enough of a change

[bidetofevil]

LGTM. One comment about whether we want to keep logging this once we've done it once for a particular data source, but not a blocking comment