diff --git a/cirq-core/cirq/ops/pauli_measurement_gate_test.py b/cirq-core/cirq/ops/pauli_measurement_gate_test.py index a304f18dbee..9960d92ac68 100644 --- a/cirq-core/cirq/ops/pauli_measurement_gate_test.py +++ b/cirq-core/cirq/ops/pauli_measurement_gate_test.py @@ -138,12 +138,12 @@ def test_op_repr(): ps = cirq.X(a) * cirq.Y(b) * cirq.Z(c) assert ( repr(cirq.measure_single_paulistring(ps)) - == 'cirq.measure_single_paulistring((cirq.X(cirq.LineQubit(0))' + == 'cirq.measure_single_paulistring(((1+0j)*cirq.X(cirq.LineQubit(0))' '*cirq.Y(cirq.LineQubit(1))*cirq.Z(cirq.LineQubit(2))))' ) assert ( repr(cirq.measure_single_paulistring(ps, key='out')) - == "cirq.measure_single_paulistring((cirq.X(cirq.LineQubit(0))" + == "cirq.measure_single_paulistring(((1+0j)*cirq.X(cirq.LineQubit(0))" "*cirq.Y(cirq.LineQubit(1))*cirq.Z(cirq.LineQubit(2))), " "key=cirq.MeasurementKey(name='out'))" ) diff --git a/cirq-core/cirq/ops/pauli_string.py b/cirq-core/cirq/ops/pauli_string.py index a7afdb664aa..dfb44ce052a 100644 --- a/cirq-core/cirq/ops/pauli_string.py +++ b/cirq-core/cirq/ops/pauli_string.py @@ -397,7 +397,7 @@ def __repr__(self) -> str: factors = [] if self._coefficient == -1: prefix = '-' - elif self._coefficient != 1: + else: factors.append(repr(self._coefficient)) if not ordered_qubits: diff --git a/cirq-core/cirq/ops/pauli_string_test.py b/cirq-core/cirq/ops/pauli_string_test.py index 936f545a680..81a0af69889 100644 --- a/cirq-core/cirq/ops/pauli_string_test.py +++ b/cirq-core/cirq/ops/pauli_string_test.py @@ -310,6 +310,12 @@ def test_repr(): cirq.testing.assert_equivalent_repr(cirq.PauliString()) +def test_repr_coefficient_of_one(): + pauli_string = cirq.Z(cirq.LineQubit(0)) * 1 + assert type(pauli_string) == type(eval(repr(pauli_string))) + cirq.testing.assert_equivalent_repr(pauli_string) + + def test_str(): q0, q1, q2 = _make_qubits(3) pauli_string = cirq.PauliString({q2: cirq.X, q1: cirq.Y, q0: cirq.Z})