Fix Shor example

[stavros11]

Fix Shor example, which is outdated since #1039, as reported in #1432.

The measurement in the end is needed because it is no longer possible to execute circuits that contain only collapse measurements using state vectors.

Specifying nshots=1 is needed because a default nshots=1000 was introduced in #1039 compared to the None being used before. This was also causing the performance issue reported in #1432, because during the repeated execution it was executing the circuit 1000 times instead of one that we need.

Checklist:

• Reviewers confirm new code works as expected.
• Tests are passing.
• Coverage does not decrease.
• Documentation is updated.

[igres26]

Thanks! This looks like the solution.

[MatteoRobbiati]

Thanks.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.10%. Comparing base (e93ec84) to head (c645237).
Report is 2 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1439   +/-   ##
=======================================
  Coverage   97.10%   97.10%           
=======================================
  Files          81       81           
  Lines       11653    11653           
=======================================
  Hits        11316    11316           
  Misses        337      337           

Flag	Coverage Δ
unittests	97.10% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.