Files
netmaker/migrate/migrate_test.go
T
Abhishek Kondur edda2868fc NM-163: Users, Groups, Roles, Networks and Hosts Table Migration (#3910)
* 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>
2026-03-17 19:36:52 +05:30

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()
}
}