Merge pull request #1166 from qiboteam/fix_test

Fix test failing and remove unnecessary prints from tests

tests/test_backends_clifford.py

@@ -136,7 +136,7 @@ def test_random_clifford_circuit(backend, prob_qubits, binary):
     clifford_bkd = construct_clifford_backend(backend)
     if not clifford_bkd:
         return
-    c = random_clifford(5, backend=backend)
+    c = random_clifford(5, seed=1, backend=backend)
     c.density_matrix = True
     c_copy = c.copy()
     c.add(gates.M(*MEASURED_QUBITS))
@@ -165,19 +165,12 @@ def test_random_clifford_circuit(backend, prob_qubits, binary):
 
         numpy_freq = numpy_result.frequencies(binary)
         clifford_freq = clifford_result.frequencies(binary)
-        print(numpy_freq)
-        print(clifford_freq)
         clifford_freq = {state: clifford_freq[state] for state in numpy_freq.keys()}
-        print(clifford_freq.values())
-        assert (
-            np.sum(
-                np.abs(
-                    np.array(list(numpy_freq.values()))
-                    - np.array(list(clifford_freq.values()))
-                )
-            )
-            < 200
-        )
+        assert len(numpy_freq) == len(clifford_freq)
+        for np_count, clif_count in zip(numpy_freq.values(), clifford_freq.values()):
+            backend.assert_allclose(
+                np_count / 1000, clif_count / 1000, atol=1e-1
+            )  # nshots = 1000
 
 
 def test_collapsing_measurements(backend):
@@ -224,7 +217,6 @@ def test_initial_state(backend):
     initial_symplectic_matrix = tmp.symplectic_matrix
     initial_state = numpy_bkd.execute_circuit(state).state()
     initial_state = np.outer(initial_state, np.transpose(np.conj(initial_state)))
-    print(type(initial_state))
     c = random_clifford(3, density_matrix=True, backend=backend)
     numpy_state = numpy_bkd.execute_circuit(c, initial_state=initial_state).state()
     clifford_state = clifford_bkd.execute_circuit(
@@ -253,5 +245,8 @@ def test_set_backend(backend):
     clifford_bkd = construct_clifford_backend(backend)
     if not clifford_bkd:
         return
-    set_backend("clifford")
+    platform = backend.platform
+    if platform is None:
+        platform = str(backend)
+    set_backend("clifford", platform=platform)
     assert isinstance(GlobalBackend(), type(clifford_bkd))

tests/test_measurements.py

@@ -457,9 +457,7 @@ def test_measurement_basis_list(backend):
     c.add(gates.X(3))
     c.add(gates.M(0, 1, 2, 3, basis=[gates.X, gates.Z, gates.X, gates.Z]))
     result = c(nshots=100)
-    print(result.frequencies())
     assert result.frequencies() == {"0011": 100}
-    print(c.draw())
     assert (
         c.draw()
         == """q0: ─H─H───M─

tests/test_quantum_info_random.py

@@ -315,7 +315,6 @@ def test_random_density_matrix(backend, dims, pure, metric, basis, normalize):
             backend.assert_allclose(norm < PRECISION_TOL, True)
         else:
             normalization = 1.0 if normalize is False else 1.0 / np.sqrt(dims)
-            print(state)
             backend.assert_allclose(state[0], normalization)
             assert all(np.abs(exp_value) <= normalization for exp_value in state[1:])
 

tests/test_transpiler_blocks.py

@@ -154,7 +154,6 @@ def test_initial_block_decomposition_measurements():
     circ.add(gates.H(3))
     circ.add(gates.H(4))
     blocks = _initial_block_decomposition(circ)
-    print(blocks[0].gates)
     assert_gates_equality(
         blocks[0].gates,
         [gates.M(0), gates.M(1), gates.H(1), gates.H(0), gates.CZ(0, 1)],
@@ -166,7 +165,6 @@ def test_initial_block_decomposition_measurements():
 def test_initial_block_decomposition_error():
     circ = Circuit(3)
     circ.add(gates.TOFFOLI(0, 1, 2))
-    print(len(circ.queue[0].qubits))
     with pytest.raises(BlockingError):
         blocks = _initial_block_decomposition(circ)
 
@@ -249,7 +247,6 @@ def test_block_decomposition_measurements():
     circ.add(gates.CZ(2, 3))  # 4 block
     circ.add(gates.M(0, 1))  # 3 block
     blocks = block_decomposition(circ)
-    print(blocks[0].gates)
     assert_gates_equality(
         blocks[0].gates,
         [gates.H(1), gates.H(0), gates.CZ(0, 1), gates.H(0), gates.M(0), gates.M(1)],