diff options
| author | a14m <[email protected]> | 2020-12-06 16:16:35 +0100 |
|---|---|---|
| committer | a14m <[email protected]> | 2020-12-06 16:16:35 +0100 |
| commit | c7a80df29dac0cb5a7b5bd78a750c15fd6146908 (patch) | |
| tree | f17b19ee2917bddd4d2a1104632c644d98c70ab6 /CLI/ruby | |
| parent | 90eb66f7e34cf6f5a0468e21d3542b20cc668e78 (diff) | |
Add the error handling for missing/invalid file/URL input
Diffstat (limited to 'CLI/ruby')
| -rw-r--r-- | CLI/ruby/lib/game_of_life/generators/input.rb | 5 | ||||
| -rw-r--r-- | CLI/ruby/spec/game_of_life/generators/input_spec.rb | 24 |
2 files changed, 29 insertions, 0 deletions
diff --git a/CLI/ruby/lib/game_of_life/generators/input.rb b/CLI/ruby/lib/game_of_life/generators/input.rb index c0d96c1..3082e70 100644 --- a/CLI/ruby/lib/game_of_life/generators/input.rb +++ b/CLI/ruby/lib/game_of_life/generators/input.rb @@ -22,6 +22,11 @@ module GameOfLife # where any alpha numeric character is considered a living cell (true) raw.map! { |row| row.chars.map { |char| char.match?(/[[:alnum:]]/) } } populate(parsed_input: raw, width: options["width"].to_i, height: options["height"].to_i) + rescue OpenURI::HTTPError, SocketError + raise GameOfLife::Error, "URL isn't avaialable, please check it again and make sure the URL is correct" + rescue Errno::ENOENT + raise GameOfLife::Error, + "File isn't avaialable, please check it again and make sure the path to the input file is correct" end # Populate the {GameOfLife::Universe} with reference to the parsed input 2D array (True/False) diff --git a/CLI/ruby/spec/game_of_life/generators/input_spec.rb b/CLI/ruby/spec/game_of_life/generators/input_spec.rb index e05dbfb..6986787 100644 --- a/CLI/ruby/spec/game_of_life/generators/input_spec.rb +++ b/CLI/ruby/spec/game_of_life/generators/input_spec.rb @@ -51,5 +51,29 @@ RSpec.describe GameOfLife::Generators::Input do expect(described_class.new(options).to_s).to eq universe end + + it "rescues Errno::ENOENT and raises GameOfLife::Error" do + options = { "input" => "not-existing-file.txt", "width" => 25, "height" => 2 } + expect { described_class.new(options) }.to raise_error( + GameOfLife::Error, + "File isn't avaialable, please check it again and make sure the path to the input file is correct", + ) + end + + it "rescues SocketError and raises GameOfLife::Error" do + options = { "input" => "https://not-existing-url.com", "width" => 25, "height" => 2 } + expect { described_class.new(options) }.to raise_error( + GameOfLife::Error, + "URL isn't avaialable, please check it again and make sure the URL is correct", + ) + end + + it "rescues OpenURI::HTTPError and raises GameOfLife::Error" do + options = { "input" => "https://example.com/non-existing", "width" => 25, "height" => 2 } + expect { described_class.new(options) }.to raise_error( + GameOfLife::Error, + "URL isn't avaialable, please check it again and make sure the URL is correct", + ) + end end end |
