feat: refactor tests

[alekseifedotov]

A bunch of small fixes:

• add function test_orb_id() returing a pre-defined orb-id to be used only in tests.
• use OrbId instead of &str for orb-id
• fix a race in a zenorb test

[github-actions]

• Compile blocker: attest/src/remote_api.rs:771 now requires OrbId: Serialize, but attest/Cargo.toml does not enable orb-info’s serde feature. The same crate’s tests import test_orb_id at line 1140 without enabling the testing feature. Please enable both required features, with testing preferably as a dev-dependency feature.

• Compile blocker: speed-test/src/lib.rs:87 changed the serialized field from &str to &OrbId, but speed-test/Cargo.toml does not enable orb-info/serde. Consequently, deriving Serialize for PackageRequest fails.

[github-actions]

No concrete correctness, security, race-condition, or regression issues found in the PR changes.

[github-actions]

No concrete correctness, security, race-condition, or merge-blocking issues found in the PR changes.

[github-actions]

No concrete correctness, security, race-condition, or regression issues found in the PR changes.

[github-actions]

No concrete correctness bugs or merge-blocking issues found in the PR diff. The OrbId conversions preserve serialization and formatting, and the Zenorb test now waits for both subscriber callbacks before querying.

Tests were not run in the read-only environment.

[sfikastheo]

I have a suspicion this was on purpose?
@vmenge should review it.
Everything else looks very good to me.

[alekseifedotov]

Yes, that was on purpose. I think there is a race in the test, and this change tries to fix it.