Provided that the capacitors are in working order, and not just bits of wire vanishing into a ceramic or plastic blob, they should be OK. I would, however, add another across the motor terminals. All capacitors should have their leads as short as possible, this to stop the capacitor lead acting as a transmitter aerial.
The actual value of the capacitors, within very broad limits, is fairly unimportant – what does matter is the type, which needs very good RF performance and high DC resistance. Fortunately, disc ceramics are among the cheapest components available.
For what is worth, I've never used capacitor to case – I hold to the view that if the single one across the motor terminals doesn't do the job, I need to pay attention to the motor by cleaning the comm and possibly doing something with the brushes. On that occasion, mending the motor with a new one worked best.
Did I mention keeping the capacitor leads short?
If the motor leads are more than a few cm long, twist the pair. The idea is that any interference that is being radiated by the wires will be equal and opposite, cancelling out. Unless one of the tag to case capacitors is duff. In which case the supression arrangement becomes an interference enhancer.