When clone_with_suffix for level copy the level_concept_difficulty too

[dmcavoy]

When we clone_with_suffix on a level (which is what we use when we do a script copy) well will now also copy the level concept difficulty over. This will help with making sure that we don't lose level_concept_difficulty tagging between years of courses.

I also removed the one-off script we used to do this copy this year since we should no longer need it.

Links

https://codedotorg.atlassian.net/browse/PLAT-1075

Testing story

Added a test to make sure the level concept difficulty is copied to the new level

[dmcavoy]

@Hamms I don't think your suggestion works because it looks like assign_attributes only works on properties. The test fails with error. I'm going to put it back to how it was

[Hamms]

Oh, I think you need to use a string rather than a symbol. See

code-dot-org/dashboard/app/models/levels/level.rb

Lines 158 to 172 in 545cf40

	# Rails won't natively assign one-to-one association attributes for
	# us, even though we've specified accepts_nested_attributes_for above.
	# So, we must do it manually.
	def assign_attributes(new_attributes)
	attributes = new_attributes.stringify_keys
	concept_difficulty_attributes = attributes.delete('level_concept_difficulty')
	if concept_difficulty_attributes
	assign_nested_attributes_for_one_to_one_association(
	:level_concept_difficulty,
	concept_difficulty_attributes
	)
	end
	super(attributes)
	end

[dmcavoy]

That looks like it works for something written like this:

    level.assign_attributes(
      'level_concept_difficulty' => {'debugging' => 5}
    )

Which I think is harder to get to here. So I'm going to merge it as is.