{"action":"created","issue":{"url":"https://api.github.com/repos/fakefs/fakefs/issues/332","repository_url":"https://api.github.com/repos/fakefs/fakefs","labels_url":"https://api.github.com/repos/fakefs/fakefs/issues/332/labels{/name}","comments_url":"https://api.github.com/repos/fakefs/fakefs/issues/332/comments","events_url":"https://api.github.com/repos/fakefs/fakefs/issues/332/events","html_url":"https://github.com/fakefs/fakefs/issues/332","id":129998965,"number":332,"title":"Dir.glob does not handle brace expansions that match multiple levels in the directory hierarchy","user":{"login":"tovodeverett","id":2568894,"avatar_url":"https://avatars.githubusercontent.com/u/2568894?v=3","gravatar_id":"","url":"https://api.github.com/users/tovodeverett","html_url":"https://github.com/tovodeverett","followers_url":"https://api.github.com/users/tovodeverett/followers","following_url":"https://api.github.com/users/tovodeverett/following{/other_user}","gists_url":"https://api.github.com/users/tovodeverett/gists{/gist_id}","starred_url":"https://api.github.com/users/tovodeverett/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tovodeverett/subscriptions","organizations_url":"https://api.github.com/users/tovodeverett/orgs","repos_url":"https://api.github.com/users/tovodeverett/repos","events_url":"https://api.github.com/users/tovodeverett/events{/privacy}","received_events_url":"https://api.github.com/users/tovodeverett/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":10,"created_at":"2016-01-30T15:28:57Z","updated_at":"2016-07-06T04:09:44Z","closed_at":null,"body":"I am replicating https://github.com/fakefs/fakefs/pull/330#issuecomment-176953806 authored by @dtognazzini here to track it as a separate issue:\r\n\r\n\r\n> @tovodeverett Thanks for driving these changes!\r\n> \r\n> I found another case that isn't supported by `Dir.glob`, added to `FakeFSTest`:\r\n> ```\r\n>   def test_dir_glob_handles_recursive_globs_bash_brace\r\n>     FileUtils.mkdir_p '/one/two/three'\r\n>     File.open('/zero.rb', 'w')\r\n>     File.open('/one/two/three/four.rb', 'w')\r\n>     File.open('/one/five.rb', 'w')\r\n>     assert_equal ['/zero.rb', '/one/five.rb'], Dir['{,*,*/*}.rb']\r\n>   end\r\n> ```\r\n> Should pass, but fails with:\r\n> ```\r\n> Minitest::Assertion: Expected: [\"/zero.rb\", \"/one/five.rb\"]\r\n>   Actual: [\"/zero.rb\"]\r\n> ```\r\n> Bundler uses a glob pattern like this looking for gemspecs.\r\n"},"comment":{"url":"https://api.github.com/repos/fakefs/fakefs/issues/comments/230666670","html_url":"https://github.com/fakefs/fakefs/issues/332#issuecomment-230666670","issue_url":"https://api.github.com/repos/fakefs/fakefs/issues/332","id":230666670,"user":{"login":"tovodeverett","id":2568894,"avatar_url":"https://avatars.githubusercontent.com/u/2568894?v=3","gravatar_id":"","url":"https://api.github.com/users/tovodeverett","html_url":"https://github.com/tovodeverett","followers_url":"https://api.github.com/users/tovodeverett/followers","following_url":"https://api.github.com/users/tovodeverett/following{/other_user}","gists_url":"https://api.github.com/users/tovodeverett/gists{/gist_id}","starred_url":"https://api.github.com/users/tovodeverett/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tovodeverett/subscriptions","organizations_url":"https://api.github.com/users/tovodeverett/orgs","repos_url":"https://api.github.com/users/tovodeverett/repos","events_url":"https://api.github.com/users/tovodeverett/events{/privacy}","received_events_url":"https://api.github.com/users/tovodeverett/received_events","type":"User","site_admin":false},"created_at":"2016-07-06T04:09:44Z","updated_at":"2016-07-06T04:09:44Z","body":"I have identified what I believe to be a bug in both the implementation and the test suite.  Can anyone confirm that https://github.com/fakefs/fakefs/blob/master/test/fakefs_test.rb#L1120 is incorrect?  I believe that it should read:\r\n```\r\n    assert_equal ['/path/bar', '/path/bar2/baz', '/path/foo'], Dir['{/nowhere,/path/{foo,{bar,bar2/baz}}}']\r\n```\r\nThis is similar to the issue @dtognazzini reported originally, although this is simpler."}}
{"actor":{"display_login":"tovodeverett"}}