Fix crash when defining multiple versions (#151)

* Fix crash when defining multiple validation versions

* Add versions test

* Run in correct scope

* Fix missing method in test

Author: thexa4

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 2.3.3
+
+ - Fix crash when defining multiple versions of a validator
+
 ## 2.3.2
 
 - Fix that causes bailout in nested rule to stop validation altogether

lib/media_types/validations.rb

@@ -97,7 +97,7 @@ def version(version, &block)
     def versions(versions, &block)
       versions.each do |v|
         Validations.new(media_type.version(v), registry) do
-          block(v)
+          instance_exec(v, &block)
         end
       end
     end

lib/media_types/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module MediaTypes
-  VERSION = '2.3.2'
+  VERSION = '2.3.3'
 end

test/media_types/dsl/versions_test.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+require_relative '../../test_helper'
+
+module MediaTypes
+  module Dsl
+    class VersionsTestTest < Minitest::Test
+
+      class AttributeType
+        include MediaTypes::Dsl
+
+        def self.organisation
+          'acme'
+        end
+
+        use_name 'AttributeType'
+
+        validations do
+          versions [1,2] do |v|
+            attribute :foo, Numeric
+            attribute :bar, Numeric if v == 2
+          end
+        end
+      end
+
+      def test_versions
+        assert AttributeType.version(1).valid?({ foo: 1 }), 'Version 1 should validate'
+        
+        assert AttributeType.version(2).valid?({ foo: 1, bar: 42 }), 'Version 2 should validate'
+        refute AttributeType.version(1).valid?({ foo: 1, bar: 42 }), 'bar should only exist in version 2'
+      end
+    end
+  end
+end