Ritual checks what the trashed card cost, not what it costs. That is to say, the card cost before it was trashed is what matters. This is inconsistent with all other such cards, like Apprentice or Salvager, but it's what the card says and it's confirmed by Donald.
So if you have Quarry in play (your Inherited Estate costs $0) and you buy Ritual and trash an Inherited Estate, you should get 0 VP tokens. But currently you get 2 VP tokens, based on the fact that the Estate in trash is not an Action card anymore.
The other thing of course is that the cost of the Inherited Estate is not correct visually. I was looking at the known card bugs and it says there that this is purely visual and the gameplay cost is correct. It links to a thread where Ingix is actually saying that the gameplay cost is also wrong, so this is confusing. But I assume it means that the gameplay cost was corrected at some point (but not the visual cost).