opal · Feb 4, 2014 · Feb 4, 2014
Showing with 44 additions and 2 deletions.

+1 −1 opal/browser/dom/element/attributes.rb

+1 −1 spec/dom/element/attributes_spec.rb

+42 −0 spec/dom/element_spec.rb
diff --git a/opal/browser/dom/element/attributes.rb b/opal/browser/dom/element/attributes.rb
@@ -65,7 +65,7 @@ def merge!(hash)
     self
   end
 
-  alias set []
+  alias set []=
 end
 
 end; end; end
diff --git a/spec/dom/element/attributes_spec.rb b/spec/dom/element/attributes_spec.rb
@@ -19,7 +19,7 @@
     end
   end
 
-  describe '#[]' do
+  describe '#[]=' do
     it 'sets an attribute' do
       $document[:lol].attributes[:a] = :foo
 

diff --git a/spec/dom/element_spec.rb b/spec/dom/element_spec.rb
@@ -114,4 +114,46 @@
       expect(el.inspect).to match(/: div.omg!.lol.wut>/)
     end
   end
+
+  describe '#[]' do
+    html <<-HTML
+      <div id="lol" class="name" for="hue"></div>
+    HTML
+
+    it 'gets an attribute' do
+      expect($document[:lol][:id]).to eq(:lol)
+    end
+
+    it 'gets the class attribute' do
+      expect($document[:lol][:class]).to eq(:name)
+    end
+
+    it 'gets the for attribute' do
+      expect($document[:lol][:for]).to eq(:hue)
+    end
+  end
+
+  describe '#[]=' do
+    html <<-HTML
+      <div id="lol" class="name" for="hue"></div>
+    HTML
+
+    it 'sets an attribute' do
+      $document[:lol][:a] = :foo
+
+      expect($document[:lol][:a]).to eq(:foo)
+    end
+
+    it 'sets the class attribute' do
+      $document[:lol][:class] = :bar
+
+      expect($document[:lol][:class]).to eq(:bar)
+    end
+
+    it 'sets the for attribute' do
+      $document[:lol][:for] = :baz
+
+      expect($document[:lol][:for]).to eq(:baz)
+    end
+  end
 end