Another way, which will test whether macros are enabled, code in the right place etc.:
Put a break point at the first line of the BeforeSave macro (put your cursor on any one of the lines, including the very first line (but it has to be on or before the Cancel = True), and press F9 on the keyboard).
Save the file. At this point it should stop and highlight the line where it has stopped.
In the vbe, click the Reset button (a small square). At this point the file should save.
Remove the break point (or just close the file).
Test.