863e578f-15a5-4c93-8357-08f855de0bbc

Committed 24 Nov 2025 02:23PM UTC coverage: 87.366% (-0.7%) from 88.064%

Build # 863e578f-15a5-4c93-8357-08f855de0bbc

Build Type

Pull #2235

circleci

Committed by

carolyncole

Commit Message

Do not redirect from home page when login is disabled
fixes #2188

Pull Request Pull Request #2235: Do not redirect from home page when login is disabled

Run Details

5 of 5 new or added lines in 2 files covered. (100.0%)

834 existing lines in 48 files now uncovered.

2773 of 3174 relevant lines covered (87.37%)

499.78 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	4✔
2	def validate(project)	4✔
3	@schema = TigerdataSchema.new(namespace: "tigerdata", type: "tigerdata:project")	1,192✔
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	1,192✔
7
8	# Required fields, always validate
9	validate_role(project:, netid: project.metadata_model.data_manager, role: "Data Manager")	1,192✔
10	validate_role(project:, netid: project.metadata_model.data_sponsor, role: "Data Sponsor")	1,192✔
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")}	1,409✔
14	project.metadata_model.data_user_read_write&.each { \|read_write\| validate_role(project:, netid: read_write, role: "Data User Read Write")}	1,255✔
15
16	# validate all required fields
17	required_metadata_field_errors = []	1,192✔
18	required_metadata = required_attributes(project:)	1,192✔
19	if required_metadata.values.include?(nil)	1,192✔
UNCOV 20	required_keys.each do \|attr\|	6✔
UNCOV 21	value = required_metadata[attr]	42✔
UNCOV 22	required_metadata_field_errors << "Missing metadata value for #{attr}" if value.nil? && project.metadata_json.include?(attr)	42✔
23	end
24	end
25	if required_metadata_field_errors.count > 0	1,192✔
UNCOV 26	project.errors.add :base, "Invalid Project Metadata it does not match the schema #{TigerdataSchema::SCHEMA_VERSION}\n #{required_metadata_field_errors.join(", ")}"	6✔
27	end
28	end
29
30	private	4✔
31
32	def validate_role(project:, netid:, role:)	4✔
33	if netid.blank?	2,664✔
UNCOV 34	project.errors.add :base, "Missing netid for role #{role}"	3✔
35	elsif User.where(uid: netid).empty?	2,661✔
36	project.errors.add :base, "Invalid netid: #{netid} for role #{role}"	34✔
37	end
38	end
39
40	def required_field_labels	4✔
41	@schema.required_project_schema_fields.pluck(:name)	28,790✔
42	end
43
44	def required_keys	4✔
45	tableized = required_field_labels.map { \|v\| v.underscore }	230,320✔
46	tableized	28,790✔
47	end
48
49	def required_attributes(project:)	4✔
50	project.metadata_json.select { \|k, _v\| required_keys.include?(k) }	29,976✔
51	end
52
53	end

pulibrary / tigerdata-app / 863e578f-15a5-4c93-8357-08f855de0bbc

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