What happened is correct. When checking it seems you fell into a trap that I also fall into from time to time: A mismatch between the baseline from theoretical consideration to practical experiment.
I reloaded game 98156893 as well, turn 4. For the 3 pictures, you may need to scroll the posting horizontally, but I wanted to capture the whole state, from point totals on the left to log on the right, in one screen shot each.
Here is the baseline of the theoretcial consideration:
Before you start playing Bishop, you have 5 VP, including 0 VP token:
After you play the Bishop, but before you select what to trash, the game already executes the part that is supposed to happen before the trash, giving you +$1 and +1 VP token:
After you select the Estate to trash, the game does the rest, it gives you the additional +1VP token, but adjusts your overall VP count for the now missing Estate:
The error I am sometimes making, and which I suspect you did as well, was to disregard the first picture when actually playing, and only compare the second and third one, which does not show an increase in overall VP. Which is correct, for the time period considered
, which is after you already got the the initial +1VP token from Bishop