[NDB_BVL_Instrument] Test name should match CommentID

[CamilleBeau]

Describe the bug
Currently when instantiating an instrument with a Test_name and CommentID, the CommentID does not have to belong to the same instrument as the Test_name. As long as the Test_name exists in the database, it can be used as an argument for any CommentID.

There should be a warning or error thrown in these cases to ensure that the right Test_name is being passed as an argument to the factory method.

[driusan]

What is the current behaviour if they don't match/how is this coming up?

[CamilleBeau]

What is the current behaviour if they don't match/how is this coming up?

@driusan If they don't match, it makes the instrument from the CommentID. This is coming up because we were making a script on CCNA that should have failed during testing as CommentIDs were being matched with different Test_names, but it did not.

[CamilleBeau]

Still an issue