fix: add measure qubit targets in braket_program_context

Issue #, if available:

Description of changes: Currently, when a measure instruction is added in braket_program_context.py it is not added to the circuit's measure_targets which results in measure instructions getting added twice.

Changes made

addMeasure will now add the qubit targets to the Circuit's measure_targets
updated the round trip test to properly check that the OpenQASM ir gets translated correctly

To reproduce the issue this solves:

qasm = """
OPENQASM 3.0;

def ghz(int[32] n) {
    h q[0];
    for int i in [0:n - 1] {
        cnot q[i], q[i + 1];
    }
}

int[32] n = 5;
bit[n + 1] c;
qubit[n + 1] q;

ghz(n);

c = measure q;
"""
circuit = Circuit.from_ir(qasm)
print(sv1.run(circuit, shots=1000).result().measurement_counts)

returns a ValidationException: An error occurred (ValidationException) when calling the CreateQuantumTask operation: [line 16] qubit q[0] is already measured or captured. Cannot perform a new operation on it since the measurements are added twice.

Testing done: tox and tox -e integ-tests

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

General

[x] I have read the CONTRIBUTING doc
[x] I used the commit message format described in CONTRIBUTING
[ ] I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

[x] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[ ] I have checked that my tests are not configured for a specific region or account (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

amazon-braket / amazon-braket-sdk-python