feat: allow specifying components when using edit-validation-sets

[andrewphelpsj]

• Have you followed the guidelines for contributing?
• Have you signed the CLA?
• Have you successfully run tox run -m lint?
• Have you successfully run tox run -e test-py310? (supported versions: py39, py310, py311, py312)

---

Hello! This should enable using edit-validation-sets with components. Let me know if I'm missing any testing. I don't see any tests that specifically test the allowed format of validation sets, so I don't know if that is something that we need/want to be added.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.69%. Comparing base (654871d) to head (7dba20a).
Report is 612 commits behind head on main.

❗ There is a different number of reports uploaded between BASE (654871d) and HEAD (7dba20a). Click for more details.

HEAD has 1 upload less than BASE

Flag	BASE (654871d)	HEAD (7dba20a)
	2	1

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5059      +/-   ##
==========================================
- Coverage   94.88%   89.69%   -5.19%     
==========================================
  Files         658      341     -317     
  Lines       55189    22607   -32582     
==========================================
- Hits        52364    20278   -32086     
+ Misses       2825     2329     -496     

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

[mr-cal]

Thanks for the PR! I have one comment and one request for updating some tests.

Will you update the dictionaries in tests/legacy/unit/store/v2/test_validation_sets.py to include components?

Do you happen to know if the store side is ready for this? Testing your branch gives me a store error Issues encountered with validation set: Additional properties are not allowed ('components' was unexpected) at /snaps/0

[andrewphelpsj]

@mr-cal, for the store side, that is expected to be released this week. @suligap might have more specific details. We can hold off on merging this until that change reaches the store.

I’ll update those tests now.

[mr-cal]

@mr-cal, for the store side, that is expected to be released this week. @suligap might have more specific details. We can hold off on merging this until that change reaches the store.

I’ll update those tests now.

Sounds good.

Once the PR lands, it will be available in the edge channel. It will be part of Snapcraft 8.5, which I'm planning to cut and release to stable in the second half of October. Is that timing OK for you?

[andrewphelpsj]

I think that is around the right time. I'll talk with my team to confirm timelines.

[mr-cal]

Thanks! Once the store changes are completed and we verify it works, we can land this PR.

[cmatsuoka]

Looks good, just added a question about component revision validation.