Add explicit status test, feedback:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7333#note_20058993 https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7333#note_20059060 https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7333#note_20059357
This commit is contained in:
		
							parent
							
								
									b4a7e7cfbf
								
							
						
					
					
						commit
						d9000184e5
					
				|  | @ -426,6 +426,7 @@ describe Ci::Pipeline, models: true do | |||
| 
 | ||||
|       it 'returns the latest status for the same ref and different sha' do | ||||
|         expect(latest_status).to eq(described_class.latest.status) | ||||
|         expect(latest_status).to eq('failed') | ||||
|       end | ||||
|     end | ||||
| 
 | ||||
|  | @ -434,6 +435,7 @@ describe Ci::Pipeline, models: true do | |||
| 
 | ||||
|       it 'returns the latest status for ref and different sha' do | ||||
|         expect(latest_status).to eq(described_class.latest_status('ref')) | ||||
|         expect(latest_status).to eq('failed') | ||||
|       end | ||||
|     end | ||||
|   end | ||||
|  |  | |||
|  | @ -225,6 +225,7 @@ eos | |||
| 
 | ||||
|       it 'gives compound status from latest pipelines' do | ||||
|         expect(commit.status).to eq(Ci::Pipeline.latest_status) | ||||
|         expect(commit.status).to eq('pending') | ||||
|       end | ||||
|     end | ||||
| 
 | ||||
|  | @ -252,6 +253,7 @@ eos | |||
| 
 | ||||
|       it 'gives compound status from latest pipelines if ref is nil' do | ||||
|         expect(commit.status(nil)).to eq(Ci::Pipeline.latest_status) | ||||
|         expect(commit.status(nil)).to eq('failed') | ||||
|       end | ||||
|     end | ||||
|   end | ||||
|  |  | |||
		Loading…
	
		Reference in New Issue