quby-compiler 0.5.7 → 0.5.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9585291a3d8622c9a7d8af491999e99258f50f85ba7af81b2189418fb7ba581a
-  data.tar.gz: 41c7638af19d0837ffce3a0c6d57f8d06cdf8401c2c9864f8a167c2cf0b02087
+  metadata.gz: 392bac29f2cc7b4b00ce1344f010b39941148e96e5e529d6487f8e0d49a5a5fe
+  data.tar.gz: 7c0538c6243f30481c3f282bfc2892874494454b64f5eb0013b0a0a4aa56e0c8
 SHA512:
-  metadata.gz: 491d4aad787b486403d23dab908931da75d81afc02d13dcabef924e6ac108aab65e8822a1ae450c32cd8cbce7d6789a0ce18618fc0f0a3172312cc06ba288674
-  data.tar.gz: 8f27a6cabf5bf5451348255f81bab64483d690797786860739f69d43637e87698df3cce4c1726f4a4c01c5473797b0b08832a05a6fd3408f9460001a2c8541d5
+  metadata.gz: 8360a9fd3ec70b80766ece8d51bbbc301067ae7eb8c441515c06dc8c8003205527290c10b1249948deb5b4fab798a1e61ba56e03594f1e5188f5a43d3fbffa9f
+  data.tar.gz: 16b077a6db915f93dd30f6c6bb8f411c5c21e8cd2adc3ba279abbc11596a34598bd3e80dbea4527b431e7f1719f5eecb78c39f1c1252011c7d71a2332b576331

data/CHANGELOG.md

@@ -1,3 +1,12 @@
+# 0.5.9
+
+* added `require 'ostruct'` since running quby-compile suddenly fails on this.
+
+# 0.5.8
+
+* DefinitionValidator: Add uniqueness check over all question, flags, textvars and score keys. 
+* Moved flag description validation to definition_validator
+
 # 0.5.7
 
 * quby2.json

data/lib/quby/compiler/entities/flag.rb

@@ -19,21 +19,12 @@ module Quby
                                                   # does not have an effect outside of the interface
           super(key, description_true, description_false, description, internal, trigger_on, shows_questions,
                 hides_questions, Array.wrap(depends_on).map(&:to_s), default_in_interface)
-          ensure_valid_descriptions
         end
         # rubocop:enable ParameterLists
 
         def if_triggered_by(answer_flags)
           yield if answer_flags[key] == trigger_on
         end
-
-        private
-
-        def ensure_valid_descriptions
-          unless (description_false.present? && description_true.present?) || description.present?
-            raise "Flag '#{key}' Requires at least either both description_true and description_false or a description"
-          end
-        end
       end
     end
   end

data/lib/quby/compiler/entities/questionnaire.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require 'ostruct'
 require 'active_model'
 require 'quby/settings'
 require 'quby/compiler/entities/flag'

data/lib/quby/compiler/services/definition_validator.rb

@@ -21,6 +21,7 @@ module Quby
           validate_scores(questionnaire)
           validate_table_edgecases(questionnaire)
           validate_flags(questionnaire)
+          validate_key_uniqueness(questionnaire)
           validate_respondent_types(questionnaire)
           validate_outcome_tables(questionnaire)
           validate_markdown_fields(questionnaire) if questionnaire.validate_html
@@ -127,12 +128,19 @@ module Quby
 
         def validate_flags(questionnaire)
           questionnaire.flags.each_value do |flag|
+            ensure_valid_descriptions(questionnaire, flag)
             validate_flag_shows(questionnaire, flag)
             validate_flag_hides(questionnaire, flag)
             validate_flag_depends_on(questionnaire, flag)
           end
         end
 
+        def ensure_valid_descriptions(questionnaire, flag)
+          unless (flag.description_false.present? && flag.description_true.present?) || flag.description.present?
+            fail ArgumentError, "Flag '#{flag.key}' Requires at least either both description_true and description_false or a description"
+          end
+        end
+
         def validate_flag_shows(questionnaire, flag)
           unknown_questions = flag.shows_questions.select { |key| !questionnaire.key_in_use?(key) }
           return if unknown_questions.blank?
@@ -153,6 +161,18 @@ module Quby
           fail ArgumentError, "Flag #{flag.key} depends_on nonexistent flag '#{missing.to_sentence}'"
         end
 
+        def validate_key_uniqueness(questionnaire)
+          keys = questionnaire.fields.answer_keys.to_a \
+            + questionnaire.score_schemas.keys.map(&:to_sym) \
+            + questionnaire.flags.keys.map { delete_prefix(_1, questionnaire).to_sym } \
+            + questionnaire.textvars.keys.map { delete_prefix(_1, questionnaire).to_sym }
+
+          return if keys.size == keys.uniq.size
+
+          duplicates = keys.tally.filter_map { |k,v| k if v > 1 }
+          fail ArgumentError, "Duplicate keys: #{duplicates.to_sentence}"
+        end
+
         def validate_respondent_types(questionnaire)
           valid_respondent_types = Entities::Questionnaire::RESPONDENT_TYPES
 
@@ -339,6 +359,10 @@ scores_schema tables to the resulting seed."
 
           fail "#{key || html} contains invalid html: #{fragment.errors.map(&:to_s).join(', ')}."
         end
+
+        def delete_prefix(key, questionnaire)
+          key.delete_prefix("#{questionnaire.key}_")
+        end
       end
     end
   end

data/lib/quby/compiler/services/transform_quby1_values_into_quby2_values.rb

@@ -1,3 +1,5 @@
+require 'ostruct'
+
 # Copied from RoQua for default_answer_values.
 # Temporary: should change the values when quby1 is off the table
 class Quby::Compiler::Services::TransformQuby1ValuesIntoQuby2Values

data/lib/quby/compiler/version.rb

@@ -1,5 +1,5 @@
 module Quby
   module Compiler
-    VERSION = "0.5.7"
+    VERSION = "0.5.9"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: quby-compiler
 version: !ruby/object:Gem::Version
-  version: 0.5.7
+  version: 0.5.9
 platform: ruby
 authors:
 - Marten Veldthuis
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-11-03 00:00:00.000000000 Z
+date: 2024-04-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
@@ -245,7 +245,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.19
+rubygems_version: 3.5.7
 signing_key:
 specification_version: 4
 summary: Quby::Compiler compiles a DSL for questionnaires to JSON