Fixed concept ancestry bug and added tests to make sure this is detected

[shanmukhasampath]

This is for issue #51

Added tests and they are working.

[javier-gracia-tabuenca-tuni]

adding the whole ICD9 usagi seems too much for testing one line only
this affects the speed of the other unit tests and total size
can you just add only the needed line to the ICD10 usagi file
or
is it even necesary the ICD9 if in the ICD10 the ancesto to the own usagi file is also contemplated ??

[shanmukhasampath]

adding the whole ICD9 usagi seems too much for testing one line only this affects the speed of the other unit tests and total size can you just add only the needed line to the ICD10 usagi file or is it even necesary the ICD9 if in the ICD10 the ancesto to the own usagi file is also contemplated ??

Hi @javier-gracia-tabuenca-tuni, it is not necessary I can change it to ICD10 Usagi file. Initially, I wanted to check whether the NA and empty string match happens in other vocabularies as well. There are instances in ICD10 where this is the case, I can fix this

[shanmukhasampath]

Hi @javier-gracia-tabuenca-tuni, I have fixed all the files and it showed no problems in my Rstudio but somehow it fails in the check. Will you please let me know what it is that I am missing on my side.

[javier-gracia-tabuenca-tuni]

DQD package has been updated with new tests,
it tests if some tables are empty, which they are in this test database, as we dont use all tables here
it fails in github bcs it takes the latest version whereas in your laptop you may have a previus version
I will fix this

[shanmukhasampath]

Hi @javier-gracia-tabuenca-tuni,
thanks for the message.