summaryrefslogtreecommitdiff
path: root/spec
diff options
context:
space:
mode:
authorGitLab Bot <gitlab-bot@gitlab.com>2020-02-18 03:08:54 +0000
committerGitLab Bot <gitlab-bot@gitlab.com>2020-02-18 03:08:54 +0000
commit5ee120f46740efac7b8a460d7a92e4da82f4fb0b (patch)
treeb44d3bef04e9db472913289e6b53e58a14cb3e61 /spec
parent72721699f11187199e89631ce0b5e3d2f7c167e9 (diff)
downloadgitlab-ce-5ee120f46740efac7b8a460d7a92e4da82f4fb0b.tar.gz
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'spec')
-rw-r--r--spec/controllers/oauth/token_info_controller_spec.rb71
-rw-r--r--spec/frontend/monitoring/components/charts/time_series_spec.js5
-rw-r--r--spec/lib/gitlab/database_importers/self_monitoring/project/create_service_spec.rb8
3 files changed, 84 insertions, 0 deletions
diff --git a/spec/controllers/oauth/token_info_controller_spec.rb b/spec/controllers/oauth/token_info_controller_spec.rb
new file mode 100644
index 00000000000..35ad0dcf98c
--- /dev/null
+++ b/spec/controllers/oauth/token_info_controller_spec.rb
@@ -0,0 +1,71 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe Oauth::TokenInfoController do
+ describe '#show' do
+ context 'when the user is not authenticated' do
+ it 'responds with a 401' do
+ get :show
+
+ expect(response.status).to eq 401
+ expect(JSON.parse(response.body)).to include('error' => 'invalid_request')
+ end
+ end
+
+ context 'when the request is valid' do
+ let(:application) { create(:oauth_application, scopes: 'api') }
+ let(:access_token) do
+ create(:oauth_access_token, expires_in: 5.minutes, application: application)
+ end
+
+ it 'responds with the token info' do
+ get :show, params: { access_token: access_token.token }
+
+ expect(response.status).to eq 200
+ expect(JSON.parse(response.body)).to eq(
+ 'scope' => %w[api],
+ 'scopes' => %w[api],
+ 'created_at' => access_token.created_at.to_i,
+ 'expires_in' => access_token.expires_in,
+ 'application' => { 'uid' => application.uid },
+ 'resource_owner_id' => access_token.resource_owner_id,
+ 'expires_in_seconds' => access_token.expires_in
+ )
+ end
+ end
+
+ context 'when the doorkeeper_token is not recognised' do
+ it 'responds with a 401' do
+ get :show, params: { access_token: 'unknown_token' }
+
+ expect(response.status).to eq 401
+ expect(JSON.parse(response.body)).to include('error' => 'invalid_request')
+ end
+ end
+
+ context 'when the token is expired' do
+ let(:access_token) do
+ create(:oauth_access_token, created_at: 2.days.ago, expires_in: 10.minutes)
+ end
+
+ it 'responds with a 401' do
+ get :show, params: { access_token: access_token.token }
+
+ expect(response.status).to eq 401
+ expect(JSON.parse(response.body)).to include('error' => 'invalid_request')
+ end
+ end
+
+ context 'when the token is revoked' do
+ let(:access_token) { create(:oauth_access_token, revoked_at: 2.days.ago) }
+
+ it 'responds with a 401' do
+ get :show, params: { access_token: access_token.token }
+
+ expect(response.status).to eq 401
+ expect(JSON.parse(response.body)).to include('error' => 'invalid_request')
+ end
+ end
+ end
+end
diff --git a/spec/frontend/monitoring/components/charts/time_series_spec.js b/spec/frontend/monitoring/components/charts/time_series_spec.js
index 4871619c85a..cfd812002a4 100644
--- a/spec/frontend/monitoring/components/charts/time_series_spec.js
+++ b/spec/frontend/monitoring/components/charts/time_series_spec.js
@@ -3,6 +3,7 @@ import { setTestTimeout } from 'helpers/timeout';
import { GlLink } from '@gitlab/ui';
import { GlAreaChart, GlLineChart, GlChartSeriesLabel } from '@gitlab/ui/dist/charts';
import { shallowWrapperContainsSlotText } from 'helpers/vue_test_utils_helper';
+import { chartColorValues } from '~/monitoring/constants';
import { createStore } from '~/monitoring/stores';
import TimeSeries from '~/monitoring/components/charts/time_series.vue';
import * as types from '~/monitoring/stores/mutation_types';
@@ -315,6 +316,10 @@ describe('Time series component', () => {
it('formats line width correctly', () => {
expect(chartData[0].lineStyle.width).toBe(2);
});
+
+ it('formats line color correctly', () => {
+ expect(chartData[0].lineStyle.color).toBe(chartColorValues[0]);
+ });
});
describe('chartOptions', () => {
diff --git a/spec/lib/gitlab/database_importers/self_monitoring/project/create_service_spec.rb b/spec/lib/gitlab/database_importers/self_monitoring/project/create_service_spec.rb
index d643a2df46b..0b8f64e97a1 100644
--- a/spec/lib/gitlab/database_importers/self_monitoring/project/create_service_spec.rb
+++ b/spec/lib/gitlab/database_importers/self_monitoring/project/create_service_spec.rb
@@ -76,6 +76,14 @@ describe Gitlab::DatabaseImporters::SelfMonitoring::Project::CreateService do
it_behaves_like 'has prometheus service', 'http://localhost:9090'
+ it 'is idempotent' do
+ result1 = subject.execute
+ expect(result1[:status]).to eq(:success)
+
+ result2 = subject.execute
+ expect(result2[:status]).to eq(:success)
+ end
+
it "tracks successful install" do
expect(::Gitlab::Tracking).to receive(:event).twice
expect(::Gitlab::Tracking).to receive(:event).with('self_monitoring', 'project_created')