mirror of
https://github.com/gravitl/netmaker.git
synced 2026-04-22 16:07:11 +08:00
edda2868fc
* feat(go): add user schema; * feat(go): migrate to user schema; * feat(go): add audit fields; * feat(go): remove unused fields from the network model; * feat(go): add network schema; * feat(go): migrate to network schema; * refactor(go): add comment to clarify migration logic; * fix(go): test failures; * fix(go): test failures; * feat(go): change membership table to store memberships at all scopes; * feat(go): add schema for access grants; * feat(go): remove nameservers from new networks table; ensure db passed for schema functions; * feat(go): set max conns for sqlite to 1; * fix(go): issues updating user account status; * refactor(go): remove converters and access grants; * refactor(go): add json tags in schema models; * refactor(go): rename file to migrate_v1_6_0.go; * refactor(go): add user groups and user roles tables; use schema tables; * refactor(go): inline get and list from schema package; * refactor(go): inline get network and list users from schema package; * fix(go): staticcheck issues; * fix(go): remove test not in use; fix test case; * fix(go): validate network; * fix(go): resolve static checks; * fix(go): new models errors; * fix(go): test errors; * fix(go): handle no records; * fix(go): add validations for user object; * fix(go): set correct extclient status; * fix(go): test error; * feat(go): make schema the base package; * feat(go): add host schema; * feat(go): use schema host everywhere; * feat(go): inline get host, list hosts and delete host; * feat(go): use non-ptr value; * feat(go): use save to upsert all fields; * feat(go): use save to upsert all fields; * feat(go): save turn endpoint as string; * feat(go): check for gorm error record not found; * fix(go): test failures; * fix(go): update all network fields; * fix(go): update all network fields; * feat(go): add paginated list networks api; * feat(go): add paginated list users api; * feat(go): add paginated list hosts api; * feat(go): add pagination to list groups api; * fix(go): comment; * fix(go): implement marshal and unmarshal text for custom types; * fix(go): implement marshal and unmarshal json for custom types; * fix(go): just use the old model for unmarshalling; * fix(go): implement marshal and unmarshal json for custom types; * feat(go): remove paginated list networks api; * feat(go): use custom paginated response object; * fix(go): ensure default values for page and per_page are used when not passed; * fix(go): rename v1.6.0 to v1.5.1; * fix(go): check for gorm.ErrRecordNotFound instead of database.IsEmptyRecord; * fix(go): use host id, not pending host id; * feat(go): add filters to paginated apis; * feat(go): add filters to paginated apis; * feat(go): remove check for max username length; * feat(go): add filters to count as well; * feat(go): use library to check email address validity; * feat(go): ignore pagination if params not passed; * fix(go): pagination issues; * fix(go): check exists before using; * fix(go): remove debug log; * fix(go): use gorm err record not found; * fix(go): use gorm err record not found; * fix(go): use user principal name when creating pending user; * fix(go): use schema package for consts; * fix(go): prevent disabling superadmin user; Co-authored-by: tenki-reviewer[bot] <262613592+tenki-reviewer[bot]@users.noreply.github.com> * fix(go): swap is admin and is superadmin; Co-authored-by: tenki-reviewer[bot] <262613592+tenki-reviewer[bot]@users.noreply.github.com> * fix(go): remove dead code block; https://github.com/gravitl/netmaker/pull/3910#discussion_r2928837937 * fix(go): incorrect message when trying to disable self; https://github.com/gravitl/netmaker/pull/3910#discussion_r2928837934 * fix(go): use correct header; Co-authored-by: tenki-reviewer[bot] <262613592+tenki-reviewer[bot]@users.noreply.github.com> * fix(go): return after error response; Co-authored-by: tenki-reviewer[bot] <262613592+tenki-reviewer[bot]@users.noreply.github.com> * fix(go): use correct order of params; https://github.com/gravitl/netmaker/pull/3910#discussion_r2929593036 * fix(go): set default values for page and page size; use v2 instead of /list; * Update logic/auth.go Co-authored-by: tenki-reviewer[bot] <262613592+tenki-reviewer[bot]@users.noreply.github.com> * Update schema/user_roles.go Co-authored-by: tenki-reviewer[bot] <262613592+tenki-reviewer[bot]@users.noreply.github.com> * fix(go): syntax error; * fix(go): set default values when page and per_page are not passed or 0; * fix(go): use uuid.parse instead of uuid.must parse; * fix(go): review errors; * fix(go): review errors; * Update controllers/user.go Co-authored-by: tenki-reviewer[bot] <262613592+tenki-reviewer[bot]@users.noreply.github.com> * Update controllers/user.go Co-authored-by: tenki-reviewer[bot] <262613592+tenki-reviewer[bot]@users.noreply.github.com> * NM-163: fix errors: * Update db/types/options.go Co-authored-by: tenki-reviewer[bot] <262613592+tenki-reviewer[bot]@users.noreply.github.com> * fix(go): persist return user in event; * Update db/types/options.go Co-authored-by: tenki-reviewer[bot] <262613592+tenki-reviewer[bot]@users.noreply.github.com> * NM-163: duplicate lines of code * NM-163: fix(go): fix missing return and filter parsing in user controller - Add missing return after error response in updateUserAccountStatus to prevent double-response and spurious ext-client side-effects - Use switch statements in listUsers to skip unrecognized account_status and mfa_status filter values * fix(go): check for both min and max page size; * fix(go): enclose transfer superadmin in transaction; * fix(go): review errors; * fix(go): remove free tier checks; * fix(go): review fixes; --------- Co-authored-by: VishalDalwadi <dalwadivishal26@gmail.com> Co-authored-by: Vishal Dalwadi <51291657+VishalDalwadi@users.noreply.github.com> Co-authored-by: tenki-reviewer[bot] <262613592+tenki-reviewer[bot]@users.noreply.github.com>
213 lines
6.1 KiB
Go
213 lines
6.1 KiB
Go
// Package migrate provides tests for migration functions, including
|
|
// performance tests for large-scale user migrations.
|
|
//
|
|
// To run the large-scale user performance tests:
|
|
//
|
|
// go test -v ./migrate -run TestSyncUsersLargeScale
|
|
// go test -v ./migrate -run TestMigrateToUUIDsLargeScale
|
|
//
|
|
// To run all tests (excluding large-scale):
|
|
//
|
|
// go test -v ./migrate -short
|
|
//
|
|
// To run benchmarks:
|
|
//
|
|
// go test -bench=. ./migrate
|
|
package migrate
|
|
|
|
import (
|
|
"context"
|
|
"testing"
|
|
"time"
|
|
|
|
"github.com/google/uuid"
|
|
"github.com/gravitl/netmaker/database"
|
|
"github.com/gravitl/netmaker/db"
|
|
"github.com/gravitl/netmaker/logic"
|
|
"github.com/gravitl/netmaker/schema"
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
"gorm.io/datatypes"
|
|
)
|
|
|
|
// TestSyncUsersLargeScale tests syncUsers() with a large number of users
|
|
// to verify performance improvements
|
|
func TestSyncUsersLargeScale(t *testing.T) {
|
|
// Skip if short test flag is set (allows quick test runs)
|
|
if testing.Short() {
|
|
t.Skip("Skipping large scale test in short mode")
|
|
}
|
|
|
|
// Initialize test database
|
|
err := db.InitializeDB(schema.ListModels()...)
|
|
require.NoError(t, err)
|
|
defer db.CloseDB()
|
|
|
|
err = database.InitializeDatabase()
|
|
require.NoError(t, err)
|
|
defer database.CloseDB()
|
|
|
|
// Create test users with various roles
|
|
numUsers := 1000
|
|
t.Logf("Creating %d test users...", numUsers)
|
|
startCreate := time.Now()
|
|
|
|
for i := 0; i < numUsers; i++ {
|
|
user := schema.User{
|
|
Username: "testuser" + uuid.New().String()[:8],
|
|
Password: "testpassword123",
|
|
DisplayName: "Test User " + uuid.New().String()[:8],
|
|
PlatformRoleID: schema.ServiceUser, // Most are service users
|
|
}
|
|
|
|
// Assign different platform roles
|
|
if i%100 == 0 {
|
|
user.PlatformRoleID = schema.SuperAdminRole
|
|
} else if i%10 == 0 {
|
|
user.PlatformRoleID = schema.AdminRole
|
|
} else if i%5 == 0 {
|
|
user.PlatformRoleID = schema.PlatformUser
|
|
}
|
|
|
|
// Some users have user groups
|
|
if i%4 == 0 {
|
|
user.UserGroups = datatypes.NewJSONType(make(map[schema.UserGroupID]struct{}))
|
|
}
|
|
|
|
err := logic.UpsertUser(user)
|
|
require.NoError(t, err, "Failed to create user %d", i)
|
|
}
|
|
|
|
createDuration := time.Since(startCreate)
|
|
t.Logf("Created %d users in %v (avg: %v per user)", numUsers, createDuration, createDuration/time.Duration(numUsers))
|
|
|
|
// Verify users were created
|
|
users, err := (&schema.User{}).ListAll(db.WithContext(context.TODO()))
|
|
require.NoError(t, err)
|
|
assert.GreaterOrEqual(t, len(users), numUsers, "Expected at least %d users", numUsers)
|
|
|
|
// Test syncUsers() performance
|
|
t.Log("Running syncUsers() migration...")
|
|
startSync := time.Now()
|
|
syncUsers()
|
|
syncDuration := time.Since(startSync)
|
|
|
|
t.Logf("syncUsers() completed in %v for %d users (avg: %v per user)",
|
|
syncDuration, len(users), syncDuration/time.Duration(len(users)))
|
|
|
|
// Verify users were migrated correctly
|
|
usersAfter, err := (&schema.User{}).ListAll(db.WithContext(context.TODO()))
|
|
require.NoError(t, err)
|
|
assert.Equal(t, len(users), len(usersAfter), "User count should remain the same")
|
|
|
|
// Verify migration correctness - check a sample of users
|
|
sampleSize := 100
|
|
if len(usersAfter) < sampleSize {
|
|
sampleSize = len(usersAfter)
|
|
}
|
|
|
|
for i := 0; i < sampleSize; i++ {
|
|
user := usersAfter[i]
|
|
|
|
// Verify platform role is set
|
|
assert.NotEmpty(t, user.PlatformRoleID.String(), "User %s should have PlatformRoleID", user.Username)
|
|
|
|
// Verify user groups are initialized
|
|
assert.NotNil(t, user.UserGroups, "User should have UserGroups map")
|
|
}
|
|
|
|
// Performance assertion - syncUsers should complete in reasonable time
|
|
// With optimizations, 1000 users should complete in under 10 seconds
|
|
maxDuration := 30 * time.Second
|
|
assert.Less(t, syncDuration, maxDuration,
|
|
"syncUsers() took too long: %v (expected < %v)", syncDuration, maxDuration)
|
|
|
|
t.Logf("✓ syncUsers() performance test passed: %v for %d users", syncDuration, len(users))
|
|
}
|
|
|
|
// TestMigrateToUUIDsLargeScale tests MigrateToUUIDs() with a large number of users
|
|
func TestMigrateToUUIDsLargeScale(t *testing.T) {
|
|
// Skip if short test flag is set
|
|
if testing.Short() {
|
|
t.Skip("Skipping large scale test in short mode")
|
|
}
|
|
|
|
// Initialize test database
|
|
err := db.InitializeDB(schema.ListModels()...)
|
|
require.NoError(t, err)
|
|
defer db.CloseDB()
|
|
|
|
err = database.InitializeDatabase()
|
|
require.NoError(t, err)
|
|
defer database.CloseDB()
|
|
|
|
// Create test users with user groups (needed for UUID migration)
|
|
numUsers := 1000
|
|
t.Logf("Creating %d test users with user groups...", numUsers)
|
|
startCreate := time.Now()
|
|
|
|
for i := 0; i < numUsers; i++ {
|
|
user := schema.User{
|
|
Username: "testuser" + uuid.New().String()[:8],
|
|
Password: "testpassword123",
|
|
DisplayName: "Test User " + uuid.New().String()[:8],
|
|
PlatformRoleID: schema.ServiceUser,
|
|
UserGroups: datatypes.NewJSONType(make(map[schema.UserGroupID]struct{})),
|
|
}
|
|
|
|
// Add some user groups with non-UUID IDs (to trigger migration)
|
|
if i%2 == 0 {
|
|
user.UserGroups.Data()[("old-group-1")] = struct{}{}
|
|
}
|
|
if i%3 == 0 {
|
|
user.UserGroups.Data()[("old-group-2")] = struct{}{}
|
|
}
|
|
|
|
err := logic.UpsertUser(user)
|
|
require.NoError(t, err, "Failed to create user %d", i)
|
|
}
|
|
|
|
createDuration := time.Since(startCreate)
|
|
t.Logf("Created %d users in %v", numUsers, createDuration)
|
|
|
|
// Verify users were created
|
|
users, err := (&schema.User{}).ListAll(db.WithContext(context.TODO()))
|
|
require.NoError(t, err)
|
|
assert.GreaterOrEqual(t, len(users), numUsers, "Expected at least %d users", numUsers)
|
|
}
|
|
|
|
// BenchmarkSyncUsers benchmarks syncUsers() performance
|
|
func BenchmarkSyncUsers(b *testing.B) {
|
|
// Initialize test database
|
|
err := db.InitializeDB(schema.ListModels()...)
|
|
if err != nil {
|
|
b.Fatal(err)
|
|
}
|
|
defer db.CloseDB()
|
|
|
|
err = database.InitializeDatabase()
|
|
if err != nil {
|
|
b.Fatal(err)
|
|
}
|
|
defer database.CloseDB()
|
|
|
|
// Create test users
|
|
numUsers := 1000
|
|
for i := 0; i < numUsers; i++ {
|
|
user := schema.User{
|
|
Username: "benchuser" + uuid.New().String()[:8],
|
|
Password: "password",
|
|
PlatformRoleID: schema.ServiceUser,
|
|
}
|
|
if i%10 == 0 {
|
|
user.PlatformRoleID = schema.AdminRole
|
|
}
|
|
_ = logic.UpsertUser(user)
|
|
}
|
|
|
|
b.ResetTimer()
|
|
for i := 0; i < b.N; i++ {
|
|
syncUsers()
|
|
}
|
|
}
|