Allow string write concern keys

lib/moped/session.rb

@@ -172,12 +172,12 @@ def logout
     # Setup validation of allowed write concern options.
     #
     # @since 2.0.0
-    option(:write).allow(w: Optionable.any(Integer))
-    option(:write).allow(w: Optionable.any(String))
-    option(:write).allow(j: true)
-    option(:write).allow(j: false)
-    option(:write).allow(fsync: true)
-    option(:write).allow(fsync: false)
+    option(:write).allow({ w: Optionable.any(Integer) }, { "w" => Optionable.any(Integer) })
+    option(:write).allow({ w: Optionable.any(String) }, { "w" => Optionable.any(String) })
+    option(:write).allow({ j: true }, { "j" => true })
+    option(:write).allow({ j: false }, { "j" => false })
+    option(:write).allow({ fsync: true }, { "fsync" => true })
+    option(:write).allow({ fsync: false }, { "fsync" => false })
 
     # Setup validation of allowed read preference options.
     #

lib/moped/write_concern.rb

@@ -22,7 +22,8 @@ module WriteConcern
     #
     # @since 2.0.0
     def get(value)
-      if value[:w] == 0 || value[:w] == -1
+      propagate = value[:w] || value["w"]
+      if propagate == 0 || propagate == -1
         Unverified.new
       else
         Propagate.new(value)

spec/moped/session_spec.rb

@@ -294,12 +294,26 @@
 
     context "when a write option is provided" do
 
-      let(:unverified) do
-        described_class.new([ "127.0.0.1:27017" ], write: { w: 0 })
+      context "when the option is a symbol" do
+
+        let(:unverified) do
+          described_class.new([ "127.0.0.1:27017" ], write: { w: 0 })
+        end
+
+        it "returns the corresponding write concern" do
+          expect(unverified.write_concern).to be_a(Moped::WriteConcern::Unverified)
+        end
       end
 
-      it "returns the corresponding write concern" do
-        expect(unverified.write_concern).to be_a(Moped::WriteConcern::Unverified)
+      context "when the option is a string" do
+
+        let(:unverified) do
+          described_class.new([ "127.0.0.1:27017" ], write: { "w" => 0 })
+        end
+
+        it "returns the corresponding write concern" do
+          expect(unverified.write_concern).to be_a(Moped::WriteConcern::Unverified)
+        end
       end
     end
 

spec/moped/write_concern_spec.rb

@@ -18,7 +18,7 @@
     context "when provided -1" do
 
       let(:concern) do
-        described_class.get(w: -1)
+        described_class.get("w" => -1)
       end
 
       it "returns an unverified write concern" do