mirror of
				https://codeberg.org/forgejo/forgejo.git
				synced 2025-11-04 00:11:04 +00:00 
			
		
		
		
	[GITEA] improve unit test for caching
- Configure `setting.CacheService.TTL` which will force the code to go trough the caching mechanism. - Remove the TODO and uncomment the test code. (cherry picked from commit5ccdff0b5d) (cherry picked from commit5fe0484984)
This commit is contained in:
		
					parent
					
						
							
								510b7467d3
							
						
					
				
			
			
				commit
				
					
						a201f2f189
					
				
			
		
					 1 changed files with 37 additions and 39 deletions
				
			
		
							
								
								
									
										76
									
								
								modules/cache/cache_test.go
									
										
									
									
										vendored
									
									
								
							
							
						
						
									
										76
									
								
								modules/cache/cache_test.go
									
										
									
									
										vendored
									
									
								
							| 
						 | 
				
			
			@ -18,6 +18,7 @@ func createTestCache() {
 | 
			
		|||
		Adapter: "memory",
 | 
			
		||||
		TTL:     time.Minute,
 | 
			
		||||
	})
 | 
			
		||||
	setting.CacheService.TTL = 30
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
func TestNewContext(t *testing.T) {
 | 
			
		||||
| 
						 | 
				
			
			@ -54,12 +55,12 @@ func TestGetString(t *testing.T) {
 | 
			
		|||
	assert.NoError(t, err)
 | 
			
		||||
	assert.Equal(t, "", data)
 | 
			
		||||
 | 
			
		||||
	// data, err = GetString("key", func() (string, error) {
 | 
			
		||||
	// 	return "some data", nil
 | 
			
		||||
	// })
 | 
			
		||||
	// assert.NoError(t, err)
 | 
			
		||||
	// assert.Equal(t, "", data)
 | 
			
		||||
	// Remove("key")
 | 
			
		||||
	data, err = GetString("key", func() (string, error) {
 | 
			
		||||
		return "some data", nil
 | 
			
		||||
	})
 | 
			
		||||
	assert.NoError(t, err)
 | 
			
		||||
	assert.Equal(t, "", data)
 | 
			
		||||
	Remove("key")
 | 
			
		||||
 | 
			
		||||
	data, err = GetString("key", func() (string, error) {
 | 
			
		||||
		return "some data", nil
 | 
			
		||||
| 
						 | 
				
			
			@ -67,13 +68,12 @@ func TestGetString(t *testing.T) {
 | 
			
		|||
	assert.NoError(t, err)
 | 
			
		||||
	assert.Equal(t, "some data", data)
 | 
			
		||||
 | 
			
		||||
	// data, err = GetString("key", func() (string, error) {
 | 
			
		||||
	// 	return "", fmt.Errorf("some error")
 | 
			
		||||
	// })
 | 
			
		||||
	// assert.NoError(t, err)
 | 
			
		||||
	// assert.Equal(t, "some data", data)
 | 
			
		||||
 | 
			
		||||
	// TODO: uncommented code works in IDE but not with go test
 | 
			
		||||
	data, err = GetString("key", func() (string, error) {
 | 
			
		||||
		return "", fmt.Errorf("some error")
 | 
			
		||||
	})
 | 
			
		||||
	assert.NoError(t, err)
 | 
			
		||||
	assert.Equal(t, "some data", data)
 | 
			
		||||
	Remove("key")
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
func TestGetInt(t *testing.T) {
 | 
			
		||||
| 
						 | 
				
			
			@ -91,12 +91,12 @@ func TestGetInt(t *testing.T) {
 | 
			
		|||
	assert.NoError(t, err)
 | 
			
		||||
	assert.Equal(t, 0, data)
 | 
			
		||||
 | 
			
		||||
	// data, err = GetInt("key", func() (int, error) {
 | 
			
		||||
	// 	return 100, nil
 | 
			
		||||
	// })
 | 
			
		||||
	// assert.NoError(t, err)
 | 
			
		||||
	// assert.Equal(t, 0, data)
 | 
			
		||||
	// Remove("key")
 | 
			
		||||
	data, err = GetInt("key", func() (int, error) {
 | 
			
		||||
		return 100, nil
 | 
			
		||||
	})
 | 
			
		||||
	assert.NoError(t, err)
 | 
			
		||||
	assert.Equal(t, 0, data)
 | 
			
		||||
	Remove("key")
 | 
			
		||||
 | 
			
		||||
	data, err = GetInt("key", func() (int, error) {
 | 
			
		||||
		return 100, nil
 | 
			
		||||
| 
						 | 
				
			
			@ -104,13 +104,12 @@ func TestGetInt(t *testing.T) {
 | 
			
		|||
	assert.NoError(t, err)
 | 
			
		||||
	assert.Equal(t, 100, data)
 | 
			
		||||
 | 
			
		||||
	// data, err = GetInt("key", func() (int, error) {
 | 
			
		||||
	// 	return 0, fmt.Errorf("some error")
 | 
			
		||||
	// })
 | 
			
		||||
	// assert.NoError(t, err)
 | 
			
		||||
	// assert.Equal(t, 100, data)
 | 
			
		||||
 | 
			
		||||
	// TODO: uncommented code works in IDE but not with go test
 | 
			
		||||
	data, err = GetInt("key", func() (int, error) {
 | 
			
		||||
		return 0, fmt.Errorf("some error")
 | 
			
		||||
	})
 | 
			
		||||
	assert.NoError(t, err)
 | 
			
		||||
	assert.Equal(t, 100, data)
 | 
			
		||||
	Remove("key")
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
func TestGetInt64(t *testing.T) {
 | 
			
		||||
| 
						 | 
				
			
			@ -128,12 +127,12 @@ func TestGetInt64(t *testing.T) {
 | 
			
		|||
	assert.NoError(t, err)
 | 
			
		||||
	assert.EqualValues(t, 0, data)
 | 
			
		||||
 | 
			
		||||
	// data, err = GetInt64("key", func() (int64, error) {
 | 
			
		||||
	// 	return 100, nil
 | 
			
		||||
	// })
 | 
			
		||||
	// assert.NoError(t, err)
 | 
			
		||||
	// assert.EqualValues(t, 0, data)
 | 
			
		||||
	// Remove("key")
 | 
			
		||||
	data, err = GetInt64("key", func() (int64, error) {
 | 
			
		||||
		return 100, nil
 | 
			
		||||
	})
 | 
			
		||||
	assert.NoError(t, err)
 | 
			
		||||
	assert.EqualValues(t, 0, data)
 | 
			
		||||
	Remove("key")
 | 
			
		||||
 | 
			
		||||
	data, err = GetInt64("key", func() (int64, error) {
 | 
			
		||||
		return 100, nil
 | 
			
		||||
| 
						 | 
				
			
			@ -141,11 +140,10 @@ func TestGetInt64(t *testing.T) {
 | 
			
		|||
	assert.NoError(t, err)
 | 
			
		||||
	assert.EqualValues(t, 100, data)
 | 
			
		||||
 | 
			
		||||
	// data, err = GetInt64("key", func() (int, error) {
 | 
			
		||||
	// 	return 0, fmt.Errorf("some error")
 | 
			
		||||
	// })
 | 
			
		||||
	// assert.NoError(t, err)
 | 
			
		||||
	// assert.EqualValues(t, 100, data)
 | 
			
		||||
 | 
			
		||||
	// TODO: uncommented code works in IDE but not with go test
 | 
			
		||||
	data, err = GetInt64("key", func() (int64, error) {
 | 
			
		||||
		return 0, fmt.Errorf("some error")
 | 
			
		||||
	})
 | 
			
		||||
	assert.NoError(t, err)
 | 
			
		||||
	assert.EqualValues(t, 100, data)
 | 
			
		||||
	Remove("key")
 | 
			
		||||
}
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue