2622 implement unit for global templates

[kdp-cloud]

Closes #2622

Implements the unit field when uploading a global template JSON file.

[Copilot]

Pull request overview

Adds support for specifying a unit for template attributes when importing global templates from JSON, integrating unit lookup into the ISA template extraction pipeline and verifying behavior via controller/job tests.

Changes:

• Extend ISA template attribute JSON schema(s) to allow a unit field.
• Populate TemplateAttribute.unit_id from the imported JSON during template extraction.
• Add fixtures and tests covering unit import, missing unit, and invalid unit handling (job + controller flows).

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
lib/seek/isa_templates/template_extractor.rb	Looks up Unit by symbol from JSON and assigns unit_id on extracted TemplateAttributes.
lib/seek/isa_templates/template_attributes_schema.json	Allows unit in production schema for template attributes.
lib/seek/isa_templates/template_attributes_schema_test.json	Allows unit in test schema for template attributes.
test/unit/jobs/populate_templates_job_test.rb	Adds unit-related job tests (valid unit, missing unit, invalid unit).
test/functional/templates_controller_test.rb	Adds controller tests for create/update with unit and JSON upload population including unit.
test/fixtures/files/upload_json_sample_type_template/test_unit_template.json	New fixture template JSON including "unit": "g".
test/fixtures/files/upload_json_sample_type_template/invalid_unit_template.json	New fixture template JSON including an invalid unit symbol.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.