Add tests for dataservices scopes

Makefile

@@ -310,6 +310,7 @@ SPEC_HELPER_MIN_SPECS = \
 	spec/lib/carto/styles/presenters/cartocss_spec.rb \
 	spec/lib/carto/forms_definition_spec.rb \
 	spec/lib/carto/form_spec.rb \
+	spec/lib/carto/oauth_provider/scopes_spec.rb \
 	spec/models/carto/legend_spec.rb \
 	spec/requests/carto/api/legends_controller_spec.rb \
 	spec/lib/carto/legend_definition_validator_spec.rb \

spec/lib/carto/oauth_provider/scopes_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper_min'
+require 'carto/oauth_provider/scopes'
+
+describe Carto::OauthProvider::Scopes do
+  describe Carto::OauthProvider::Scopes::DataservicesScope do
+    describe '#add_to_api_key_grants' do
+      let(:scope) { Carto::OauthProvider::Scopes::DataservicesScope.new('geocoding', 'GC') }
+
+      it 'adds SQL api and dataservice' do
+        grants = [{ type: 'apis', apis: []}]
+        scope.add_to_api_key_grants(grants)
+        expect(grants).to(eq([{ type: 'apis', apis: ['sql']}, { type: 'dataservices', services: ['geocoding']}]))
+      end
+
+      it 'does not add duplicate SQL api' do
+        grants = [{ type: 'apis', apis: ['sql']}]
+        scope.add_to_api_key_grants(grants)
+        expect(grants).to(include({ type: 'apis', apis: ['sql']}))
+      end
+    end
+  end
+end