Add some basic test support for SourcePluginManager as a starting point.

CommentFileSizeAuthor
#2 sourcepluginmanager_test-3091959-2.patch7.54 KBplopesc

Comments

plopesc created an issue. See original summary.

plopesc’s picture

plopesc
he/him
Primary language Spanish
commented
Issue summary: View changes
StatusFileSize
new sourcepluginmanager_test-3091959-2.patch7.54 KB

Besides some plugin tests, removed dependency on migrate_source_yaml module. This module is only necessary when using the yaml source plugin, but not in case we add more source plugin instances in the future.

plopesc’s picture

plopesc
he/him
Primary language Spanish
commented
Status: Active » Needs review

Update status

nickdjm’s picture

nickdjm
he/him
commented
Assigned: Unassigned » nickdjm

Reviewing now. I am going to spilt out the dependency changes into their own issue since that's big enough to warrant it.

nickdjm’s picture

nickdjm
he/him
commented
Parent issue: » #3418730: 2.0 Roadmap
nickdjm’s picture

nickdjm
he/him
commented

This will go into v3 now.