Codecov Report

Merging#1248 intomaster willnot change coverage.
The diff coverage isn/a.

@@           Coverage Diff           @@##           master    #1248   +/-   ##=======================================  Coverage   86.25%   86.25%           =======================================  Files           1        1             Lines         291      291           =======================================  Hits          251      251             Misses         40       40

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

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last updatef808166...1a3d75d. Read thecomment docs.

What about interfaces?

What about interfaces?

That is already handled by#1246. The calculation of whether something has an indexer or not is done the same way for classes and interfaces. It is all based on the implementation class.

@danabr just to clarify. So if I haveinterface IExtra: IIndexed with indexer defined inIIndexed, but not inIExtra, and in Python doextra = IExtra(extra_impl) (e.g. explicit cast toIExtra), I would be able to doextra[1234]?

If so, I think a test for that scenario should be added totest_indexer_accessed_through_interface introduced in your previous PR.

Yes. The requested test has been added.

This closes#256