Commit a062656e authored by Vamsi Vempati's avatar Vamsi Vempati Committed by Phil Hughes

Removed fetching unused properties for subscription

- users was not used anywhere for addon purchase
- guests used to be used by new subscription but we no longer need it
parent 9798b19e
......@@ -3,7 +3,6 @@ query State {
id
accountId
name
users
}
activeSubscription @client {
name
......
......@@ -65,8 +65,6 @@ module SubscriptionsHelper
id: namespace.id,
account_id: account_id,
name: namespace.name,
users: namespace.member_count,
guests: namespace.guest_count,
full_path: namespace.full_path
}
end
......
......@@ -33,8 +33,8 @@ export const mockStoragePlans = [
];
export const mockNamespaces = `
[{"id":132,"accountId":"${accountId}","name":"Gitlab Org","users":3},
{"id":483,"accountId":null,"name":"Gnuwget","users":12}]
[{"id":132,"accountId":"${accountId}","name":"Gitlab Org"},
{"id":483,"accountId":null,"name":"Gnuwget"}]
`;
export const mockParsedNamespaces = JSON.parse(mockNamespaces).map((namespace) => ({
......
......@@ -51,7 +51,7 @@ RSpec.describe SubscriptionsHelper do
it { is_expected.to include(plan_id: 'bronze_id') }
it { is_expected.to include(namespace_id: group.id.to_s) }
it { is_expected.to include(source: 'some_source') }
it { is_expected.to include(group_data: %Q{[{"id":#{group.id},"account_id":null,"name":"My Namespace","users":2,"guests":1,"full_path":"my_namespace"}]}) }
it { is_expected.to include(group_data: %Q{[{"id":#{group.id},"account_id":null,"name":"My Namespace","full_path":"my_namespace"}]}) }
it { is_expected.to include(trial: 'false') }
it { is_expected.to include(new_trial_registration_path: '/-/trial_registrations/new') }
......@@ -163,7 +163,7 @@ RSpec.describe SubscriptionsHelper do
it { is_expected.to include(namespace_id: group.id.to_s) }
it { is_expected.to include(active_subscription: active_subscription) }
it { is_expected.to include(source: 'some_source') }
it { is_expected.to include(group_data: %Q{[{"id":#{group.id},"account_id":"#{account_id}","name":"My Namespace","users":1,"guests":0,"full_path":"my_namespace"}]}) }
it { is_expected.to include(group_data: %Q{[{"id":#{group.id},"account_id":"#{account_id}","name":"My Namespace","full_path":"my_namespace"}]}) }
it { is_expected.to include(redirect_after_success: group_usage_quotas_path(group, anchor: anchor, purchased_product: purchased_product)) }
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