From 34b8ba3164d77a3c793b15928eaeb7f6a398c7e1 Mon Sep 17 00:00:00 2001 From: Dave Machado Date: Mon, 10 Jul 2017 00:42:14 -0400 Subject: [PATCH] Add HTTPS Support check for Yes or No --- build/validate.rb | 20 +++++++++++++------- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git a/build/validate.rb b/build/validate.rb index 3d13b803..9191db6a 100755 --- a/build/validate.rb +++ b/build/validate.rb @@ -1,11 +1,11 @@ #!/usr/bin/env ruby auth_keys = ['apiKey', 'OAuth', 'X-Mashape-Key', 'No'] +https_keys = ['Yes', 'No'] args = ARGV filename = args[0] fail_flag = false -fail_count = 0 File.foreach(filename).with_index do |line, line_num| line_num += 1 # puts "#{line_num}: #{line}" @@ -14,14 +14,20 @@ File.foreach(filename).with_index do |line, line_num| if line.eql? "|---|---|---|---|---|\n" next end - values = line.split("|") - # Check Auth Values to conform to valid options only + + # Check Auth values to conform to valid options only auth_val = values[3].lstrip.chop.tr('``', '') - if !auth_keys.include? auth_val - puts "(#{line_num}) Invalid Auth (#{auth_val}): #{line}" - fail_flag = false - fail_count += 1 + if !auth_keys.include?(auth_val) + puts "(#{line_num}) Invalid Auth (not a valid option): #{auth_val}" + fail_flag = true + end + + # Check HTTPS Support values to be either "Yes" or "No" + https_val = values[4].lstrip.chop + if !https_keys.include?(https_val) + puts "(#{line_num}) Invalid HTTPS: (must use \"Yes\" or \"No\"): #{https_val}" + fail_flag = true end end end