diff options
author | Thong Kuah <tkuah@gitlab.com> | 2018-12-04 21:11:48 +1300 |
---|---|---|
committer | Thong Kuah <tkuah@gitlab.com> | 2018-12-05 10:16:44 +1300 |
commit | 885ea5c33c8e99935e04b6f929f0bafcf3f8eed8 (patch) | |
tree | 71e7899e73be1ae1152723b80b7c43ee4c8c407e /Gemfile.rails4 | |
parent | 6c642c087bef3b7925ca5c8cf93078b58a8efe98 (diff) | |
download | gitlab-ce-885ea5c33c8e99935e04b6f929f0bafcf3f8eed8.tar.gz |
Make subgroup specs :nested_groups
Subgroups are not supported in mySQL. I changed Namespace#root_ancestor
to return from self_and_ancestors as a bugfix.
```
184 # Returns all the ancestors of the current namespaces
185 def ancestors
186 return self.class.none unless parent_id
187
188 Gitlab::GroupHierarchy
189 .new(self.class.where(id: parent_id))
190 .base_and_ancestors
191 end
```
So it seems like on mySQL we accidentally returned the parent group :
```
ancestors = self.class.where(id: parent_id)
ancestors.reorder(nil).find_by(parent_id: nil)
```
Project#root_namespace is used only by shared_runners_limit_namespace
and all the tests for shared_runner_minutes_on_root_namespace are only
enabled on `:nested_groups`
`when :shared_runner_minutes_on_root_namespace is enabled', :nested_groups`
We very clearly state in https://docs.gitlab.com/ee/user/group/subgroups/ that `
Nested groups are only supported when you use PostgreSQL`, so I think I
will fix forward and add `:nested_groups` to the two failing feature
specs.
Diffstat (limited to 'Gemfile.rails4')
0 files changed, 0 insertions, 0 deletions