2e85105d-f9c8-41e9-8b05-fe438b698516

Committed 26 Nov 2025 12:16PM UTC coverage: 71.597% (-8.0%) from 79.623%

Build # 2e85105d-f9c8-41e9-8b05-fe438b698516

Build Type

push

circleci

Committed by

web-flow

Commit Message

Display the data sponsor and data manager in the dashboard listing (#2247)

Closes #2236 

<img width="1036" height="675" alt="Screenshot 2025-11-25 at 3 07 41 PM"
src="https://github.com/user-attachments/assets/3b6a2189-7f71-4b17-97e5-e66c8fed3cb3"
/>

Run Details

1 of 3 new or added lines in 1 file covered. (33.33%)

346 existing lines in 26 files now uncovered.

2367 of 3306 relevant lines covered (71.6%)

147.24 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

100.0

/app/models/project_validator.rb

class ProjectValidator < ActiveModel::Validator
    def validate(project)
        @schema = TigerdataSchema.new(namespace: "tigerdata", type: "tigerdata:project")

        # we need this because this method references the metadata_json which is not updated until the project is saved
        project.metadata = project.metadata_model

        # Required fields, always validate
        validate_role(project:, netid: project.metadata_model.data_manager, role: "Data Manager")
        validate_role(project:, netid: project.metadata_model.data_sponsor, role: "Data Sponsor")

        # Validate if present
        project.metadata_model.data_user_read_only&.each { |read_only| validate_role(project:, netid: read_only, role: "Data User Read Only")}
        project.metadata_model.data_user_read_write&.each { |read_write| validate_role(project:, netid: read_write, role: "Data User Read Write")}

        # validate all required fields
        required_metadata_field_errors = []
        required_metadata = required_attributes(project:)
        if required_metadata.values.include?(nil)
            required_keys.each do |attr|
                value = required_metadata[attr]
                required_metadata_field_errors << "Missing metadata value for #{attr}" if value.nil? && project.metadata_json.include?(attr)
            end
        end
        if required_metadata_field_errors.count > 0
            project.errors.add :base, "Invalid Project Metadata it does not match the schema #{TigerdataSchema::SCHEMA_VERSION}\n #{required_metadata_field_errors.join(", ")}"
        end
    end

    private

    def validate_role(project:, netid:, role:)
        if netid.blank?
            project.errors.add :base, "Missing netid for role #{role}"
        elsif User.where(uid: netid).empty?
            project.errors.add :base, "Invalid netid: #{netid} for role #{role}"
        end
    end

    def required_field_labels
        @schema.required_project_schema_fields.pluck(:name)
    end

    def required_keys
        tableized = required_field_labels.map { |v| v.underscore }
        tableized
    end

    def required_attributes(project:)
        project.metadata_json.select { |k, _v| required_keys.include?(k) }
    end

end

1	class ProjectValidator < ActiveModel::Validator	2✔
2	def validate(project)	2✔
3	@schema = TigerdataSchema.new(namespace: "tigerdata", type: "tigerdata:project")	403✔
4
5	# we need this because this method references the metadata_json which is not updated until the project is saved
6	project.metadata = project.metadata_model	403✔
7
8	# Required fields, always validate
9	validate_role(project:, netid: project.metadata_model.data_manager, role: "Data Manager")	403✔
10	validate_role(project:, netid: project.metadata_model.data_sponsor, role: "Data Sponsor")	403✔
11
12	# Validate if present
13	project.metadata_model.data_user_read_only&.each { \|read_only\| validate_role(project:, netid: read_only, role: "Data User Read Only")}	477✔
14	project.metadata_model.data_user_read_write&.each { \|read_write\| validate_role(project:, netid: read_write, role: "Data User Read Write")}	423✔
15
16	# validate all required fields
17	required_metadata_field_errors = []	403✔
18	required_metadata = required_attributes(project:)	403✔
19	if required_metadata.values.include?(nil)	403✔
UNCOV 20	required_keys.each do \|attr\|	1✔
UNCOV 21	value = required_metadata[attr]	7✔
UNCOV 22	required_metadata_field_errors << "Missing metadata value for #{attr}" if value.nil? && project.metadata_json.include?(attr)	7✔
23	end
24	end
25	if required_metadata_field_errors.count > 0	403✔
UNCOV 26	project.errors.add :base, "Invalid Project Metadata it does not match the schema #{TigerdataSchema::SCHEMA_VERSION}\n #{required_metadata_field_errors.join(", ")}"	1✔
27	end
28	end
29
30	private	2✔
31
32	def validate_role(project:, netid:, role:)	2✔
33	if netid.blank?	900✔
UNCOV 34	project.errors.add :base, "Missing netid for role #{role}"	2✔
35	elsif User.where(uid: netid).empty?	898✔
36	project.errors.add :base, "Invalid netid: #{netid} for role #{role}"	38✔
37	end
38	end
39
40	def required_field_labels	2✔
41	@schema.required_project_schema_fields.pluck(:name)	9,809✔
42	end
43
44	def required_keys	2✔
45	tableized = required_field_labels.map { \|v\| v.underscore }	78,472✔
46	tableized	9,809✔
47	end
48
49	def required_attributes(project:)	2✔
50	project.metadata_json.select { \|k, _v\| required_keys.include?(k) }	10,211✔
51	end
52
53	end

pulibrary / tigerdata-app / 2e85105d-f9c8-41e9-8b05-fe438b698516

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous