Merge pull request #14812 from jntullo/bug/miq_group_migration_fix

Fix Migration Spec

spec/migrations/20150904181202_miq_groups_add_ldap_role_spec.rb

@@ -1,19 +1,21 @@
 require_migration
 
 describe MiqGroupsAddLdapRole do
-  let(:miq_user_role_stub) { migration_stub(:MiqUserRole) }
-  let(:miq_group_stub)     { migration_stub(:MiqGroup) }
+  migration_context :up do
+    let(:miq_user_role_stub) { migration_stub(:MiqUserRole) }
+    let(:miq_group_stub) { migration_stub(:MiqGroup) }
 
-  it "migrates ldap groups" do
-    role = miq_user_role_stub.create!("EvmRole-my_group")
-    unchanged_group = miq_group_stub.create!(:description => "EvmGroup-super_administrator", :group_type => "abc")
-    changed_group = miq_group_stub.create!(:description => "EvmGroup-my_group", :group_type => "ldap")
+    it "migrates ldap groups" do
+      role = miq_user_role_stub.create!(:name => "EvmRole-my_group")
+      unchanged_group = miq_group_stub.create!(:description => "EvmGroup-super_administrator", :group_type => "abc")
+      changed_group = miq_group_stub.create!(:description => "EvmGroup-my_group", :group_type => "ldap")
 
-    migrate
-    changed_group.reload
+      migrate
+      changed_group.reload
 
-    expect(unchanged_group.reload.group_type).to eq("abc")
-    expect(changed_group.group_type).to eq("system")
-    expect(changed_group.role).to eq(role)
+      expect(unchanged_group.reload.group_type).to eq("abc")
+      expect(changed_group.group_type).to eq("system")
+      expect(changed_group.miq_user_role_id).to eq(role.id)
+    end
   end
 end