Comments by C Pottinger (Top 19 by date)

'IConfigurable is missing a "Name"'
Actually, no. That was the idea. IConfigurableShape would not expose Name, but IDisplayOnlyShape would.

'IConfigurable maybe should inherit IReadOnly instead'
I think you might have meant to say 'IDisplayOnlyShape would...'. That is the "read-only" interface. Perhaps my example was not clear.

Despite that, you have offered what I think will be best solution: by having 2 private constructions, each called by different static factory methods. I can then have IConfigurableShape expose one method and IDisplayOnlyShape expose the other, thus limiting the use of the constructors.
THANK YOU.

However, PIEBALDconsult's comment above still holds true - I can't stop a user from casting from one type to another. Still, a little control is better than none.

I see your point. Thank you for the speedy reply.
In the future I will stick with creating two derived classes, but for this project, I will continue with the above structure and run the risk of being taken out by a Predator drone.

I see what you are saying, but
a) I put production in quotes because what I really wanted to refer to was my 'program code' as opposed to my 'testing code'. I think my program code should not have to know anything about my testing code.
b) #DEFINE would be fine to separate my debugging code from my final code. But even if I did that, I would still have the issue that my program code contains knowledge of my unit tests.
c) Even with conditional compilation, I would still run into the issue that Assembly.GetTypes() would pick up types dynamically generated by Moq whenever I unit test.

Thanks Richard. But that does seem like an awful lot of work to go through just to not have Moq interfere with Assembly.GetTypes().

In the end, it turned out I had to refactor my code, for unrelated reasons, and in doing so it allowed me to use dependency injection to provide a class that does the actual loading of the Types from a dll. I can now mock this new IPluginTypesLoader interface and avoid calling System.Reflection.Assembly.GetTypes() from within my unit tests.

Thank you, Richard. That worked - and better yet: I understand why it worked!

And thanks for the links - I will be looking into utilizing both suggestions.

Ah - answered by kieran501 on StackOverflow (https://stackoverflow.com/questions/65425079/how-do-i-get-mysqldataadapter-to-set-the-table-name-and-columns-using-tablemappi/65425599#65425599)
I just needed to reverse the order the parameters in the ColumnMappings calls (which I did try), AND fill "testtable" instead of "MyData" (which I hadn't).

Nope - this did not do it. I was sure it did, but upon further inspection it does set the ds.Tables[0] TableName = "MyData". but looking for ds.Tables["MyData"].Rows[0]["MyDataA"] fails while ds.Tables["MyData"].Rows[0]["column_a"] returns "entry_1".
Aaaaaaaarrrrrrrrggggggggghhhhhhhhh !!!!!!!!

Thank you Gerry. I'll do that once the other issue is resolved, namely, why does the code above not map the retrieved data to a table in ds called MyData and columns called MyDataA and MyDataB?
Or am I missing some fundamental concept about what TableMapping is for?

Thank you, Richard.
I added the XmlElementAttribute override to the overrides object and applied the overrides to the Instruction type, as you suggested, The code is now working perfectly.
And thank you for the explanation of how to apply the override to just the Extractor class. I'm not doing that, in this case, but it is good know that it is an option.

Perfect answer.

Thank you, Graeme. That makes perfect sense as I look it now.

T E R R I F I C ! ! !
Thank you, Richard. This is exactly what I wanted.

Last night I was thinking about how I had constructed my example for the question and realized that I could have done better. When I saw your answer this morning I tested it using my new model:

Func<double, Func<double, double>> areaOfCircle = factor => r => { Console.WriteLine("area factor is {0}", factor); return (double)(factor * Math.Pow(r, 2)); }; 
Func<double, Func<double, double>> diameterOfCircle = factor => r => { Console.WriteLine("diameter factor is {0}", factor); return (double)(2 * factor * r); };		

var funcList = new List<Func<double, double>>() 
{
	diameterOfCircle(Math.PI),
	areaOfCircle(Math.PI),
	diameterOfCircle(3),
	areaOfCircle(3.14),
};

foreach(var iNeed in funcList)
{
	//Console.WriteLine(iNeed(5));
	Console.WriteLine(iNeed(10));
}

It worked perfectly.

And the article by Jon Skeet explained the workings and background very well.
Thanks again.

Yup. Makes sense. Feeling silly now...

Thanks Dave. Sometimes you can't see the forest for all the trees.

Thank you. That taught me enough to come up with this script:

#!/bin/bash

MIN_ACT=200

echo Looking for running qBitorrent. Please wait...
TORR_PID=$( ps aux | grep '[q]bittorrent' | awk '{ print $2 }' )
if [ -z "$TORR_PID" ]; then
    echo qBittorrent not found.
else
    echo Collecting traffic infomation. Please wait...
    ACTIVITY=$( collectl -c1 -i5 -sn | awk 'NF==4 { print $1 }' )
    if [ $ACTIVITY -lt $MIN_ACT ]; then
        echo Traffic is low. Killing qBittorrent PID:$TORR_PID. Please wait...
        kill -s SIGTERM $TORR_PID
    fi
fi
echo killqb done.