Note: The practice of foam insulating the attic has raised eyebrows in the building industry because "standard" roofing techniques call for the attic to be ventilated; however, in a vented attic situation it will become approximately 130 degrees in the summer. There's no reason for an air-conditioning and vent-ductwork to have to work in that type of severe conditions. By applying Icynene right on the underside of the roof deck, the severe temperatures no longer exist in the attic. In short, the attic is now a "conditioned" space of the house that is just as comfortable as any other room in the home. This is called a "Compact Roof", which means you can frame right up against it. The one drawback of using expanded foam on the inside of the roof is that this will cause the temperature of the shingles to rise, but how much is not yet known. And how much damage a rise in temperatures could cause is debatable.
I don't know if it's common or uncommon, but installing a layer of rigid foam above your roof sheathing in order to interrupt thermal bridging is always a good idea, and is preferable in all respects to a roof assembly without any rigid foam above the sheathing. If you decide to install rigid foam above the roof sheathing, you shouldn't use closed-cell spray foam under the sheathing; either use open-cell spray foam (which is vapor-permeable) or a fluffy insulation like cellulose, mineral wool, or fiberglass (any of which will allow the roof sheathing to dry to the interior).
This is more common with closed cell foam, but it happens with open cell foam, too. Since closed cell foam has a higher R-value per inch, installers generally spray 2" in walls and 3" in rooflines to meet the energy code requirements of R-13 and R-19, respectively. (I'm not going to dive into the energy code here, but these numbers apply to many climate zones, the latter being allowed under the UA tradeoffs rule. See the Energy Nerd's blog on this topic if you want to argue.)