Invoices kept in document with user information in no sql database is recommended?
Almost always with databases, it depends on what you're going to do with the data after you're stored it.  

How are you going to retrieve and process it? 

If you only ever access this data by Tenant, then carrying a block of "stuff" around with each Tenant "document" is probably OK.
If you wanted to start querying and comparing bills across your tenants, though, this structure might be inefficient. 


  ... won't do any computations on bills ... 


Do you not have to summarise your rental income for your annual Tax Return? 

That would mean pulling out any Current or Previous "rent" entries that fall within the last Tax Year - I suggest that might get just a bit fiddly with the data structure you have (and, although you've not shown any, the Date formats that you're using).