using string as a parameter name is bad practice because it overwrites the string module

Ah, we've gone full circle now.

I see what you mean now - "remove" is definitely not the correct word to describe the expected operation for this Kata. Thank you for your input.

Well not just implies, that's literally what the word "remove" means
Well it's not that big of a deal since I eventually got the memo from the types

removing implies that the input will get shorter because we move the characters

"people would assume you want move the characters from a string to a different string" - that's precisely the challenge. Extract the n-th string term and return it. You must do so by allocating the characters in a new string. Am I missing something?

follows is pretty simple
put the 1 at the end

You really need to work on your wording. Because from the description people would assume you want move the characters from a string to a different string.

But it's a const string which is the only reason I could guess otherwise

maybe 6 or something

Second test case says otherwise:

Assert::That(PythagoreanTriple(5,3,4) , Equals(true));

lol

ok, it's not

Quite funny it is legal :)

It does not affect on code.
I think Blackw00D forgot to errase that line

a typo