fix: lock azure-storage-blob as breaking tests + enabling ignored tests

microsoft · Jul 23, 2024 · abbf594 · abbf594
1 parent 99ad5cc
commit abbf594
Show file tree

Hide file tree

Showing 6 changed files with 214 additions and 242 deletions.
diff --git a/azure-quantum/requirements.txt b/azure-quantum/requirements.txt
@@ -1,6 +1,8 @@
 azure-core>=1.30,<2.0
 azure-identity>=1.17,<2.0
-azure-storage-blob>=12.20,<13.0
+# TODO: recent versions break recordings. 
+# More than one match for "https://mystorage.blob.core.windows.net/.../rawOutputData"
+azure-storage-blob==12.20
 msrest>=0.7.1,<1.0
 numpy>=1.21.0,<2.0
 deprecated>=1.2.12,<2.0

diff --git a/azure-quantum/tests/unit/recordings/test_qiskit_submit_to_rigetti.yaml b/azure-quantum/tests/unit/recordings/test_qiskit_submit_to_rigetti.yaml
diff --git a/azure-quantum/tests/unit/test_cirq.py b/azure-quantum/tests/unit/test_cirq.py
@@ -159,7 +159,6 @@ def test_plugins_cirq_nonexistent_target(self):
 
     @pytest.mark.ionq
     @pytest.mark.live_test
-    @pytest.mark.skip(reason="TODO: ignoring for now due to undefined failure reason. Possibly, transient dependency update causing this")
     def test_plugins_ionq_cirq(self):
         with unittest.mock.patch.object(
             Job,

diff --git a/azure-quantum/tests/unit/test_ionq.py b/azure-quantum/tests/unit/test_ionq.py
@@ -48,20 +48,17 @@ def test_estimate_cost_ionq(self):
 
     @pytest.mark.ionq
     @pytest.mark.live_test
-    @pytest.mark.skip(reason="TODO: ignoring for now due to undefined failure reason. Possibly, transient dependency update causing this")
     def test_job_submit_ionq(self):
         self._test_job_submit_ionq(shots=None)
 
     @pytest.mark.ionq
     @pytest.mark.live_test
-    @pytest.mark.skip(reason="TODO: ignoring for now due to undefined failure reason. Possibly, transient dependency update causing this")
     def test_job_submit_ionq_100_shots(self):
         self._test_job_submit_ionq(shots=100)
 
 
     @pytest.mark.ionq
     @pytest.mark.live_test
-    @pytest.mark.skip(reason="TODO: ignoring for now due to undefined failure reason. Possibly, transient dependency update causing this")
     def test_job_submit_ionq_100_shots_with_deprecated_num_shots(self):
         # Call submit with a depteracted 'num_shots' argument, need to emit a deptecation warning.
         with pytest.warns(
@@ -134,7 +131,6 @@ def test_job_submit_ionq_with_conflicting_shots_from_input_params(self):
 
     @pytest.mark.ionq
     @pytest.mark.live_test
-    @pytest.mark.skip(reason="TODO: ignoring for now due to undefined failure reason. Possibly, transient dependency update causing this")
     def test_job_submit_ionq_cost_estimate(self):
         job = self._test_job_submit_ionq(shots=None)
         self.assertIsNotNone(job.details)

diff --git a/azure-quantum/tests/unit/test_microsoft_elements_dft.py b/azure-quantum/tests/unit/test_microsoft_elements_dft.py
@@ -9,7 +9,6 @@
 class TestMicrosoftElementsDftJob(QuantumTestBase):
 
     @pytest.mark.microsoft_elements_dft
-    @pytest.mark.skip(reason="TODO: ignoring for now due to undefined failure reason. Possibly, transient dependency update causing this")
     def test_dft_success(self):
         dft_input_params = {
             "tasks": [
@@ -52,7 +51,6 @@ def test_dft_failure_invalid_input(self):
 
 
     @pytest.mark.microsoft_elements_dft
-    @pytest.mark.skip(reason="TODO: ignoring for now due to undefined failure reason. Possibly, transient dependency update causing this")
     def test_dft_failure_algorithm_produces_output(self):
         dft_input_params = {
             "tasks": [

diff --git a/azure-quantum/tests/unit/test_qiskit.py b/azure-quantum/tests/unit/test_qiskit.py
@@ -286,7 +286,6 @@ def test_qir_to_qiskit_bitstring(self):
         ))
         self.assertEqual(AzureQuantumJob._qir_to_qiskit_bitstring(bitstring), bitstring)
 
-    @pytest.mark.skip(reason="TODO: ignoring for now due to undefined failure reason. Possibly, transient dependency update causing this")
     def test_qiskit_submit_ionq_5_qubit_superposition(self):
         workspace = self.create_workspace()
         provider = AzureQuantumProvider(workspace=workspace)
@@ -407,14 +406,12 @@ def test_plugins_estimate_cost_qiskit_ionq(self):
 
     @pytest.mark.ionq
     @pytest.mark.live_test
-    @pytest.mark.skip(reason="TODO: ignoring for now due to undefined failure reason. Possibly, transient dependency update causing this")
     def test_plugins_submit_qiskit_to_ionq(self):
         circuit = self._3_qubit_ghz()
         self._test_qiskit_submit_ionq(circuit)
 
     @pytest.mark.ionq
     @pytest.mark.live_test
-    @pytest.mark.skip(reason="TODO: ignoring for now due to undefined failure reason. Possibly, transient dependency update causing this")
     def test_plugins_submit_qiskit_circuit_as_list_to_ionq(self):
         circuit = self._3_qubit_ghz()
         self._test_qiskit_submit_ionq([circuit])
@@ -435,7 +432,6 @@ def test_plugins_submit_qiskit_multi_circuit_experiment_to_ionq(self):
 
     @pytest.mark.ionq
     @pytest.mark.live_test
-    @pytest.mark.skip(reason="TODO: ignoring for now due to undefined failure reason. Possibly, transient dependency update causing this")
     def test_plugins_submit_qiskit_qobj_to_ionq(self):
         from qiskit import assemble
 
@@ -804,7 +800,6 @@ def test_qiskit_get_ionq_native_gateset(self):
 
     @pytest.mark.ionq
     @pytest.mark.live_test
-    @pytest.mark.skip(reason="TODO: ignoring for now due to undefined failure reason. Possibly, transient dependency update causing this")
     def test_plugins_retrieve_job(self):
         workspace = self.create_workspace()
         provider = AzureQuantumProvider(workspace=workspace)
@@ -1096,7 +1091,6 @@ def test_configuration_quantinuum_backends(self):
 
     @pytest.mark.rigetti
     @pytest.mark.live_test
-    @pytest.mark.skip(reason="TODO: ignoring for now due to undefined failure reason. Possibly, transient dependency update causing this")
     def test_qiskit_submit_to_rigetti(self):
         from azure.quantum.target.rigetti import RigettiTarget