[willcosgrove]

Currently strict_ivars will produce incorrect code if it encounters an embedded variable node:

"#@foo"

will become

"#(defined?(@foo) ? @foo : (::Kernel.raise(::StrictIvars::NameError.new(self, :@foo))))"
# ^ no { so no interpolation

Here's the situation where I noticed this was happening:

I added test cases, and a visitor for the EmbeddedVariableNode. I also added an early return if the variable is already in context. But it's not strictly required, it just seemed like a nice thing to do 🤷‍♂️

[joeldrapper]

This looks great. We need to copy this into empirical too.