AF-2937 Add hooks to support tracing

[greglittlefield-wf]

Ultimate problem:

I forgot to add the core changes I needed in #193 🤦‍♂️

• We needed some new lifecyle hooks to to implement tracing of Flux components. These changes mirror those in AF-2888 Accommodate tracing of Store triggers and resulting redraws w_flux#120; see that PR for more info.

How it was fixed:

• Add lifecycle methods to allow subclasses to safely hook into and override store redraw logic These changes mirror those in AF-2888 Accommodate tracing of Store triggers and resulting redraws w_flux#120; see that PR for more info.
  - listenToStoreForRedraw
  - handleRedrawOn

Also fixed test setup for changes I made in #193:

• 0a7f393 "Make stateful test components actually stateful >_>"

Testing suggestions:

• Verify tests pass

Potential areas of regression:

Flux component store subscriptions

---

FYA: @greglittlefield-wf @aaronlademann-wf @kealjones-wk @evanweible-wf @maxwellpeterson-wf

[aviary-wf]

Security Insights

No security relevant content was detected by automated scans.

Action Items

• Review PR for security impact; comment "security review required" if needed or unsure
• Verify aviary.yaml coverage of security relevant code

---

Questions or Comments? Reach out on HipChat: InfoSec Forum.

[greglittlefield-wf]

Whitespace-agnostic diff in this file may help when reviewing

[codecov-io]

Codecov Report

Merging #195 into master will increase coverage by 0.27%.
The diff coverage is 90.91%.

@@            Coverage Diff             @@
##           master     #195      +/-   ##
==========================================
+ Coverage   94.27%   94.53%   +0.27%     
==========================================
  Files          34       34              
  Lines        1640     1645       +5     
==========================================
+ Hits         1546     1555       +9     
+ Misses         94       90       -4

[aaronlademann-wf]

Should this have @mustCallSuper as well?

[greglittlefield-wf]

Good question. No, since there are some cases where the listening setup might use a different mechanism.

Here's an example override

  @override
  void listenToStoreForRedraw(Store store) {
    if (store is TracedStore) {
      listenToStream(store.streamWithPayload, handleRedrawOnWithPayload);
    } else {
      super.listenToStoreForRedraw(store);
    }
  }

[aaronlademann-wf]

+1

[kealjones-wk]

+1