Views Integration - Consider compatibility with internal page cache

My apologies for the comment in #13. I scanned the related issue without checking for the implementation of cache tags. In fact, there is explicit test coverage that shows the index list tag is missing for the 'None' plugin in ViewsDisplayCachingTest.

I agree with #12 that adding the list tag on the query is the preferred approach here, since it saves us the headache of adding an upgrade path for existing Search API views that use the existing None plugin.

Attached patch implements the following changes compared to #6:

1. Adds test coverage to check the search_api_list cache tag is added when using the 'None' cache plugin
2. Removed the getCacheTags implementation on the Search API Tag cache plugin, as its tags are now available through the SearchApiQuery

ViewsDisplayCachingTest still passes without changes to the Search API Time cache plugin, so it appears it is not necessary to remove the list tag again as suggested in #12

The last submitted patch, 15: 2905497-15.patch, failed testing. View results →

Attached patch implements the following changes:

1. Updated assertions for existing tests to include the list cache tag.
2. Turns out the Search API Time-based cache plugin needs to removed the list cache tag after all.

The last submitted patch, 17: 2905497-17-test-only.patch, failed testing. View results →

Tested on our project and the correct search_api_list cache tag now gets added to all pages that show Search API generated Views.

Thanks, great work! Also thanks @Lendude for testing.
Just two tiny corrections – if the test bot is happy with those then I’ll commit.

Committed.
Thanks a lot again, everyone!

Automatically closed - issue fixed for 2 weeks with no activity.