yes the switch needs to be grounded [normally by mounting], but if the light is on all the time that means it's getting a ground when it shouldn't - if you have a meter, disconnect the wire from the GRD terminal on the switch - if the light stays on you might have nicked that wire when fishing it up to the light and it's shorted to ground along the way - use the ohmmeter to track down the unwanted ground
Bill