Yeah, the way we think about rails now is not how they thought about rails then.
rails can be really flexible, but you can also build some really fragile code if you're not careful.
I tend to write a lot of small service classes organized by module. module FEATURE class ACTION. So you want to pull out the code so when you go to test the thing, you only have to test input to output.
I've never been a fan of trying to parse erb files. So much ugly surrounding the pretty. I dont actually see on what line check_tank is even invoked. A search through givess me the title of one document and the method but no calls.