Add proper tests
Currently there is very little testing. This needs to be addressed.
Comments (43)
-
reporter -
reporter Split the RDSR import test into a new file. Checked it still functions. Refs
#427→ <<cset 0b58cf2d4eff>>
-
reporter Added in tests for patient ID being stored and being stored hashed. Will probably break this into a new file. Refs
#427→ <<cset 2834e2e1d3e1>>
-
reporter Moved the patient ID tests to a new file. Refs
#427→ <<cset 761e376c53d7>>
-
reporter Moved the patient ID tests to a new file. Refs
#427→ <<cset a171fffd8a97>>
-
reporter Adding some docs for running tests. Refs
#427→ <<cset c3f4ca1ae69f>>
-
reporter Adding some docs for running tests. Refs
#427→ <<cset 66f2944f7395>>
-
reporter Added yml file for bitbucket pipelines. Doesn't do anything yet, but refs
#427→ <<cset 6f4ca10a8e2f>>
-
reporter Modified tox to replicate working version locally. Added it to bb pipelines yaml. Refs
#427→ <<cset ab5d2eea2d3f>>
-
reporter Added setting up the local settings/wsgi. Refs
#427→ <<cset 1beb0ff0f1a7>>
-
reporter Removed comented out commands that aren't needed in the tox.ini file. Refs
#427→ <<cset b4334460eaa8>>
-
reporter updating tests in develop. Refs
#427→ <<cset 9963e8824f1c>>
-
reporter First attempt to integrate with coveralls.io. Refs
#427→ <<cset 7dc982fa5126>>
-
reporter removed verbosity to commit and prompt coveralls. Refs
#427→ <<cset d0bf3e0cb68f>>
-
reporter Adding the code for coveralls integration for develop. Refs
#427→ <<cset 7a84321fb130>>
-
reporter Added more tests for single source CT import. Refs
#427→ <<cset 44e92204a146>>
-
reporter Added some tests for the hash_id function. Refs
#427→ <<cset 3c9cf1b30454>>
-
reporter New test for get_value_kw with non-ASCII values. Refs
#427. Also some tidying, (spaces, new lines).→ <<cset 82bc81162684>>
-
reporter Bringing in the newer tests to develop. Refs
#427→ <<cset a8097efbc894>>
-
reporter Adding MG image object with much reduced pixel data. Contains UTF-8 characters, not currently processed correctly. Refs
#427→ <<cset 4539960dc53f>>
-
reporter Modified age test for mg. Refs
#427→ <<cset 4c09c3e3fbdd>>
-
reporter Added a test for importing mammo with patient identifiers being stored. Refs
#427→ <<cset d8ffbc5303e5>>
-
reporter Forgot to save the patient ID settings. Also ensured the test values were unicode. Refs
#427→ <<cset 80173fa47383>>
-
reporter Added hashed version of mammo import ID check. UFT-8 strings need to be bytecode before hashing. Need to fix on other extractors too. Refs
#427,#434→ <<cset 0079f3ac4d3c>>
-
reporter Adding check of hashed ASCII accession number. Need to have a non-ASCII version somewhere. Refs
#427→ <<cset 6d19de39acc0>>
-
reporter Added an initial test for exposure content of mammo test file. Refs
#427→ <<cset 9be6ff22afc1>>
-
reporter Another small test added to mammo import. Refs
#427→ <<cset dc4a5f1db04d>>
-
reporter Added exposure, kV and grid to mammo test. Refs
#427→ <<cset c7164e069f26>>
-
reporter Added target/filter to mammo test. Refs
#427→ <<cset 1963190d8470>>
-
reporter Main elements of mammo now tested on import. Need to add a second image and uplicate to finish off GE mammo. Refs
#427→ <<cset f629a8142d0d>>
-
reporter Fresh start for testing mammo export, making use of import test. Refs
#427→ <<cset 721615ac74d2>>
-
reporter Getting a little closer - I think the export won;t actually happen as it is tied into using celery... Refs
#427→ <<cset cc0a07daa5a3>>
-
reporter Committing for now, but just experimenting to see how best to deal with None task id when not using Celery. Refs
#427→ <<cset fcd35d4c5671>>
-
reporter I think this is the right solution for testing export without using celery. It should be noted that the triggerring of exports through the web interface isn't tested by this method. Refs
#427→ <<cset 42d5ed73bd71>>
-
reporter Now tests all the headers and values for the non pid export in one test. Refs
#427→ <<cset c5e47fe60177>>
-
reporter Pipelines build test failed due to pandas not being installed. Refs
#427→ <<cset 21e2f42e0c2e>>
-
reporter Added pandas install in pipelines yaml Refs
#427→ <<cset 7da41f20bc1f>>
-
reporter Added to setup instead. Only used in testing for now, but might be useful later! Refs
#427→ <<cset cb1a3f6b21c6>>
-
reporter Adding CT GE Optima CT660 RDSR for automated testing against. Refs
#427and will be useful for testing ref#443→ <<cset a5cac1383338>>
-
reporter Adding GE VCT sample RDSR for testing purposes. Refs
#427→ <<cset 6aa26ae5c5a1>>
-
reporter Start of tests for CT exports. Refs
#427,#443→ <<cset ba2c89e9d1e4>>
-
reporter Modified ctxlsx to allow exports when not using Celery. Fixed mistake in assert. Test fails, as expected. Refs
#427,#443→ <<cset d42359a316ed>>
-
reporter - changed status to closed
Range of tests now included - many more tests still need to be created! But this merge closes
#427→ <<cset 731a2a87db99>>
- Log in to comment
Added an anonymised Siemens single tube CT RDSR, and an initial simple test (that passes). Refs
#427→ <<cset b444b000d6b6>>