Generally, it's best not to pass an error class to `expect { }.not_to raise_error`, because if an error is raised, but it is not a `SyntaxError`, this expectation will pass. Really, it shouldn't raise any kind of error, right?  Given that, and the fact that you already have an expectation in there, I think I'd remove the `raise_error` expectation entirely.