mirror of
				https://codeberg.org/forgejo/forgejo.git
				synced 2025-10-30 22:11:07 +00:00 
			
		
		
		
	* Create API endpoints for repo topics.
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Generate swagger
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Add documentation to functions
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Grammar fix
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Fix function comment
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Can't use FindTopics when looking for a single repo topic, as it doesnt use exact match
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Add PUT /repos/{owner}/{repo}/topics and remove GET /repos/{owner}/{repo}/topics
* Ignore if topic is sent twice in same request, refactoring.
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Fix topic dropdown with api changes.
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Style fix
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Update API documentation
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Better way to handle duplicate topics in slice
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Make response element TopicName an array of strings, instead of using an array of TopicName
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Add test cases for API Repo Topics.
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Fix format of tests
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Fix comments
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Fix unit tests after adding some more topics to the test fixture.
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Update models/topic.go
Limit multiple if else if ...
Co-Authored-By: Antoine GIRARD <sapk@users.noreply.github.com>
* Engine as first parameter in function
Co-Authored-By: Antoine GIRARD <sapk@users.noreply.github.com>
* Replace magic numbers with http status code constants.
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Fix variable scope
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Test one read with login and one with token
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Add some more tests
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Apply suggestions from code review
Use empty struct for efficiency
Co-Authored-By: Lauris BH <lauris@nix.lv>
* Add test case to check access for user with write access
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Fix access, repo admin required to change topics
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Correct first test to be without token
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
* Any repo reader should be able to access topics.
* No need for string pointer
Signed-off-by: David Svantesson <davidsvantesson@gmail.com>
		
	
			
		
			
				
	
	
		
			77 lines
		
	
	
	
		
			2 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
			
		
		
	
	
			77 lines
		
	
	
	
		
			2 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
| // Copyright 2018 The Gitea Authors. All rights reserved.
 | |
| // Use of this source code is governed by a MIT-style
 | |
| // license that can be found in the LICENSE file.
 | |
| 
 | |
| package models
 | |
| 
 | |
| import (
 | |
| 	"testing"
 | |
| 
 | |
| 	"github.com/stretchr/testify/assert"
 | |
| )
 | |
| 
 | |
| func TestAddTopic(t *testing.T) {
 | |
| 	totalNrOfTopics := 6
 | |
| 	repo1NrOfTopics := 3
 | |
| 	repo2NrOfTopics := 2
 | |
| 
 | |
| 	assert.NoError(t, PrepareTestDatabase())
 | |
| 
 | |
| 	topics, err := FindTopics(&FindTopicOptions{})
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.EqualValues(t, totalNrOfTopics, len(topics))
 | |
| 
 | |
| 	topics, err = FindTopics(&FindTopicOptions{
 | |
| 		Limit: 2,
 | |
| 	})
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.EqualValues(t, 2, len(topics))
 | |
| 
 | |
| 	topics, err = FindTopics(&FindTopicOptions{
 | |
| 		RepoID: 1,
 | |
| 	})
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.EqualValues(t, repo1NrOfTopics, len(topics))
 | |
| 
 | |
| 	assert.NoError(t, SaveTopics(2, "golang"))
 | |
| 	repo2NrOfTopics = 1
 | |
| 	topics, err = FindTopics(&FindTopicOptions{})
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.EqualValues(t, totalNrOfTopics, len(topics))
 | |
| 
 | |
| 	topics, err = FindTopics(&FindTopicOptions{
 | |
| 		RepoID: 2,
 | |
| 	})
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.EqualValues(t, repo2NrOfTopics, len(topics))
 | |
| 
 | |
| 	assert.NoError(t, SaveTopics(2, "golang", "gitea"))
 | |
| 	repo2NrOfTopics = 2
 | |
| 	totalNrOfTopics++
 | |
| 	topic, err := GetTopicByName("gitea")
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.EqualValues(t, 1, topic.RepoCount)
 | |
| 
 | |
| 	topics, err = FindTopics(&FindTopicOptions{})
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.EqualValues(t, totalNrOfTopics, len(topics))
 | |
| 
 | |
| 	topics, err = FindTopics(&FindTopicOptions{
 | |
| 		RepoID: 2,
 | |
| 	})
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.EqualValues(t, repo2NrOfTopics, len(topics))
 | |
| }
 | |
| 
 | |
| func TestTopicValidator(t *testing.T) {
 | |
| 	assert.True(t, ValidateTopic("12345"))
 | |
| 	assert.True(t, ValidateTopic("2-test"))
 | |
| 	assert.True(t, ValidateTopic("test-3"))
 | |
| 	assert.True(t, ValidateTopic("first"))
 | |
| 	assert.True(t, ValidateTopic("second-test-topic"))
 | |
| 	assert.True(t, ValidateTopic("third-project-topic-with-max-length"))
 | |
| 
 | |
| 	assert.False(t, ValidateTopic("$fourth-test,topic"))
 | |
| 	assert.False(t, ValidateTopic("-fifth-test-topic"))
 | |
| 	assert.False(t, ValidateTopic("sixth-go-project-topic-with-excess-length"))
 | |
| }
 |