fix: remove Status method from provider

[sahidvelji]

This PR

This PR removes the Status() method from the provider.

Related Issues

Related to #299

Notes

Follow-up Tasks

How to test

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.76%. Comparing base (9908b97) to head (ac8b047).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #395      +/-   ##
==========================================
+ Coverage   85.64%   85.76%   +0.11%     
==========================================
  Files          20       20              
  Lines        1442     1440       -2     
==========================================
  Hits         1235     1235              
+ Misses        186      184       -2     
  Partials       21       21              

Flag	Coverage Δ
e2e	85.76% <ø> (+0.11%)	⬆️
unit	85.76% <ø> (+0.11%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[aepfli]

[question] is this breaking? Could this be expected somewhere?

Anyways thank you for your continuous efforts!!!!!

[sahidvelji]

[question] is this breaking? Could this be expected somewhere?

It is theoretically breaking, yes. Practically speaking though, I can't imagine many users querying the status of a noop provider via noopProvider.Status() anywhere. The Status method on providers is definitely not expected anywhere in this SDK though. There is no interface that requires it, and a search for "status" across the repo doesn't turn up anything notable.