List upload: add tests for organism

See comments here: #36

The testmodel has an idea of an extraValue. I forget what it is, company department maybe?

@yochannah

Also, @yochannah you updated the tutorial to include this new feature. Is that right? Can you make that live too?

@julie-sullivan

Hi @julie-sullivan @yochannah ,
I am new to this org. I would like to take up this issue to start contributing.

I explored the test structure. Here are my observations ->

The test for lists are stored in test_lists.py and mock data of test lists is stored in tests/testservice/service/list. But this test file contains test for only getting list while changes in Committing organism specification on list upload functionality #36 are in create_list.
Then I saw that you have mentioned extraValue. This is in test_core.py file. Since organism has constraint ONE OF it should be part of testMultiConstraint.

So, should I make changes to testMultiConstraint. Or am I making any mistake?

Also, as you had asked -> it appears extraValue is indeed Employee.department as used in line 865 in test_core.py

@22PoojaGaur

Hey @22PoojaGaur! You would be very welcome to take a look at this :) your deducations sound correct to me!

Navigation Menu