Commit a3b3da72 authored by Stan Hu's avatar Stan Hu

Merge branch 'bw-fix-api-routing-specs' into 'master'

Fix spec for routes to the GraphqlController

See merge request gitlab-org/gitlab-ce!26231
parents 81e63951 a481c54c
...@@ -7,25 +7,17 @@ describe 'api', 'routing' do ...@@ -7,25 +7,17 @@ describe 'api', 'routing' do
end end
it 'does not route to the GraphqlController' do it 'does not route to the GraphqlController' do
expect(get('/api/graphql')).not_to route_to('graphql#execute') expect(post('/api/graphql')).not_to route_to('graphql#execute')
end
it 'does not expose graphiql' do
expect(get('/-/graphql-explorer')).not_to route_to('graphiql/rails/editors#show')
end end
end end
context 'when graphql is disabled' do context 'when graphql is enabled' do
before do before do
stub_feature_flags(graphql: true) stub_feature_flags(graphql: true)
end end
it 'routes to the GraphqlController' do it 'routes to the GraphqlController' do
expect(get('/api/graphql')).not_to route_to('graphql#execute') expect(post('/api/graphql')).to route_to('graphql#execute')
end
it 'exposes graphiql' do
expect(get('/-/graphql-explorer')).not_to route_to('graphiql/rails/editors#show')
end end
end end
end end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment