uploads_controller_spec.rb 7.55 KB
Newer Older
Douwe Maan's avatar
Douwe Maan committed
1 2 3
require('spec_helper')

describe Projects::UploadsController do
4
  let(:project) { create(:empty_project) }
Douwe Maan's avatar
Douwe Maan committed
5 6 7 8
  let(:user)    { create(:user) }
  let(:jpg)     { fixture_file_upload(Rails.root + 'spec/fixtures/rails_sample.jpg', 'image/jpg') }
  let(:txt)     { fixture_file_upload(Rails.root + 'spec/fixtures/doc_sample.txt', 'text/plain') }

9
  describe "POST #create" do
Douwe Maan's avatar
Douwe Maan committed
10 11 12 13 14 15
    before do
      sign_in(user)
      project.team << [user, :developer]
    end

    context "without params['file']" do
16
      it "returns an error" do
17
        post :create,
18
          namespace_id: project.namespace.to_param,
19
          project_id: project.to_param,
20
          format: :json
21
        expect(response).to have_http_status(422)
Douwe Maan's avatar
Douwe Maan committed
22 23 24 25 26 27
      end
    end

    context 'with valid image' do
      before do
        post :create,
Douwe Maan's avatar
Douwe Maan committed
28
          namespace_id: project.namespace.to_param,
Douwe Maan's avatar
Douwe Maan committed
29 30 31 32 33 34 35
          project_id: project.to_param,
          file: jpg,
          format: :json
      end

      it 'returns a content with original filename, new link, and correct type.' do
        expect(response.body).to match '\"alt\":\"rails_sample\"'
36
        expect(response.body).to match "\"url\":\"/uploads"
Douwe Maan's avatar
Douwe Maan committed
37 38 39 40 41
      end
    end

    context 'with valid non-image file' do
      before do
42
        post :create,
43
          namespace_id: project.namespace.to_param,
44 45
          project_id: project.to_param,
          file: txt,
46
          format: :json
Douwe Maan's avatar
Douwe Maan committed
47 48 49 50
      end

      it 'returns a content with original filename, new link, and correct type.' do
        expect(response.body).to match '\"alt\":\"doc_sample.txt\"'
51
        expect(response.body).to match "\"url\":\"/uploads"
Douwe Maan's avatar
Douwe Maan committed
52 53 54
      end
    end
  end
Douwe Maan's avatar
Douwe Maan committed
55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79

  describe "GET #show" do
    let(:go) do
      get :show,
        namespace_id: project.namespace.to_param,
        project_id:   project.to_param,
        secret:       "123456",
        filename:     "image.jpg"
    end

    context "when the project is public" do
      before do
        project.update_attribute(:visibility_level, Project::PUBLIC)
      end

      context "when not signed in" do
        context "when the file exists" do
          before do
            allow_any_instance_of(FileUploader).to receive(:file).and_return(jpg)
            allow(jpg).to receive(:exists?).and_return(true)
          end

          it "responds with status 200" do
            go

80
            expect(response).to have_http_status(200)
Douwe Maan's avatar
Douwe Maan committed
81 82 83 84 85 86 87
          end
        end

        context "when the file doesn't exist" do
          it "responds with status 404" do
            go

88
            expect(response).to have_http_status(404)
Douwe Maan's avatar
Douwe Maan committed
89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106
          end
        end
      end

      context "when signed in" do
        before do
          sign_in(user)
        end

        context "when the file exists" do
          before do
            allow_any_instance_of(FileUploader).to receive(:file).and_return(jpg)
            allow(jpg).to receive(:exists?).and_return(true)
          end

          it "responds with status 200" do
            go

107
            expect(response).to have_http_status(200)
Douwe Maan's avatar
Douwe Maan committed
108 109 110 111 112 113 114
          end
        end

        context "when the file doesn't exist" do
          it "responds with status 404" do
            go

115
            expect(response).to have_http_status(404)
