Handle list of QuantumCircuit objects in job memory mapping

[lasys]

Summary

Fixes #46

Details and comments

set self.qobj to given QuantumCircuit

[lasys]

@mtreinish could you please review the PR? Thanks :)

[mtreinish]

Thanks for pushing the PR. I think it will work, but I left an inline suggestion to make the intent a bit clearer.

[mtreinish]

Is this needed?

[lasys]

The test failed locally, so i added this.
If it doesn't fail on GitHub and/or your system, i can removed it.

[mtreinish]

It's going to block CI, the lint rules block unused imports and nothing is using mock plain (it's only used as unittest.mock)

[lasys]

You're right. I removed it and ran tox, so hopefully the CI goes green.

[mtreinish]

The other thing which would be good to have here is a test case for a job with a list of quantum circuits to verify the change works. Something like test_job_counts_measurement_mapping but passing [qc] to the fake job instead of using qc.

[lasys]

The other thing which would be good to have here is a test case for a job with a list of quantum circuits to verify the change works. Something like test_job_counts_measurement_mapping but passing [qc] to the fake job instead of using qc.

Good point. I will write one.

[mtreinish]

LGTM, thanks for the quick updates and fixing this!