Do not depend on universe when checking parent in `StringPath`

1 files changed, 12 insertions, 3 deletions

diff --git a/lib/gitlab/string_path.rb b/lib/gitlab/string_path.rb
index 9948502e8ea..4d024b3ff73 100644
--- a/lib/gitlab/string_path.rb
+++ b/lib/gitlab/string_path.rb
@@ -7,6 +7,7 @@ module Gitlab
   #
   class StringPath
     attr_reader :path, :universe
+    attr_accessor :name
 
     def initialize(path, universe, metadata = [])
       @path = sanitize(path)
@@ -35,7 +36,7 @@ module Gitlab
     end
 
     def has_parent?
-      @universe.include?(@path.sub(basename, ''))
+      nodes > 1
     end
 
     def parent
@@ -48,7 +49,7 @@ module Gitlab
     end
 
     def name
-      @path.split(::File::SEPARATOR).last
+      @name || @path.split(::File::SEPARATOR).last
     end
 
     def has_descendants?
@@ -75,7 +76,11 @@ module Gitlab
     end
 
     def directories!
-      @path =~ %r{^\./[^/]/} ? directories.prepend(parent) : directories
+      return directories unless has_parent? && directory?
+
+      dotted_parent = parent
+      dotted_parent.name = '..'
+      directories.prepend(dotted_parent)
     end
 
     def files
@@ -88,6 +93,10 @@ module Gitlab
       @metadata[index]
     end
 
+    def nodes
+      @path.count('/') + (file? ? 1 : 0)
+    end
+
     def ==(other)
       @path == other.path && @universe == other.universe
     end