search

co9olguy / qdata

1 stars 3 forks source link

Add tests #9

Closed josh146 closed 4 years ago

josh146 commented 4 years ago

The beginnings of a test suite have been added in #8; the existing tests only test the ArithmeticOp class at the moment. However, we should extend this to include unit tests, plus integration tests that run on the example QASM scripts.

therooler commented 4 years ago

I can work on improving the test suite today.

co9olguy commented 4 years ago

@therooler Great! Let us know if you have any questions. Note that everything is still very much at a prototyping level, so if things don't make sense, nothing is set in stone yet

therooler commented 4 years ago

Cool. Btw, are you guys on Python 3.8 as well? And do you have:

pytest==6.1.0
lark-parser==0.10.0
sympy==1.6.2

?

co9olguy commented 4 years ago

For what it's worth, I'm using Python 3.7, lark-parser==0.9.0 and sympy==1.6.2 (no pytest installed in this env yet), but I don't think you'll see any difference using newer versions