Douwe Maan's avatar
Douwe Maan committed
116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140
          end
        end
      end
    end

    context "when the project is private" do
      before do
        project.update_attribute(:visibility_level, Project::PRIVATE)
      end

      context "when not signed in" do
        context "when the file exists" do
          before do
            allow_any_instance_of(FileUploader).to receive(:file).and_return(jpg)
            allow(jpg).to receive(:exists?).and_return(true)
          end

          context "when the file is an image" do
            before do
              allow_any_instance_of(FileUploader).to receive(:image?).and_return(true)
            end

            it "responds with status 200" do
              go

141
              expect(response).to have_http_status(200)
Douwe Maan's avatar
Douwe Maan committed
142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 192
            end
          end

          context "when the file is not an image" do
            it "redirects to the sign in page" do
              go

              expect(response).to redirect_to(new_user_session_path)
            end
          end
        end

        context "when the file doesn't exist" do
          it "redirects to the sign in page" do
            go

            expect(response).to redirect_to(new_user_session_path)
          end
        end
      end

      context "when signed in" do
        before do
          sign_in(user)
        end

        context "when the user has access to the project" do
          before do
            project.team << [user, :master]
          end

          context "when the user is blocked" do
            before do
              user.block
              project.team << [user, :master]
            end

            context "when the file exists" do
              before do
                allow_any_instance_of(FileUploader).to receive(:file).and_return(jpg)
                allow(jpg).to receive(:exists?).and_return(true)
              end

              context "when the file is an image" do
                before do
                  allow_any_instance_of(FileUploader).to receive(:image?).and_return(true)
                end

                it "responds with status 200" do
                  go

193
                  expect(response).to have_http_status(200)
Douwe Maan's avatar
Douwe Maan committed
194 195 196 197 198 199 200 201 202 203 204 205 206 207 208 209 210 211 212 213 214 215 216 217 218 219 220 221 222 223 224
                end
              end

              context "when the file is not an image" do
                it "redirects to the sign in page" do
                  go

                  expect(response).to redirect_to(new_user_session_path)
                end
              end
            end

            context "when the file doesn't exist" do
              it "redirects to the sign in page" do
                go

                expect(response).to redirect_to(new_user_session_path)
              end
            end
          end

          context "when the user isn't blocked" do
            context "when the file exists" do
              before do
                allow_any_instance_of(FileUploader).to receive(:file).and_return(jpg)
                allow(jpg).to receive(:exists?).and_return(true)
              end

              it "responds with status 200" do
                go

225
                expect(response).to have_http_status(200)
Douwe Maan's avatar
Douwe Maan committed
226 227 228 229 230 231 232
              end
            end

            context "when the file doesn't exist" do
              it "responds with status 404" do
                go

233
                expect(response).to have_http_status(404)
Douwe Maan's avatar
Douwe Maan committed
234 235 236 237 238 239 240 241 242 243 244 245 246 247 248 249 250 251 252 253
              end
            end
          end
        end

        context "when the user doesn't have access to the project" do
          context "when the file exists" do
            before do
              allow_any_instance_of(FileUploader).to receive(:file).and_return(jpg)
              allow(jpg).to receive(:exists?).and_return(true)
            end

            context "when the file is an image" do
              before do
                allow_any_instance_of(FileUploader).to receive(:image?).and_return(true)
              end

              it "responds with status 200" do
                go

254
                expect(response).to have_http_status(200)
Douwe Maan's avatar
Douwe Maan committed
255 256 257 258 259 260 261
              end
            end

            context "when the file is not an image" do
              it "responds with status 404" do
                go

262
                expect(response).to have_http_status(404)
Douwe Maan's avatar
Douwe Maan committed
263 264 265 266 267 268 269 270
              end
            end
          end

          context "when the file doesn't exist" do
            it "responds with status 404" do
              go

271
              expect(response).to have_http_status(404)
Douwe Maan's avatar
Douwe Maan committed
272 273 274 275 276 277
            end
          end
        end
      end
    end
  end
Douwe Maan's avatar
Douwe Maan committed
278
end