Skip to content

Commit 338ca81

Browse files
committed
update validation msg
1 parent 49d9fac commit 338ca81

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

lib/superset/services/duplicate_dashboard.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,7 @@ def validate_params
197197
# schema validations
198198
raise ValidationError, "Schema #{target_schema} does not exist in target database: #{target_database_id}" unless target_database_available_schemas.include?(target_schema)
199199
raise ValidationError, "The source dashboard datasets are required to point to one schema only. Actual schema list is #{source_dashboard_schemas.join(',')}" if source_dashboard_has_more_than_one_schema?
200-
raise ValidationError, "One or more source dashboard filters point to a different dataset than the dashboard charts. Identified Unpermittied Filter Dataset Ids are #{unpermitted_filter_dataset_ids.to_s}" if unpermitted_filter_dataset_ids.any?
200+
raise ValidationError, "One or more source dashboard filters point to a different schema than the dashboard charts. Identified Unpermittied Filter Dataset Ids are #{unpermitted_filter_dataset_ids.to_s}" if unpermitted_filter_dataset_ids.any?
201201

202202
# new dataset validations
203203
raise ValidationError, "DATASET NAME CONFLICT: The Target Schema #{target_schema} already has existing datasets named: #{target_schema_matching_dataset_names.join(',')}" unless target_schema_matching_dataset_names.empty?

spec/superset/services/duplicate_dashboard_spec.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -299,7 +299,7 @@
299299
expect(Superset::Dataset::Get).to receive(:new).with(invalid_filter_dataset_id).and_return(double(schema: invalid_filter_dataset_schema))
300300

301301
expect { subject.perform }.to raise_error(Superset::Request::ValidationError,
302-
"One or more source dashboard filters point to a different dataset than the dashboard charts. " \
302+
"One or more source dashboard filters point to a different schema than the dashboard charts. " \
303303
"Identified Unpermittied Filter Dataset Ids are " \
304304
"[{:filter_dataset_id=>#{invalid_filter_dataset_id}, :filter_schema=>\"#{invalid_filter_dataset_schema}\"}]")
305305
end
@@ -319,7 +319,7 @@
319319
expect(Superset::Dataset::Get).to receive(:new).with(invalid_filter_dataset_id_2).and_return(double(schema: invalid_filter_dataset_schema_2))
320320

321321
expect { subject.perform }.to raise_error(Superset::Request::ValidationError,
322-
"One or more source dashboard filters point to a different dataset than the dashboard charts. " \
322+
"One or more source dashboard filters point to a different schema than the dashboard charts. " \
323323
"Identified Unpermittied Filter Dataset Ids are " \
324324
"[{:filter_dataset_id=>#{invalid_filter_dataset_id}, :filter_schema=>\"#{invalid_filter_dataset_schema}\"}, " \
325325
"{:filter_dataset_id=>#{invalid_filter_dataset_id_2}, :filter_schema=>\"#{invalid_filter_dataset_schema_2}\"}]")

0 commit comments

Comments
 (0)