There is no try (#30)

* Replace use of .try with .dig

Author: adamrdavid

CHANGELOG.md

@@ -7,6 +7,7 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/) and this p
 ### Added
 
 ### Changed
+- Fixed bug for mappings with multiple keys and a default (resolves: [#26](https://github.com/bugcrowd/vrt-ruby/issues/26))
 
 ### Removed
 - Removed `Gemfile.lock` from source control

lib/vrt/mapping.rb

@@ -27,7 +27,7 @@ def get(id_list, version)
             id_list: id_list,
             mapping: mapping,
             key: key
-          ) || default&.try(:[], key)
+          ) || default&.dig(key)
         end
       else
         get_key(id_list: id_list, mapping: mapping, key: @scheme) || default

spec/sample_vrt/999.999/mappings/test_mapping.json

@@ -0,0 +1,15 @@
+{
+  "metadata": {
+    "default": {
+      "one_thing": "the thing one is"
+    },
+    "keys": ["one_thing", "another_thing"]
+  },
+  "content": [
+    {
+      "id": "server_security_misconfiguration",
+      "one_thing": "the thing",
+      "another_thing": "another one"
+    }
+  ]
+}

spec/vrt/mappings_spec.rb

@@ -126,6 +126,22 @@
             )
           end
         end
+
+        context 'with a default' do
+          subject { mapping.get(id_list, version) }
+
+          let(:mapping) { described_class.new(:test_mapping) }
+          let(:id_list) { %i[server_security_misconfiguration] }
+          let(:version) { '999.999' }
+
+          it { is_expected.to include(:one_thing, :another_thing) }
+
+          context 'when falling back to default' do
+            let(:id_list) { %i[broken_authentication_and_session_management] }
+
+            it { is_expected.to include(:one_thing) }
+          end
+        end
       end
 
       context 'with arrays as the mapping values' do