Fix bug in QPY symengine payload version handling (backport #13259)

[mergify]

Summary

This commit fixes a bug that slipped into the sole fix that made up the now yanked 1.2.3 release, #13251. The fix logic had a typo/mistake when evaluating the major version in the symengine payload that meant if you were trying to load a QPY payload generated with a different symengine release it would always error. This only got through CI because there was no test coverage of the edge case we were trying to fix. This commit addresses both issues. It first fixes the typo in the QPY parsing (which is a 2 character change) and then updates the QPY compat tests to explicitly test using multiple symengine versions with QPY to make sure we're exercising this code path moving forward (and validating this fix).

Details and comments

---

This is an automatic backport of pull request #13259 done by Mergify.

[mergify]

Cherry-pick of e5ec413 has failed:

On branch mergify/bp/stable/1.2/pr-13259
Your branch is up to date with 'origin/stable/1.2'.

You are currently cherry-picking commit e5ec41358.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   qiskit/qpy/common.py
	modified:   qiskit/qpy/interface.py
	new file:   releasenotes/notes/fix-qpy-parsing-123-75357c3709e35963.yaml
	modified:   releasenotes/notes/fix-qpy-symengine-compat-858970a9a1d6bc14.yaml
	modified:   test/qpy_compat/get_versions.py
	modified:   test/qpy_compat/run_tests.sh

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   qiskit/qpy/__init__.py

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

[qiskit-bot]

Thank you for opening a new pull request.

Before your PR can be merged it will first need to pass continuous integration tests and be reviewed. Sometimes the review process can be slow, so please be patient.

While you're waiting, please feel free to review other open PRs. While only a subset of people are authorized to approve pull requests for merging, everyone is encouraged to review open pull requests. Doing reviews helps reduce the burden on the core team and helps make the project's code better for everyone.

One or more of the following people are relevant to this code:

• @Qiskit/terra-core
• @mtreinish
• @nkanazawa1989

[coveralls]

Pull Request Test Coverage Report for Build 11156654634

Details

• 0 of 1 (0.0%) changed or added relevant line in 1 file are covered.
• 15 unchanged lines in 3 files lost coverage.
• Overall coverage decreased (-0.007%) to 89.747%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
qiskit/qpy/common.py	0	1	0.0%

Files with Coverage Reduction	New Missed Lines	%
crates/qasm2/src/expr.rs	1	94.02%
crates/qasm2/src/lex.rs	2	92.48%
crates/qasm2/src/parse.rs	12	97.15%

Totals
Change from base Build 11148098131:	-0.007%
Covered Lines:	66885
Relevant Lines:	74526

---

💛 - Coveralls

[ElePT]

I'd say the conflict fix looks good. Any reason not to add this PR to the queue @mtreinish?