Fix docs how to register all possible interceptors

[Vampire]

No description provided.

[Vampire]

Current dependencies on/for this PR:

• master
  • PR Fix docs how to register all possible interceptors #1824 👈
    • PR Test the interceptor lifecycle #1668
      • PR Add SpecInfo#getAll...Methods() methods #1770
        • PR Fix the lifecycle of simple features #1675
          • PR Verify and fix interceptor contexts  #1676
            • PR Verify the feature is given to method invocations #1677
              • PR Verify the iteration is given to method invocations #1678
                • PR Verify the instances are given correctly to method invocations #1679
                  • PR Verify the target is given correctly to method invocations #1680
                  • PR Verify the method is given to method invocations #1681
                  • PR Verify the method reflection is given to method invocations #1682
                  • PR Verify the method name is given to method invocations #1683
                  • PR Verify the specification is given to specification context #1684
                  • PR Verify the feature is given to specification context #1685
                  • PR Verify the iteration is given to specification context #1686
      • PR Add SpecInfo#getAllSharedInitializerMethods() #1669
        • PR Add SpecInfo#getAllSetupSpecMethods() #1670
          • PR Add SpecInfo#getAllInitializerMethods() #1671
            • PR Add SpecInfo#getAllSetupMethods() #1672
              • PR Add SpecInfo#getAllCleanupMethods() #1673
                • PR Add SpecInfo#getAllCleanupSpecMethods() #1674

This stack of pull requests is managed by Graphite.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (19c2bde) 80.64% compared to head (1c49943) 80.64%.

Additional details and impacted files

@@            Coverage Diff            @@
##             master    #1824   +/-   ##
=========================================
  Coverage     80.64%   80.64%           
- Complexity     4258     4260    +2     
=========================================
  Files           434      434           
  Lines         13265    13265           
  Branches       1687     1688    +1     
=========================================
  Hits          10698    10698           
  Misses         1952     1952           
  Partials        615      615           

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

[leonard84]

🤔 I think we should add a disclaimer that although this will thoroughly intercept all possible things, this isn't actually what we recommend doing.
Furthermore, shouldn't it be specInfo.bottomSpec.allFeatures*., at least for IAnnotationDrivenExtension? IGlobalExtension will always apply only to the bottom spec.

We should probably document the specification hierarchy to better explain why bottomSpec/specsBottomToTop might be necessary and when it isn't.

[Vampire]

I added a disclaimer and there also mention considerations for local extensions regarding features vs. allFeatures and bottomSpec vs. not bottomSpec.